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

Improve test coverage for PolarizationCorrectionsHelpers methods #38602

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yusufjimoh
Copy link
Contributor

Description of work

  • added unit tests to test splitSpinStateString
  • added unit tests to test indexOfWorkspaceForSpinState
  • added unit tests to test workSpaceForSpinState

Fixes #37749

To test:

  • review unit tests
  • tests should pass

This does not require release notes because it involves updating the unit tests

…lper methods

- added unit tests to splitSpinStateString
- added unit tests to test indexOfWorkspaceForSpinState
- added unit tests to test workSpaceForSpinState
@yusufjimoh yusufjimoh marked this pull request as ready for review January 17, 2025 09:18
@yusufjimoh yusufjimoh added Reflectometry Issues and pull requests related to reflectometry SANS Issues and pull requests related to SANS Maintenance Unassigned issues to be addressed in the next maintenance period. ISIS Team: LSS Issue and pull requests managed by the LSS subteam at ISIS labels Jan 17, 2025
@yusufjimoh yusufjimoh added this to the Release 6.13 milestone Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ISIS Team: LSS Issue and pull requests managed by the LSS subteam at ISIS Maintenance Unassigned issues to be addressed in the next maintenance period. Reflectometry Issues and pull requests related to reflectometry SANS Issues and pull requests related to SANS
Projects
Status: Review
Development

Successfully merging this pull request may close these issues.

Improve test coverage for PolarizationCorrectionsHelpers methods
1 participant