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

Some tests for Spectrum viewer #2220

Merged
merged 3 commits into from
Jun 7, 2024
Merged

Some tests for Spectrum viewer #2220

merged 3 commits into from
Jun 7, 2024

Conversation

samtygier-stfc
Copy link
Collaborator

Issue

Work on #2032

Description

Some refactoring and tests for SpectrumROI
Tests for handle_roi_clicked

Testing

Adds some tests

Acceptance Criteria

Open spectrum viewer with data
Add some ROIs
Check that clicking on ROIs updates coords in the properties
Check that the same ROI is showing when moving back and forth between ROIs and Image tabs

Documentation

Not needed

samtygier-stfc and others added 3 commits June 6, 2024 16:51
Co-authored-by: JackEAllen <jack.allen@stfc.ac.uk>
Co-authored-by: Mike Sullivan <michael.sullivan@stfc.ac.uk>
Co-authored-by: JackEAllen <jack.allen@stfc.ac.uk>
Co-authored-by: Mike Sullivan <michael.sullivan@stfc.ac.uk>
Co-authored-by: JackEAllen <jack.allen@stfc.ac.uk>
Co-authored-by: Mike Sullivan <michael.sullivan@stfc.ac.uk>
@samtygier-stfc samtygier-stfc marked this pull request as ready for review June 6, 2024 16:05
@coveralls
Copy link

Coverage Status

coverage: 72.745% (+0.03%) from 72.713%
when pulling d0f137a on 2032-spec-viewer-tests
into 937bb36 on main.

Copy link
Collaborator

@ashmeigh ashmeigh left a comment

Choose a reason for hiding this comment

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

The acceptance criteria were met. I ran the new tests which confirmed the spectrum viewer operates as expected with data. Interactions with ROIs are seamless, clicking on ROIs updates coordinates appropriately, and navigating between ROI and Image tabs consistently displays the same ROI. The code changes was also fine.

@samtygier-stfc samtygier-stfc added this pull request to the merge queue Jun 7, 2024
Merged via the queue into main with commit 4492304 Jun 7, 2024
8 checks passed
@samtygier-stfc samtygier-stfc deleted the 2032-spec-viewer-tests branch June 7, 2024 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants