Summary: | NRWT still confused about test count with --repeat-each and --iterations | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jussi Kukkonen (jku) <jussi.kukkonen> | ||||||||
Component: | Tools / Tests | Assignee: | Jussi Kukkonen (jku) <jussi.kukkonen> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | abarth, dpranke, eric, jussi.kukkonen, ojan, webkit.review.bot | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Jussi Kukkonen (jku)
2013-01-10 07:10:44 PST
Created attachment 182132 [details]
Patch
Adding dpranke and ojan (from bug 95789), maybe either of you wants to review? Comment on attachment 182132 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=182132&action=review > Tools/Scripts/webkitpy/layout_tests/views/printing.py:101 > + num_unique_tests = num_to_run Assuming that this works, you can just use num_to_run wherever num_unique_tests is used. Created attachment 182136 [details]
remove the useless variable as well
Comment on attachment 182136 [details]
remove the useless variable as well
Okay, so it appears that this patch changes things so that we print the # of distinct tests we found, the number of those that we'll actually run, the number of unique tests that we'll actually skip, and the --repeat-each and --iterations values, right?
So, we're not printing the aggregate totals, i.e., if we're running 100 tests ten times each, we'll never print 1000.
Did your change break any tests? It sure seems like it should've. If it doesn't, that means that this isn't being tested, and since it's behavior we've probably broken at least twice, we should probably have a test for it (even though this is just a cosmetic thing). Mind adding a test?
Created attachment 182833 [details]
Add test for print_found()
Comment on attachment 182833 [details]
Add test for print_found()
looks great, thanks!
Comment on attachment 182833 [details] Add test for print_found() Clearing flags on attachment: 182833 Committed r139841: <http://trac.webkit.org/changeset/139841> All reviewed patches have been landed. Closing bug. Comment on attachment 182833 [details] Add test for print_found() View in context: https://bugs.webkit.org/attachment.cgi?id=182833&action=review > Tools/Scripts/webkitpy/layout_tests/views/printing.py:106 > + print (found_str) turns out this got left in accidentally and I missed it in the review. Fixed in http://trac.webkit.org/changeset/139913 . (In reply to comment #10) > turns out this got left in accidentally and I missed it in the review. Fixed in http://trac.webkit.org/changeset/139913 . Thanks for handling it. |