Skip to content

Commit 28a4f5c

Browse files
committed
[ai_assistant] remove import from kbn/test-suites-xpack (elastic#228650)
## Summary I just spotted that elastic#228009 made an import from deprecated `x-pack/test` helper. It usually leads to circular dependencies issue, so I updated the import to the same helper already in solution directory and removed duplicated directory (it was temp solution to avoid circular deps when moving tests around, sorry for confusion) I moved a few oblt helpers directly to Search solution search_playground test dir, but we probably need to decouple "platform" part of it and move to `kbn/test-suites-xpack-platform` --------- Co-authored-by: kibanamachine <[email protected]> (cherry picked from commit 122cd9a) # Conflicts: # x-pack/test/api_integration/deployment_agnostic/apis/observability/ai_assistant/utils/knowledge_base.ts # x-pack/test/tsconfig.json
1 parent b8ecda8 commit 28a4f5c

File tree

20 files changed

+3
-1118
lines changed

20 files changed

+3
-1118
lines changed

.github/CODEOWNERS

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1926,7 +1926,6 @@ x-pack/test/functional/services/index.ts @elastic/appex-qa
19261926
/x-pack/platform/test/serverless @elastic/appex-qa
19271927
/x-pack/test/api_integration/deployment_agnostic @elastic/appex-qa
19281928
/x-pack/platform/test/functional/services/ml/api.ts @elastic/appex-qa # temporarily due to SKA tests relocation
1929-
/x-pack/test/api_integration/deployment_agnostic/apis/ @elastic/appex-qa # temporarily due to SKA tests relocation
19301929
/x-pack/solutions/observability/test/functional/config.base.ts @elastic/appex-qa
19311930
/x-pack/solutions/observability/test/functional/ftr_provider_context.d.ts @elastic/appex-qa
19321931
/x-pack/solutions/observability/test/functional/page_objects/index.ts @elastic/appex-qa

x-pack/solutions/observability/test/api_integration_deployment_agnostic/apis/ai_assistant/complete/functions/retrieve_elastic_doc.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,12 @@ import { ChatCompletionStreamParams } from 'openai/lib/ChatCompletionStream';
1010
import { ChatCompletionMessageParam } from 'openai/resources';
1111
import { last } from 'lodash';
1212
import { MessageAddEvent, MessageRole } from '@kbn/observability-ai-assistant-plugin/common';
13-
import { TINY_ELSER_INFERENCE_ID } from '@kbn/test-suites-xpack/api_integration/deployment_agnostic/apis/observability/ai_assistant/utils/model_and_inference';
1413
import { LlmProxy, createLlmProxy } from '../../utils/create_llm_proxy';
1514
import { chatComplete } from '../../utils/conversation';
1615
import type { DeploymentAgnosticFtrProviderContext } from '../../../../ftr_provider_context';
1716
import { installProductDoc, uninstallProductDoc } from '../../utils/product_doc_base';
1817
import {
18+
TINY_ELSER_INFERENCE_ID,
1919
deployTinyElserAndSetupKb,
2020
teardownTinyElserModelAndInferenceEndpoint,
2121
} from '../../utils/model_and_inference';

x-pack/test/api_integration/deployment_agnostic/apis/observability/ai_assistant/utils/advanced_settings.ts

Lines changed: 0 additions & 26 deletions
This file was deleted.

x-pack/test/api_integration/deployment_agnostic/apis/observability/ai_assistant/utils/alerts.ts

Lines changed: 0 additions & 81 deletions
This file was deleted.

x-pack/test/api_integration/deployment_agnostic/apis/observability/ai_assistant/utils/conversation.ts

Lines changed: 0 additions & 225 deletions
This file was deleted.

0 commit comments

Comments
 (0)