This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 240302 - Force user to provide reason for skipping testcases
Summary: Force user to provide reason for skipping testcases
Status: RESOLVED FIXED
Alias: None
Product: qa
Classification: Unclassified
Component: Synergy (show other bugs)
Version: 8.0
Hardware: PC Windows 7
: P3 normal with 1 vote (vote)
Assignee: Vladimir Riha
URL:
Keywords:
: 240313 (view as bug list)
Depends on:
Blocks:
 
Reported: 2014-01-11 17:25 UTC by markiewb
Modified: 2014-01-18 20:26 UTC (History)
1 user (show)

See Also:
Issue Type: ENHANCEMENT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description markiewb 2014-01-11 17:25:20 UTC
If you encounter a wrong testcase while running a testcase, currently I can skip or fail the test. But the information that the testcase is invalid is lost.

How about another state like "test case is wrong. Needs an update"? Or if you skip an entry you have to choose a reason "test case is wrong", "cannot execute testcase" ... from a dropdown?
Comment 1 skygo 2014-01-12 01:01:04 UTC
Good idea, it may help to review and fix better the test case
Comment 2 skygo 2014-01-12 01:04:36 UTC
Sorry validating too quickly. Event if green (passed) comment may be a cool think to have. (free text, dropdown, ...)
Comment 3 Vladimir Riha 2014-01-13 14:58:41 UTC
*** Bug 240313 has been marked as a duplicate of this bug. ***
Comment 4 Jiri Kovalsky 2014-01-13 20:54:08 UTC
Having such functionality is A but we also need the B i.e. what will happen to the gathered comments, who will process it etc.
Comment 5 Vladimir Riha 2014-01-13 20:58:09 UTC
True, I was only thinking about generating "comments" page to test run with all comments made to particular cases during testing so it would not affect specification as such. After that, someone would have to go through these comments (specification owner, anyone?) and resolve them
Comment 6 Lou Dasaro 2014-01-14 03:04:59 UTC
I was thinking more along the lines of a code table, but then it occurred to me that the only reasons I could think of were along the lines of:

Unclear or terse instructions / Don't understand
Steps are missing or incorrect for my Operating System
Steps are missing or incorrect in general
Other: [comment]

Any of which should indicate that the owner of the test specification in question should review that particular test case with a fine eye.
Comment 7 skygo 2014-01-15 14:10:14 UTC
Hi

I was thinking on maybe complementary idea with #240258. 

1) I imagine skipping a test A knowing it will take 30min and go to test B to have a maximum of coverture due to personnal schedule.
2) I skip the test A saying free text or a list as proposed by Lou.

Deadline is not reached:
3a) I use #240258 to replay A, it complete it, it remove my now useless reason to skip A.
Deadline is over:
3b) reason is kept on test A for final review.
Comment 8 Vladimir Riha 2014-01-16 09:02:33 UTC
I'd like to avoid free text option (it would allow accurate filtering in page with comments) , so here's a couple of possible predefined comments (thanks Lou and Eric):

Unclear instructions / Don't understand
Steps are missing or incorrect for my Operating System
Steps are missing or incorrect in general
Depends on previous case(s) which failed
Missing sample file(s) required in test case
Test case is obsolete
No time to finish test case

Any other ideas?
Comment 9 Jiri Kovalsky 2014-01-16 11:45:00 UTC
In debugger we had two almost identical test cases so something like "Duplicate test case" would be handy.
Comment 10 Vladimir Riha 2014-01-17 10:15:51 UTC
Thank you, I'll add it as well. I just pushed changes to repo (live in 15 minutes) which force user to provide reason for skipping case and save all comments to DB for further processing. 

TODO: page with comments + process to resolve them
Comment 11 Vladimir Riha 2014-01-18 20:26:07 UTC
Fixed, changes will be live in 15 minutes. 

Each test run has a link to comments page with list of comments added to test cases during testing. Anyone can mark the comment as "resolved" to indicate that issue with case has been resolved (since anyone can edit specification, no reason to limit resolve action to specification owners). User is also asked if he/she wants to resolve all similar comments as well (similar = same test case, same test suite and same comment). Comment can be added not only to skipped cases, but also for passed or failed (but it is required for skipped ones)

There is no way to "reopen" resolved comment as I don't think it is necessary for this use case and I don't think creating "bugzilla light" in Synergy is a good idea.