Closed tweak-wtf closed 5 months ago
The code seems to be ok and we can probably merge it. I can rewrite the reposition tests into the class based workflow afterwards. I wonder if we could add __get_reposition_operators
method as fixture - but it is beyond my knowledge at the moment.
was using fixtures before actually. issue was that i wanted to have a test case always test 1 instance of an operator in this case.
for Mirror2
i was running into the issue that i wanted to yield a single object from a list which was failing. there are 2 Mirror
effects in the used sample file.
then pivoted over to use parametrize again.
also this enabled me to check for different assertions in the same test when passing in an index for the used test parameters
adds tests for reposition effect
Transform
Crop
Mirror