tests: remove duplicate test cases in transforms#8949
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughDuplicate parameterized test cases were removed from several transform test lists. The remaining cases and expected shapes or validity flags were unchanged. Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
5a6a5ff to
20fb430
Compare
Fixes Project-MONAI#8942 Removed byte-identical duplicate entries from test case lists in transforms. Files changed: - tests/transforms/test_border_pad.py - tests/transforms/test_border_padd.py - tests/transforms/test_center_spatial_crop.py - tests/transforms/test_spatial_padd.py Signed-off-by: bhardwajjvaibhav <bhardwajvaibhav2097@gmail.com> Signed-off-by: bhardwajjvaibhav <bhardwajvaibhav2097@gmail.com>
feab14e to
bb90398
Compare
|
Sorry for not tagging the issue, I made the PR before it was created but I think this has now been solved with #8896 (merged), which removed these same duplicate entries from all four files. |
Fixes #8942
Removed duplicate entries from test case lists in the transforms test files.
Files changed:
Each file had byte-identical duplicate entries in their test case lists
as identified by the script in issue #8942. Duplicates were removed and
all affected tests verified passing locally.