Skip to content

[instruments] test plan #9781

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

adamdaudrich
Copy link
Collaborator

@adamdaudrich adamdaudrich commented May 5, 2025

Brief summary of changes

The intention of this PR is to have a complete test of the functionality of data_entry and the new view_instrument_data permissions proposed in PR 9762

This ended up on two branches because #9762 is not merged.

  • Have you updated related documentation? This is the documentation

Testing instructions (if applicable)

You can download the .md file here
instrumentsTestPlan.md

and test.
I'd like to update the .md once the full functionality of both permissions is tested with all configurations and additional permissions.

Link(s) to related issue(s)

  • Resolves # (Reference the issue this fixes, if any.)

@adamdaudrich adamdaudrich requested a review from ridz1208 May 5, 2025 13:29
@github-actions github-actions bot added the Module: instruments PR or issue related to instruments module label May 5, 2025
@adamdaudrich adamdaudrich requested a review from charliehenrib May 5, 2025 13:29
@christinerogers christinerogers added Release: Add to release notes PR whose changes should be highlighted in the release notes Category: Documentation PR or issue that aims to improve the documentation (test plans, wiki, comments...) labels Jun 10, 2025
@christinerogers
Copy link
Contributor

@adamdaudrich I haven't been part of the conversation re #9762 -- what the number of the Issue which that PR is trying to solve?
It sounds like this should be named/labelled as a Documentation PR following up on that issue/PR, no?

Let me know if there's a reason I was assigned to this rather than just review requested (when it's ready) cc @regisoc

@adamdaudrich
Copy link
Collaborator Author

@christinerogers The objective is a full test of instruments on both permissions (view, and edit). So the plan will encompass every nook and cranny of every functionality and configuration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Documentation PR or issue that aims to improve the documentation (test plans, wiki, comments...) Module: instruments PR or issue related to instruments module Release: Add to release notes PR whose changes should be highlighted in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants