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

Issue 767005 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Feature



Sign in to add a comment

Add screenshot support for CfM server side autotests

Project Member Reported by dtosic@chromium.org, Sep 20 2017

Issue description

When a test fails we should capture a screenshot. 

This is the tracking bug for the feature
 
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 2 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/b84b27871e8b5b19c54010807031ec42c36a7839

commit b84b27871e8b5b19c54010807031ec42c36a7839
Author: Denis Tosic <dtosic@chromium.org>
Date: Mon Oct 02 10:03:44 2017

Takes a screenshot in .png format when test fails

Takes a screenshot if the enterprise_CFM_Sanity test fails
and stores it in the debug dir. Also exposes the necessary APIs
on the native facade & facade adapter. This is to improve
debugging when a test fails. We will later on move this logic
to a common base class for cfm tests.

Logs a warning if screenshotting failed, but does not break the
test.

pre-cq-configs: bluestreak-pre-cq
BUG= chromium:767005 
TEST=with local setup & lab device
Change-Id: I203e5852673812a49b9e5e3ee099cb2859503d23
Reviewed-on: https://chromium-review.googlesource.com/672865
Commit-Ready: Denis Tosic <dtosic@chromium.org>
Tested-by: Denis Tosic <dtosic@chromium.org>
Reviewed-by: Kristoffer Erlandsson <kerl@google.com>
Reviewed-by: Claes Malmnäs <malmnas@google.com>

[modify] https://crrev.com/b84b27871e8b5b19c54010807031ec42c36a7839/server/site_tests/enterprise_CFM_Sanity/enterprise_CFM_Sanity.py
[modify] https://crrev.com/b84b27871e8b5b19c54010807031ec42c36a7839/client/cros/multimedia/cfm_facade_native.py
[modify] https://crrev.com/b84b27871e8b5b19c54010807031ec42c36a7839/server/cros/multimedia/cfm_facade_adapter.py

Status: Fixed (was: Assigned)

Comment 3 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Comment 4 by dchan@chromium.org, Jan 23 2018

Status: Fixed (was: Archived)

Sign in to add a comment