Make sure we test issue deduplication
@shivam-Purohit is already working on this.
Since Mar 29, 2023.
- Dominant language
- Python
- Stars
- 26
- Forks
- 40
- PR merge metrics
- No merged PRs in 30d
Description
The latest database and I/O schema versions have added support for "issues" and "incidents". Out of those two "issues" required a potentially-disruptive change, because they have two columns in their primary keys: id and version. As such, the deduplication mechanism should handle them differently from other objects, i.e. allow keeping issues with the same id field value, but different version values in the database at the same time. This is different from the rest of the objects, which all can differ in their id field only.
Make sure we have tests in kcidb/test_db.py that:
load()two issues with the sameid/versionpair and check that only one ends up in the database (using either thequery()or thedump()method);load()two issues with the sameid, but differentversionfield values, and check that both are there in the database (using the same method);load()two issues with the sameversion, but differentidfield values, and check the same;load()two issues with bothidandversiondifferent, and check the same.
Either prove tests like that already exist, or make sure they don't and add your own.
Contributor guide
No contributing guide indexed for this repository
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.
Assessment
This issue has not been assessed yet.