|
|
|
@ -102,17 +102,16 @@ class TestDatasetServiceUpdateDataset:
|
|
|
|
patch("services.dataset_service.DatasetService.get_dataset") as mock_get_dataset,
|
|
|
|
patch("services.dataset_service.DatasetService.get_dataset") as mock_get_dataset,
|
|
|
|
patch("services.dataset_service.DatasetService.check_dataset_permission") as mock_check_perm,
|
|
|
|
patch("services.dataset_service.DatasetService.check_dataset_permission") as mock_check_perm,
|
|
|
|
patch("extensions.ext_database.db.session") as mock_db,
|
|
|
|
patch("extensions.ext_database.db.session") as mock_db,
|
|
|
|
patch("services.dataset_service.datetime") as mock_datetime,
|
|
|
|
patch("services.dataset_service.naive_utc_now") as mock_naive_utc_now,
|
|
|
|
):
|
|
|
|
):
|
|
|
|
current_time = datetime.datetime(2023, 1, 1, 12, 0, 0)
|
|
|
|
current_time = datetime.datetime(2023, 1, 1, 12, 0, 0)
|
|
|
|
mock_datetime.datetime.now.return_value = current_time
|
|
|
|
mock_naive_utc_now.return_value = current_time
|
|
|
|
mock_datetime.UTC = datetime.UTC
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
yield {
|
|
|
|
yield {
|
|
|
|
"get_dataset": mock_get_dataset,
|
|
|
|
"get_dataset": mock_get_dataset,
|
|
|
|
"check_permission": mock_check_perm,
|
|
|
|
"check_permission": mock_check_perm,
|
|
|
|
"db_session": mock_db,
|
|
|
|
"db_session": mock_db,
|
|
|
|
"datetime": mock_datetime,
|
|
|
|
"naive_utc_now": mock_naive_utc_now,
|
|
|
|
"current_time": current_time,
|
|
|
|
"current_time": current_time,
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
@ -292,7 +291,7 @@ class TestDatasetServiceUpdateDataset:
|
|
|
|
"embedding_model_provider": "openai",
|
|
|
|
"embedding_model_provider": "openai",
|
|
|
|
"embedding_model": "text-embedding-ada-002",
|
|
|
|
"embedding_model": "text-embedding-ada-002",
|
|
|
|
"updated_by": user.id,
|
|
|
|
"updated_by": user.id,
|
|
|
|
"updated_at": mock_dataset_service_dependencies["current_time"].replace(tzinfo=None),
|
|
|
|
"updated_at": mock_dataset_service_dependencies["current_time"],
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
self._assert_database_update_called(
|
|
|
|
self._assert_database_update_called(
|
|
|
|
@ -327,7 +326,7 @@ class TestDatasetServiceUpdateDataset:
|
|
|
|
"indexing_technique": "high_quality",
|
|
|
|
"indexing_technique": "high_quality",
|
|
|
|
"retrieval_model": "new_model",
|
|
|
|
"retrieval_model": "new_model",
|
|
|
|
"updated_by": user.id,
|
|
|
|
"updated_by": user.id,
|
|
|
|
"updated_at": mock_dataset_service_dependencies["current_time"].replace(tzinfo=None),
|
|
|
|
"updated_at": mock_dataset_service_dependencies["current_time"],
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
actual_call_args = mock_dataset_service_dependencies[
|
|
|
|
actual_call_args = mock_dataset_service_dependencies[
|
|
|
|
@ -365,7 +364,7 @@ class TestDatasetServiceUpdateDataset:
|
|
|
|
"collection_binding_id": None,
|
|
|
|
"collection_binding_id": None,
|
|
|
|
"retrieval_model": "new_model",
|
|
|
|
"retrieval_model": "new_model",
|
|
|
|
"updated_by": user.id,
|
|
|
|
"updated_by": user.id,
|
|
|
|
"updated_at": mock_dataset_service_dependencies["current_time"].replace(tzinfo=None),
|
|
|
|
"updated_at": mock_dataset_service_dependencies["current_time"],
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
self._assert_database_update_called(
|
|
|
|
self._assert_database_update_called(
|
|
|
|
@ -422,7 +421,7 @@ class TestDatasetServiceUpdateDataset:
|
|
|
|
"collection_binding_id": "binding-456",
|
|
|
|
"collection_binding_id": "binding-456",
|
|
|
|
"retrieval_model": "new_model",
|
|
|
|
"retrieval_model": "new_model",
|
|
|
|
"updated_by": user.id,
|
|
|
|
"updated_by": user.id,
|
|
|
|
"updated_at": mock_dataset_service_dependencies["current_time"].replace(tzinfo=None),
|
|
|
|
"updated_at": mock_dataset_service_dependencies["current_time"],
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
self._assert_database_update_called(
|
|
|
|
self._assert_database_update_called(
|
|
|
|
@ -463,7 +462,7 @@ class TestDatasetServiceUpdateDataset:
|
|
|
|
"collection_binding_id": "binding-123",
|
|
|
|
"collection_binding_id": "binding-123",
|
|
|
|
"retrieval_model": "new_model",
|
|
|
|
"retrieval_model": "new_model",
|
|
|
|
"updated_by": user.id,
|
|
|
|
"updated_by": user.id,
|
|
|
|
"updated_at": mock_dataset_service_dependencies["current_time"].replace(tzinfo=None),
|
|
|
|
"updated_at": mock_dataset_service_dependencies["current_time"],
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
self._assert_database_update_called(
|
|
|
|
self._assert_database_update_called(
|
|
|
|
@ -525,7 +524,7 @@ class TestDatasetServiceUpdateDataset:
|
|
|
|
"collection_binding_id": "binding-789",
|
|
|
|
"collection_binding_id": "binding-789",
|
|
|
|
"retrieval_model": "new_model",
|
|
|
|
"retrieval_model": "new_model",
|
|
|
|
"updated_by": user.id,
|
|
|
|
"updated_by": user.id,
|
|
|
|
"updated_at": mock_dataset_service_dependencies["current_time"].replace(tzinfo=None),
|
|
|
|
"updated_at": mock_dataset_service_dependencies["current_time"],
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
self._assert_database_update_called(
|
|
|
|
self._assert_database_update_called(
|
|
|
|
@ -568,7 +567,7 @@ class TestDatasetServiceUpdateDataset:
|
|
|
|
"collection_binding_id": "binding-123",
|
|
|
|
"collection_binding_id": "binding-123",
|
|
|
|
"retrieval_model": "new_model",
|
|
|
|
"retrieval_model": "new_model",
|
|
|
|
"updated_by": user.id,
|
|
|
|
"updated_by": user.id,
|
|
|
|
"updated_at": mock_dataset_service_dependencies["current_time"].replace(tzinfo=None),
|
|
|
|
"updated_at": mock_dataset_service_dependencies["current_time"],
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
self._assert_database_update_called(
|
|
|
|
self._assert_database_update_called(
|
|
|
|
|