Set Status doesn't seem to work on the Edit page

Issue #181 resolved
xtal created an issue

What is set status supposed to do? Change the status of all the test results??? It does not seem to work. If I select set status and change it to rejected or scratch, the new status display after 'submit QA Result' is unreviewed. I am not sure that it makes sense to have set status at the test list level because status applies to each test result right. Should the status be automatically changed after any test list edit??

Comments (5)

  1. Randle Taylor repo owner
    • What is set status supposed to do? Change the status of all the test results???*

    Yup.

    • It does not seem to work. If I select set status and change it to rejected or scratch, the new status display after 'submit QA Result' is unreviewed.*

    Yes, this is a bug on the edit page. The status of all tests should be updated.

    • I am not sure that it makes sense to have set status at the test list level because status applies to each test result right.*

    The status dropdown is just a quick method of marking all tests with a certain status and is not really applied at a test-list level.

    • Should the status be automatically changed after any test list edit??*

    Anytime an edit is made and the status isn't explicitly set, I think it should be reset to the Unreviewed state. This avoids the situation where, for example, a passing test has been approved but then the original user realizes they made a mistake, changes a value and the test is now failing. That could go unnoticed if the test stays approved.

  2. Log in to comment