Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(291)

Issue 1142803002: [Password manager tests automation] Log test cases which will be run. (Closed)

Created:
5 years, 7 months ago by melandory
Modified:
5 years, 7 months ago
Reviewers:
vabr (Chromium)
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[Password manager tests automation] Log test cases which will be run. Logs test cases which will be run, which helps to catch cases when you misspell config parameter name and run all test cases instead of specified once. BUG=369521 R=vabr@chromium.org Committed: https://crrev.com/af5c7c8d3074aab432ef94d49a325a0d52e4aaaf Cr-Commit-Position: refs/heads/master@{#330353}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -0 lines) Patch
M components/test/data/password_manager/automated_tests/run_tests.py View 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (1 generated)
melandory
Hi Vaclav, review this Cl, please. Thanks in advance.
5 years, 7 months ago (2015-05-18 12:25:52 UTC) #1
vabr (Chromium)
LGTM! Thanks, Vaclav
5 years, 7 months ago (2015-05-18 12:36:12 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1142803002/1
5 years, 7 months ago (2015-05-18 12:37:47 UTC) #4
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 7 months ago (2015-05-18 13:33:42 UTC) #5
commit-bot: I haz the power
5 years, 7 months ago (2015-05-18 13:34:21 UTC) #6
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/af5c7c8d3074aab432ef94d49a325a0d52e4aaaf
Cr-Commit-Position: refs/heads/master@{#330353}

Powered by Google App Engine
This is Rietveld 408576698