Bug 690057

Summary:	[test run]the test case detail will be auto updated without click update
Product:	[Other] TCMS	Reporter:	Jin Zhao <jzhao>
Component:	Application	Assignee:	Yuguang Wang <yuwang>
Status:	CLOSED CURRENTRELEASE	QA Contact:	tools-bugs <tools-bugs>
Severity:	urgent	Docs Contact:
Priority:	urgent
Version:	3.3	CC:	ctang, dli, junzhang, ktakemur, nli, vchen
Target Milestone:	---	Keywords:	Regression
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	3.5.0-4	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2011-12-02 08:13:50 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Jin Zhao 2011-03-23 08:19:12 UTC

Description of problem:
the test case detail will be auto updated without click update

Version-Release number of selected component (if applicable):
3.3-4 

How reproducible:
100%

Steps to Reproduce:
1.Enter to test run
2.Click the case ID to enter to case detail
3.Edit the case and update the case content,return to the test run
4.Verify the update case in the run
  
Actual results:
The case content are updated,and the version can not be updated.

Expected results:
The case content and version can not be update until choose and click the "update"

Additional info:

Comment 1 Danqing Li 2011-04-22 09:21:59 UTC

*** Bug 695578 has been marked as a duplicate of this bug. ***

Comment 2 Chaobin Tang 2011-06-29 07:30:20 UTC

commit 48a63a0d27bbb14c55194356e04e5202a056f078

1. Event listener on update button fails to collect checked caserun IDs because of the corrupted DOM structure. (It has been entirely a nightmare to have used so wildly in TCMS in AJAX design that is based on DOM structure which is so unreliable and fragile, rather than using absolute positioning via id/class)

2. In the attempt of trying to get case detail information using an AJAX call, the back-end function fails to retrieve data from request because of a not-so-cute typo. More than that is this seems to have existed for some while, not a chance to have noticed this any earlier?

Comment 3 Jin Zhao 2011-07-14 05:24:47 UTC

Verify 3.5.0-1 on stage -->FAIL

Actual result:
1.The case summary / Attachment / Component / Tag can be update without click update
2.Choose the cases ,Click update, can not update it. Warning that: 0 case run(s) succeed to update, following is the list:

Excepted resut:
1.I suggest all values of test case can not be update without click update,we'd better disscuss with PM about this point.
2.update button can work

Comment 4 Danqing Li 2011-07-14 05:54:27 UTC

(In reply to comment #3)
> Verify 3.5.0-1 on stage -->FAIL
> 
> Actual result:
> 1.The case summary / Attachment / Component / Tag can be update without click
> update
> 2.Choose the cases ,Click update, can not update it. Warning that: 0 case
> run(s) succeed to update, following is the list:
> 
> Excepted resut:
> 1.I suggest all values of test case can not be update without click update,we'd
> better disscuss with PM about this point.

only case's text has history. 
If user change the case's summary, so TCMS could only reserve the historical text for case.
And,  I think that mean this case really need to update.
also for attachment/ component. 



> 2.update button can work

This is bug , PLEASE ctang review and fix.

Comment 5 Danqing Li 2011-07-14 05:57:22 UTC

(In reply to comment #4)
> (In reply to comment #3)
> > Verify 3.5.0-1 on stage -->FAIL
> > 
> > Actual result:
> > 1.The case summary / Attachment / Component / Tag can be update without click
> > update
> > 2.Choose the cases ,Click update, can not update it. Warning that: 0 case
> > run(s) succeed to update, following is the list:
> > 
> > Excepted resut:
> > 1.I suggest all values of test case can not be update without click update,we'd
> > better disscuss with PM about this point.
> 
> only case's text has history. 
> If user change the case's summary, so TCMS could only reserve the historical
> text for case.
> And,  I think that mean this case really need to update.

Sorry ,last comment has error. it should be:

Only case's text has history,so TCMS could only reserve the historical text for case.And , if user change the case's summary , I think that mean this case really need to update. similar for attachment/ component. 

> also for attachment/ component. 
> 
> 
> 
> > 2.update button can work
> 
> This is bug , PLEASE ctang review and fix.

Comment 6 Chaobin Tang 2011-07-18 06:02:27 UTC

One important point has been missing is the discussion regarding to updating testrun's case text. 

The update will only be allowed when caserun's status is in IDLE.

Try that.

Comment 7 Jin Zhao 2011-07-26 03:00:00 UTC

Chaobin,

This problem seems have been fixed in 3.4.0-4.Could you help to confirm it. If fixed, help to change the status to MODIFIED then ON_QA. thanks a lot.

Now After click update, the IDLE case run can be update. Other status can not be update.

Comment 8 Jin Zhao 2011-12-02 08:22:08 UTC

Hi chaobin,

I suggest change the bug status to CLOSED-CURRENTRELEASE is more reasonable. Thanks