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.

Sign up for GitHub

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

Jump to bottom

[battery_manager] update test plan #8739

Closed
regisoc opened this issue Jun 8, 2023 · 1 comment · Fixed by #8782
Closed

[battery_manager] update test plan #8739

regisoc opened this issue Jun 8, 2023 · 1 comment · Fixed by #8782
Assignees
@skarya22
Labels
25.0.0 - Bugs Category: Bug PR or issue that aims to report or fix a bug Category: Documentation PR or issue that aims to improve the documentation (test plans, wiki, comments...)

Comments

@regisoc
Copy link
Contributor

regisoc commented Jun 8, 2023 •
edited
Loading

Describe the bug

Test plan needs an update.

Testing Change Status column
    Press the Deactivate button in the Change Status column on an entry in the data table.
        Ensure that when you press "Yes", a success message appears and the table data refreshes.

No Yes confirmation option is shown. When clicking the deactivate button, the change is immediate.

Instrument, Stage, Cohort, Visit Label, Site, First Visit, Instrument Order, and Active (as dropdown fields with blank default option).

This line should not include Instrument Order (not a dropdown field).

Try to edit an entry so that it has the same values as another deactivated entry in the Test Battery.

We cannot create an entry with the same values, so I am not sure how to test this.

What did you expect to happen?
Change the test plan
@regisoc regisoc added Category: Bug PR or issue that aims to report or fix a bug 25.0.0 Manual Test Assignment labels Jun 8, 2023
@regisoc regisoc mentioned this issue Jun 8, 2023
Closed
14 tasks
@regisoc regisoc added the Category: Documentation PR or issue that aims to improve the documentation (test plans, wiki, comments...) label Jun 8, 2023
@skarya22 skarya22 mentioned this issue Jun 14, 2023
Merged
driusan pushed a commit that referenced this issue Jan 16, 2024
@skarya22
[battery_manager] Update Test Plan (#8782)
7b74d57 
Update test plan for Battery Manager with the requested changes

Resolves #8739
@driusan
Copy link
Collaborator

driusan commented Feb 26, 2024

fixed by linked PR, wasn't autoclosed..

@driusan driusan closed this as completed Feb 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@skarya22 skarya22

Labels
25.0.0 - Bugs Category: Bug PR or issue that aims to report or fix a bug Category: Documentation PR or issue that aims to improve the documentation (test plans, wiki, comments...)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[battery_manager] Update Test Plan skarya22/Loris
3 participants
@driusan @regisoc @skarya22