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.

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

Unit tests for Time of Flight user properties in Spectrum Viewer #2180

Merged
merged 6 commits into from
May 9, 2024

Conversation

MikeSullivan7
Copy link
Collaborator

Issue

Closes #2177

Description

Added unit tests for the Time of Flight area of the spectrum viewer

Testing

make check

Acceptance Criteria

Check that all tests pass

Documentation

Added release notes

@coveralls
Copy link

coveralls commented Apr 24, 2024

Coverage Status

coverage: 72.921% (-0.2%) from 73.161%
when pulling 32e1cfb on 2177_spectrum_viewer_tof_testing
into 45171e6 on main.

Copy link
Collaborator

@samtygier-stfc samtygier-stfc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good now. Just needs some prints removed

@samtygier-stfc samtygier-stfc added this pull request to the merge queue May 9, 2024
Merged via the queue into main with commit b44db12 May 9, 2024
8 checks passed
@samtygier-stfc samtygier-stfc deleted the 2177_spectrum_viewer_tof_testing branch May 9, 2024 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Add Unit Tests for Spectrum Viewer (ToF)
3 participants