New issue
Advanced search Search tips

Issue 638237 link

Starred by 1 user

Issue metadata

Status: Verified
Owner: ----
Closed: Aug 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Feature

Blocking:
issue 636568



Sign in to add a comment

Create browsertest_results dir for browsertest_results.py, and its test_suites dir.

Project Member Reported by scunning...@chromium.org, Aug 16 2016

Issue description

Currently, the browsertest_results.py utility is in /src/platform/crostestutils/provingground, and the test_suite files are shared from a Google Drive directory (e.g., https://drive.google.com/corp/drive/u/0/folders/0BwSZgPXs_HfCSkZaM3dXX1hBMUk).

The test_suit files need to be brought under source control. In order not to clutter up the provingground with potentially dozens of file, I will move browsertest_status.py into its own browsertest_status directory, and place the test_suites directory there.
 

Comment 2 Deleted

Project Member

Comment 3 by bugdroid1@chromium.org, Aug 16 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/crostestutils/+/428de063722fae2e6f9c4a24027840d860700c15

commit 428de063722fae2e6f9c4a24027840d860700c15
Author: Scott Cunningham <scunningham@chromium.org>
Date: Tue Aug 16 14:33:19 2016

Move browsertest_status.py and test_suites into own dir.

The test_suit files need to be brought under source control. To
prevent cluttering provingground dir with dozens of test list files,
move browsertest_status.py into its own browsertest_status directory,
and place the test_suites directory there.

BUG= chromium:638237 
TEST=None

Change-Id: Id1b2cfac354832f7295a65bb03118f99c5135e3f
Reviewed-on: https://chromium-review.googlesource.com/370804
Tested-by: Scott Cunningham <scunningham@chromium.org>
Trybot-Ready: Scott Cunningham <scunningham@chromium.org>
Reviewed-by: Krishna Gavini <krishnargv@chromium.org>
Reviewed-by: Scott Cunningham <scunningham@chromium.org>
Commit-Queue: Scott Cunningham <scunningham@chromium.org>

[add] https://crrev.com/428de063722fae2e6f9c4a24027840d860700c15/provingground/browsertest_status/test_suites/demo-retail_tests
[add] https://crrev.com/428de063722fae2e6f9c4a24027840d860700c15/provingground/browsertest_status/test_suites/wallpaper_tests
[add] https://crrev.com/428de063722fae2e6f9c4a24027840d860700c15/provingground/browsertest_status/test_suites/enterprise_tests
[rename] https://crrev.com/428de063722fae2e6f9c4a24027840d860700c15/provingground/browsertest_status/browsertest_status.py
[add] https://crrev.com/428de063722fae2e6f9c4a24027840d860700c15/provingground/browsertest_status/test_suites/goodies-page_tests
[add] https://crrev.com/428de063722fae2e6f9c4a24027840d860700c15/provingground/browsertest_status/test_suites/publicsessions_tests
[add] https://crrev.com/428de063722fae2e6f9c4a24027840d860700c15/provingground/browsertest_status/test_suites/kiosk_tests
[add] https://crrev.com/428de063722fae2e6f9c4a24027840d860700c15/provingground/browsertest_status/test_suites/wallpaper_interactive_tests
[add] https://crrev.com/428de063722fae2e6f9c4a24027840d860700c15/provingground/browsertest_status/test_suites/cfm_tests

Blocking: 636568
Status: Fixed (was: Started)
Labels: VerifyIn-54
bulk verified
Status: Verified (was: Fixed)
Owner: ----

Sign in to add a comment