-
Notifications
You must be signed in to change notification settings - Fork 120
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
Add documentation for regression tests #2000
Conversation
Looks great to me. |
These instructions don't feel core to Smart Answers.
The former gives more detail than the latter.
ca913f5
to
c072ded
Compare
I've rebase this against master and force-pushed in preparation for merging. |
…on-tests Add documentation for regression tests
# Adding new regression tests | ||
|
||
1. Update the flow to replace any single line conditionals around `Phraselist`s with multiple line conditionals. This is so that we get useful information from the running the coverage utility. Single line conditionals will show up as having been exercised irrespective of whether they caused something to be added to the `Phraselist`. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not exactly related to this PR, but I've skimmed the codebase (using GH search) and it looks like Phraselists are nearly extinct. I think it would make sense to look at removing it as a feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. Yes, I think you're right that they have (almost?) entirely disappeared from question blocks. We already have a Trello card to remove documentation associated with them: https://trello.com/c/zODxvA3Y/78-remove-documentation-references-to-phraselists
I wanted to add a link to documentation about the regression tests from
CONTRIBUTING.md
, but realised that none of the existing documentation was really suitable or detailed enough.This is my first stab at some more comprehensive documentation about the regression tests. Feedback welcome.
/cc @chrisroos @tadast
Expected user-visible changes