Skip to content

Update the test plan and add non-MRVA cases#2400

Merged
robertbrignull merged 5 commits intomainfrom
robertbrignull/test-plan
May 3, 2023
Merged

Update the test plan and add non-MRVA cases#2400
robertbrignull merged 5 commits intomainfrom
robertbrignull/test-plan

Conversation

@robertbrignull
Copy link
Copy Markdown
Contributor

Going through the test plan I noticed a few places with outdated information or steps.

We have also been considering adding non-MRVA test cases to this because currently it is a bit too MRVA-focussed. I've added cases for downloading databases and running a few types of local queries.

I realise this just makes the test plan longer. We could possibly remove the cases for "problem but not path-problem" queries, since they are largely covered by the path-problem case, but otherwise I don't think we can remove much without lower our coverage quite a bit.

Checklist

  • CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

@robertbrignull robertbrignull requested a review from a team May 3, 2023 14:44
@robertbrignull robertbrignull requested a review from a team as a code owner May 3, 2023 14:44
@robertbrignull robertbrignull merged commit e097bc1 into main May 3, 2023
@robertbrignull robertbrignull deleted the robertbrignull/test-plan branch May 3, 2023 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants