Skip to content

Comments

Added additional unit test to changes made to the new LOQ IDF#40881

Open
RabiyaF wants to merge 6 commits intomantidproject:mainfrom
RabiyaF:39366-add-automated-tests-for-LOQ-event-mode
Open

Added additional unit test to changes made to the new LOQ IDF#40881
RabiyaF wants to merge 6 commits intomantidproject:mainfrom
RabiyaF:39366-add-automated-tests-for-LOQ-event-mode

Conversation

@RabiyaF
Copy link
Member

@RabiyaF RabiyaF commented Feb 11, 2026

Description of work

Closes #39366 .

To test:


Reviewer

Your comments will be used as part of the gatekeeper process. Comment clearly on what you have checked and tested during your review. Provide an audit trail for any changes requested.

As per the review guidelines:

  • Is the code of an acceptable quality? (Code standards/GUI standards)
  • Has a thorough functional test been performed? Do the changes handle unexpected input/situations?
  • Are appropriately scoped unit and/or system tests provided?
  • Do the release notes conform to the guidelines and describe the changes appropriately?
  • Has the relevant (user and developer) documentation been added/updated?
  • If the PR author isn’t in the mantid-developers or mantid-contributors teams, add a review comment rerun ci to authorize/rerun the CI

Gatekeeper

As per the gatekeeping guidelines:

  • Has a thorough first line review been conducted, including functional testing?
  • At a high-level, is the code quality sufficient?
  • Are the base, milestone and labels correct?

@RabiyaF RabiyaF added this to the Release 6.16 milestone Feb 11, 2026
@RabiyaF RabiyaF marked this pull request as ready for review February 17, 2026 09:18
@MialLewis MialLewis assigned MialLewis and unassigned MialLewis Feb 20, 2026
@MialLewis MialLewis self-requested a review February 20, 2026 10:16
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.

Increase automated test coverage with regards to LOQ Event mode

2 participants