Switch to class-scoped collection names in the tests
Nobody has claimed this yet.
- Dominant language
- C#
- Stars
- 229
- Forks
- 21
- Avg merge
- 55m
- Merged PRs (30d)
- 1
Description
All tests currently use the same Qdrant collection name, and the initialization code (which runs before each test) removes all collections from Qdrant. This makes it impossible to run test classes in parallel (the default xunit behavior).
We can have class-scoped collections to allow for more parallelism.
Discussed in https://github.com/qdrant/qdrant-dotnet/pull/11#discussion_r1373175534
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by inspecting the test initialization code that runs before each test and the existing shared Qdrant collection naming. Review the discussion linked from the issue for context, then run the test suite with parallel test classes enabled. Done means test classes use separate collections and no longer interfere by deleting each other's collections.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- testing-qa
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100