Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Async Client: GraphQL package #328

Merged
merged 7 commits into from
Nov 16, 2024

Conversation

bevzzz
Copy link
Contributor

@bevzzz bevzzz commented Nov 15, 2024

This MR adds the async client for graphql endpoint and the test suit for it.

WIP:

  • ClientGraphQLTest
  • ClusterGraphQLTest
  • ClientGraphQLMultiTenancy
  • ClientGraphQLMultiTargetSearch

@weaviate-git-bot
Copy link

Great to see you again! Thanks for the contribution.

beep boop - the Weaviate bot 👋🤖

PS:
Are you already a member of the Weaviate Slack channel?

Add Aggregate query to GraphQL.
The 'async' test suite does not include the 5 OpenAPI-related tests,
which are currently @ignored as unstable

Also skipped the last shouldSupportSearchWithContains test,
because it's monstrously big.

Extracted common fixtures and utils into an abstract class for easier reuse.
Moved testing utils into AbstractAsyncClientTest class.
@bevzzz bevzzz force-pushed the feature/async_graphql branch from af501e7 to 7e91084 Compare November 15, 2024 21:52
Fix non-cluster tests to use WeaviateDockerCompose and not WeaviateDockerComposeCluster
@bevzzz bevzzz force-pushed the feature/async_graphql branch from 2381011 to bccb993 Compare November 15, 2024 22:47
@bevzzz bevzzz marked this pull request as ready for review November 15, 2024 23:32
@antas-marcin antas-marcin merged commit 6a0dfc1 into weaviate:main Nov 16, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants