Task #11094 (closed)
Unify `PreprocessorTest` classes
Reported by: | mtbcarroll | Owned by: | mtbcarroll |
---|---|---|---|
Priority: | minor | Milestone: | Cleanup |
Component: | OmeroFs | Version: | 4.4.9 |
Keywords: | n.a. | Cc: | fs@… |
Resources: | n.a. | Referenced By: | n.a. |
References: | n.a. | Remaining Time: | 0.0d |
Sprint: | n.a. |
Description
Review the original PreprocessorTest class and, before deleting it (and the Preprocessor getters only it relies on), ensure that the later PreprocessorTest class covers test the same cases adequately, extending it where warranted.
Change History (4)
comment:1 Changed 11 years ago by mtbcarroll
comment:2 Changed 10 years ago by mtbcarroll
- Status changed from new to accepted
- Version set to 4.4.9
comment:3 Changed 10 years ago by mtbcarroll
- Resolution set to fixed
- Status changed from accepted to closed
comment:4 Changed 10 years ago by Josh Moore <josh@…>
- Remaining Time set to 0
(In [222af1a004e5f392ccd2d773dc159f811fc64d80/ome.git] on branch develop) Merge pull request #1912 from mtbc/trac-11094-tidy-PreprocessorTest?
fix #11094: unify preprocessor test classes
Also, the later PreprocessorTest class can be tightened up by changing the arrays of acceptable values to be, "exactly these, in any order".