8000 Add http api test cases support for the test framework by Ami11111 · Pull Request #1562 · infiniflow/infinity · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add http api test cases support for the test framework #1562

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 12 commits into from
Aug 1, 2024
Merged

Add http api test cases support for the test framework #1562

merged 12 commits into from
Aug 1, 2024

Conversation

Ami11111
Copy link
Contributor

What problem does this PR solve?

Now can use the python sdk test framework to test http api, using a http adapter class.
Many test cases are skipped for http api test for following reasons:

  • do not support tensor, tensor array, sparse and their index
  • create and drop table with invalid option will not raise exception
  • do not support delete() (no filter argument)
  • do not create file that not exist(when export)
  • output do not support _row_id,_score,etc
  • can not trans embedding data from int to float(when insert)
  • unsupport fulltext knn
  • Unsupport default field when creating table
  • disconnect if input vector is too big
  • exception mismatch with pysdk
  • etc

Type of change

  • Test cases

@JinHai-CN JinHai-CN added ci PR can be test and removed ci PR can be test labels Jul 31, 2024
@JinHai-CN JinHai-CN added the ci PR can be test label Jul 31, 2024
@JinHai-CN JinHai-CN merged commit bd62548 into infiniflow:main Aug 1, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ci PR can be test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Ami11111 @JinHai-CN
0