Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Strangely commented tests #1796

Open
abuts opened this issue Dec 20, 2024 · 0 comments
Open

Strangely commented tests #1796

abuts opened this issue Dec 20, 2024 · 0 comments
Assignees
@cmarooney-stfc
Labels
bug Something isn't working Technical Debt code quality, unit tests, code duplicaton

Comments

@abuts
Copy link
Member

abuts commented Dec 20, 2024

Test _test/test_instrument_files/test_IX_detectors_3 contains strange

skipTest('should be removed by merge'); lines which sitting there from 2023.

Test should be fixed or modified
@abuts abuts added bug Something isn't working Technical Debt code quality, unit tests, code duplicaton labels Dec 20, 2024
@abuts abuts changed the title Strange commented tests Strangely commented tests Dec 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cmarooney-stfc cmarooney-stfc

Labels
bug Something isn't working Technical Debt code quality, unit tests, code duplicaton
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@abuts @cmarooney-stfc