-
Notifications
You must be signed in to change notification settings - Fork 27.7k
[BE] found some more tests still using unittest.main() #50483
Copy link
Copy link
Closed
Labels
better-engineeringRelatively self-contained tasks for better engineering contributorsRelatively self-contained tasks for better engineering contributorsmodule: testsIssues related to tests (not the torch.testing module)Issues related to tests (not the torch.testing module)triagedThis issue has been looked at a team member, and triaged and prioritized into an appropriate moduleThis issue has been looked at a team member, and triaged and prioritized into an appropriate module
Metadata
Metadata
Assignees
Labels
better-engineeringRelatively self-contained tasks for better engineering contributorsRelatively self-contained tasks for better engineering contributorsmodule: testsIssues related to tests (not the torch.testing module)Issues related to tests (not the torch.testing module)triagedThis issue has been looked at a team member, and triaged and prioritized into an appropriate moduleThis issue has been looked at a team member, and triaged and prioritized into an appropriate module
🚀 Feature
This is a follow up on #50448, some more python tests are still using unittest.main()
Not sure if they are intended since they are not using common_utils but here is the list.
test/custom_backend/*
test/custom_operator/*
test/mobile/*
test/onnx/*
modules/detectron/*
scripts/release_notes
in addition there are many more tests in /caffe2/ folder which I assume we will not address at the moment.
cc @mruberry @VitalyFedyunin @walterddr