New issue
Advanced search Search tips

Issue 877164 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: ----

Blocking:
issue 805276



Sign in to add a comment

Make android test steps not nested one

Project Member Reported by st...@chromium.org, Aug 23

Issue description

"chrome_junit_tests (with patch)" http://ci.chromium.org/b/8938082364041910752

This makes it difficult to process data in BQ query.
In addition, a bug is that the test results are uploaded with test step name "chrome_junit_tests" without the suffix.

 
Blocking: 805276
Owner: jbudorick@chromium.org
Status: Started (was: Available)
https://chromium-review.googlesource.com/c/chromium/tools/build/+/1254902
Project Member

Comment 3 by bugdroid1@chromium.org, Oct 2

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build/+/fcafeba09c8baaa96f221be06c88d258ccdf9ace

commit fcafeba09c8baaa96f221be06c88d258ccdf9ace
Author: John Budorick <jbudorick@chromium.org>
Date: Tue Oct 02 02:09:05 2018

chromium_tests: unnest AndroidTest steps.

Bug:  877164 
Change-Id: Ie4593ebf6df396ce1f2943f4da83cd65299a8854
Reviewed-on: https://chromium-review.googlesource.com/1254902
Commit-Queue: John Budorick <jbudorick@chromium.org>
Reviewed-by: Stephen Martinis <martiniss@chromium.org>

[modify] https://crrev.com/fcafeba09c8baaa96f221be06c88d258ccdf9ace/scripts/slave/recipe_modules/chromium_tests/tests/steps/web_view_cts_test.expected/with suffix.json
[modify] https://crrev.com/fcafeba09c8baaa96f221be06c88d258ccdf9ace/scripts/slave/recipe_modules/chromium_tests/tests/steps/generate_junit_test.expected/basic.json
[modify] https://crrev.com/fcafeba09c8baaa96f221be06c88d258ccdf9ace/scripts/slave/recipes/chromium.expected/dynamic_instrumentation_test.json
[modify] https://crrev.com/fcafeba09c8baaa96f221be06c88d258ccdf9ace/scripts/slave/recipe_modules/chromium_tests/tests/steps/web_view_cts_test.expected/basic pass.json
[modify] https://crrev.com/fcafeba09c8baaa96f221be06c88d258ccdf9ace/scripts/slave/recipe_modules/chromium_tests/tests/steps/android_junit_test.expected/basic.json
[modify] https://crrev.com/fcafeba09c8baaa96f221be06c88d258ccdf9ace/scripts/slave/recipe_modules/chromium_tests/tests/steps/web_view_cts_test.py
[modify] https://crrev.com/fcafeba09c8baaa96f221be06c88d258ccdf9ace/scripts/slave/recipe_modules/chromium_tests/tests/steps/web_view_cts_test.expected/basic fail.json
[modify] https://crrev.com/fcafeba09c8baaa96f221be06c88d258ccdf9ace/scripts/slave/recipe_modules/chromium_tests/tests/steps/generate_cts_test.expected/basic.json
[modify] https://crrev.com/fcafeba09c8baaa96f221be06c88d258ccdf9ace/scripts/slave/recipe_modules/chromium_tests/tests/steps/android_instrumentation_test.expected/basic.json
[modify] https://crrev.com/fcafeba09c8baaa96f221be06c88d258ccdf9ace/scripts/slave/recipe_modules/chromium_tests/tests/steps/generate_instrumentation_test.expected/basic.json
[modify] https://crrev.com/fcafeba09c8baaa96f221be06c88d258ccdf9ace/scripts/slave/recipes/chromium.expected/dynamic_junit_test.json
[modify] https://crrev.com/fcafeba09c8baaa96f221be06c88d258ccdf9ace/scripts/slave/recipe_modules/chromium_tests/steps.py
[modify] https://crrev.com/fcafeba09c8baaa96f221be06c88d258ccdf9ace/scripts/slave/recipe_modules/chromium_tests/tests/steps/android_junit_test.py
[modify] https://crrev.com/fcafeba09c8baaa96f221be06c88d258ccdf9ace/scripts/slave/recipes/chromium.expected/dynamic_instrumentation_nodefault_build.json
[modify] https://crrev.com/fcafeba09c8baaa96f221be06c88d258ccdf9ace/scripts/slave/recipe_modules/chromium_tests/tests/steps/android_instrumentation_test.py
[modify] https://crrev.com/fcafeba09c8baaa96f221be06c88d258ccdf9ace/scripts/slave/recipes/chromium.expected/dynamic_instrumentation_nodefault_test.json
[modify] https://crrev.com/fcafeba09c8baaa96f221be06c88d258ccdf9ace/scripts/slave/recipes/chromium.expected/dynamic_instrumentation_test_custom_name.json

Status: Fixed (was: Started)
Thanks for the fix! This makes things easier!

Sign in to add a comment