1244061 - Changing case summary from case list view unchecks automation indicator on automated tests

Status: RESOLVED WORKSFORME

(Testopia :: Test Cases, defect)

Description

[Antoine]

User Agent: Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.111 Safari/537.36

Steps to reproduce:

1. Click on the PLAN link in the top of the page
    ex: Product: Automated Tests  >  Plan 1288  >  Case 110538 
2. Click on "Plan 1288"
3. Click on Test Cases tab
4. Look for the test case
    CNTRL-F 110538
5. Modify the case summary for the test case
6. Make sure to save the change and wait for update to complete.
7. Click on the test case link
8. Click on Automation tab




Actual results:

Check mark in check box is to be retained. 




Expected results:

Check mark in check box  is cleared

Comment 1

[Antoine]

Sorry mistakenly I have updated the results. 

Actual results:

Check mark in check box is cleared

Expected results:

Check mark in check box is to be retained.

Comment 2

[Antoine]

Found the solution that we have not handled the "set_isautomated" flag properly.

Now I have modified the script to handle it properly. 

Line Number:66

Replace the old code by 
$case->set_isautomated(($cgi->param('isautomated') eq 'on' || $cgi->param('isautomated') == 1) ? 1 : 0) if $cgi->param('isautomated');


Please update in master repository.

Comment 3

[Antoine]

Fix was in the previous comment.