208 unit testing plan - #217
Draft
frayle-ons wants to merge 3 commits into
Draft
Conversation
Contributor
|
Can I confirm this is ready for review? (If so, I'll take it off draft mode) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See #208 on Unit Test Planning for the ClassifAI package
✨ Summary
These changes create a new
testsfolder in the root project directory. Its contents contains elementary export tests for the ClassifAI package interface (test_exports.py), as well as a breakdown of potential unit testing cases for each of the core modules of ClassifAI; as of version 1.1.1. (Vectorisers, Indexers, Servers and Evaluation).Each of these modules has its own subfolder within
tests, for exampletest_indexers. Each subfolder contains 2 files:readme.mdwhich provides a broad overview of the potential unit testing for that module, including functionality testing, edge cases, input/output validation and more.file containing some code samples of implemented unit testing using Copilot that had access to thereadme.md` file of the module to design/generate the example.Each unit testing module is designed to be separated from the others. For example, the indexers module utilises Mock Vectorisers and CSV data to test VectorStore init functionality - this is the code generated with the Copilot example. Aims to ensure that unit testing can be done independently for each module.
The unit testing plan is broken down this way in relation to the 4 tickets that have been produced on the project board: #215 #213 #214 (possibly missing one for the evaluation module at time of writing). For each ticket, the current information in each of the unit test plan subfolders would provide someone working on that module an outline of what to unit test, and examples of implemented code.
📜 Changes Introduced
🔍 How to Test
Initially just running
pytestin the usual manner on command line triggers the basic export testing as well as the Copilot implemented examples .