-
Notifications
You must be signed in to change notification settings - Fork 24
Open
Labels
On HoldNot ready for active developmentNot ready for active developmentTestsNew and improved testingNew and improved testing
Description
after #1557
we will have a DeleteUpdateWhereAnalyzerTest and SelectWhereAnalyzerTest
we want to get a single test class for all analysis strategies.
task is:
- make sure DeleteUpdateWhereAnalyzerTest has all the test cases in SelectWhereAnalyzerTest
- update the parameter suppliers in DeleteUpdateWhereAnalyzerTest so it also tests the SELECT strategy with appropriate expected error
- this will need extension so the expected warnings are also passed
- delete SelectWhereAnalyzerTest and rename DeleteUpdateWhereAnalyzerTest to WhereAnalyzerTest
end goal is there is one place we go through all the combinations of filters, and the tests are told what to expect given different strategy (error ? warnings ? none ? )
Metadata
Metadata
Assignees
Labels
On HoldNot ready for active developmentNot ready for active developmentTestsNew and improved testingNew and improved testing