New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 641471 link

Starred by 1 user

Issue metadata

Status: Duplicate
Owner:
Last visit > 30 days ago
Closed: Aug 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

test_results_presentation.py fails with python error

Project Member Reported by pkotw...@chromium.org, Aug 26 2016

Issue description

Traceback (most recent call last):
  File "/b/build_internal/scripts/slave/.recipe_deps/build/scripts/slave/recipe_modules/chromium_android/resources/test_results_presentation.py", line 154, in <module>
    sys.exit(main())
  File "/b/build_internal/scripts/slave/.recipe_deps/build/scripts/slave/recipe_modules/chromium_android/resources/test_results_presentation.py", line 147, in main
    args.master_name)
  File "/b/build_internal/scripts/slave/.recipe_deps/build/scripts/slave/recipe_modules/chromium_android/resources/test_results_presentation.py", line 36, in result_details
    return results_to_html(results_list, cs_base_url, master_name)
  File "/b/build_internal/scripts/slave/.recipe_deps/build/scripts/slave/recipe_modules/chromium_android/resources/test_results_presentation.py", line 101, in results_to_html
    if suites_summary[FAIL_COUNT]['data'] > 0:
UnboundLocalError: local variable 'FAIL_COUNT' referenced before assignment
 
Labels: -Pri-3 OS-Android Pri-1
This is making the asan-clang-phone bot fail. For instance https://uberchromegw.corp.google.com/i/internal.client.clank/builders/asan-clang-phone
Cc: pkotw...@chromium.org
Owner: hzl@chromium.org
Status: Assigned (was: Untriaged)
Assigning to hzl@

Suspecting that this bug was caused by either https://codereview.chromium.org/2224173002 or https://codereview.chromium.org/2252603002

Comment 3 by hzl@chromium.org, Aug 27 2016

Mergedinto: 640991
Status: Duplicate (was: Assigned)
Just fixed it. 

Comment 4 by hzl@chromium.org, Aug 27 2016

Thanks for pointing it out and I just created a fix cl. However I suspect that no test has been run at all in this case. Is that something we want by design?

Sign in to add a comment