History log of /external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
1a8c4704549ae17d752404c182efa4e6ce12aacd 22-Nov-2016 Aseda Aboagye <aaboagye@google.com> firmware_ECKeyboard: Add delay for dev console.

Add a short delay after invoking the dev console to prevent characters
being dropped. For some devices, the beginning characters appeared to
be missing/dropped.

BUG=chrome-os-partner:56274
BRANCH=None
TEST=Run firmware_ECKeyboard against chell 10 times and verify it passes
each time.

Change-Id: I199cea287fd9e9bb6080c66b87340484812b1ba8
Signed-off-by: Aseda Aboagye <aaboagye@google.com>
Reviewed-on: https://chromium-review.googlesource.com/413305
Commit-Ready: Nicolas Boichat <drinkcat@chromium.org>
Tested-by: Aseda Aboagye <aaboagye@chromium.org>
Reviewed-by: danny chan <dchan@chromium.org>
Reviewed-by: Wai-Hong Tam <waihong@google.com>
/external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py
3e92b8e9041fa7bb44dc9285e2c415eb611697a9 07-May-2015 Tom Wai-Hong Tam <waihong@google.com> faft: Remove do_reboot_action and use the single-entry reboot method

This change further unifies more calls, which use do_reboot_action,
to adapt the new mode_aware_reboot method.

BUG=chrome-os-partner:16231
TEST=Ran all affected FAFT tests on Jerry.

Change-Id: I74120fe091cc06a910c1e31f09fdf748a2bb028d
Reviewed-on: https://chromium-review.googlesource.com/270013
Trybot-Ready: Tom Tam <waihong@google.com>
Tested-by: Tom Tam <waihong@google.com>
Reviewed-by: Yusuf Mohsinally <mohsinally@chromium.org>
Commit-Queue: Tom Tam <waihong@google.com>
/external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py
0cc9a4fe28f221cfc73bb35c30fa79338c4b43f1 01-May-2015 Tom Wai-Hong Tam <waihong@google.com> faft: Refactor setup_mode and restore_mode

The change moves and splits the original logic on setup_dev_mode and
restore_dev_mode to FAFTCheckers and ModeSwitcher. It is also enhanced
to support setting up all firmware modes, including recovery. They are
now methods in ModeSwitcher as they are highly related to mode
switching.

BUG=chrome-os-partner:39744
TEST=Switched DUT to normal mode and ran firmware_DevMode. Then
switched DUT to dev mode and run firmware_DevMode. Checked DUT
restored to the original mode state.

Change-Id: I16314d18b785da59473df3ef5e02f7fe8f2693b0
Reviewed-on: https://chromium-review.googlesource.com/268858
Reviewed-by: Yusuf Mohsinally <mohsinally@chromium.org>
Commit-Queue: Tom Tam <waihong@google.com>
Tested-by: Tom Tam <waihong@google.com>
/external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py
a2d0311b181a51798af86348d4c81011a1370964 16-May-2014 Yusuf Mohsinally <mohsinally@google.com> Converting firmware_ECKeyboard to use new FirmwareTest class.

The new FirmwareTest class does not require the traditional dict of objects
that FAFTSequence required.

1. Converted the test to use the new flat function call style.

BUG=chromium:371185
TEST=Manually ran this test at desk against peppy

Change-Id: Ib314edde02f37dc5bd2260995935edd39bb6993a
Reviewed-on: https://chromium-review.googlesource.com/200104
Reviewed-by: Vic Yang <victoryang@chromium.org>
Commit-Queue: Yusuf Mohsinally <mohsinally@chromium.org>
Tested-by: Yusuf Mohsinally <mohsinally@chromium.org>
/external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py
1f115ae9ee3c043e9577e7ce9872dc99323fdcb3 07-Nov-2013 Yusuf Mohsinally <mohsinally@chromium.org> Correctly using initialize() rather than setup() for FAFT tests.

FAFT tests have been using setup() (which is run once at autotest init), when
it should have instead used initialize() for running code before each
test case is run.

BUG=chromium:267175
TEST=Ran tests manually against Pit device to ensure that these changes did not
introduce problems.

Change-Id: If0e76e12b9f40249154d7b95a1375b2787a56304
Reviewed-on: https://chromium-review.googlesource.com/176024
Reviewed-by: Wai-Hong Tam <waihong@chromium.org>
Commit-Queue: Yusuf Mohsinally <mohsinally@chromium.org>
Tested-by: Yusuf Mohsinally <mohsinally@chromium.org>
/external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py
73c28f3abd6ec8de4dc5a8e6afca1d5e83eb59c7 03-Aug-2013 Yusuf Mohsinally <mohsinally@google.com> Reorginazing FAFT (server side) files.

Initial reorganization step while moves FAFT (autotest/server)
files over to packages. Naming and other cleanup will be performed
in additional CLs.

BUG=None
TEST=built and ran faft tests at desk against Link

Change-Id: I83712620c0e6816ec08b9afdf8c227a800734ea6
Reviewed-on: https://gerrit.chromium.org/gerrit/64408
Reviewed-by: Yusuf Mohsinally <mohsinally@chromium.org>
Tested-by: Yusuf Mohsinally <mohsinally@chromium.org>
Commit-Queue: Yusuf Mohsinally <mohsinally@chromium.org>
/external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py
d4340315f76fcae9a5e9430d0446705f4f8fa7b7 17-Jul-2013 Tom Wai-Hong Tam <waihong@chromium.org> FAFT: Rename the module faftsequence to faft_classes

For preparing the future refactoring that splitting FAFTSequence into
multiple meaningful small classes, changes the original module name
faftsequence to faft_classes.

BUG=chrome-os-partner:21118
TEST=Manual
Ran the affected test cases, firmware_FAFTSetup, on Spring and passed.

Change-Id: I662cfc8d37a54711d7c52ff5322ed28a9a759ffe
Reviewed-on: https://gerrit.chromium.org/gerrit/62642
Reviewed-by: Yusuf Mohsinally <mohsinally@chromium.org>
Commit-Queue: Tom Wai-Hong Tam <waihong@chromium.org>
Tested-by: Tom Wai-Hong Tam <waihong@chromium.org>
/external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py
a051510c53dbbd01f768a2546098ec73ba325221 07-May-2013 Vic Yang <victoryang@chromium.org> Suppress EC console for FAFT EC tests

For EC tests, suppress EC console output so that they don't interfere
with our tests.

BUG=None
TEST=Run firmware_ECBattery on Spring

Change-Id: Iff312951d056d3f8a3d54b26896f83100209a4ad
Signed-off-by: Vic Yang <victoryang@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/50245
Reviewed-by: Yusuf Mohsinally <mohsinally@chromium.org>
/external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py
1c903d62b3381af452cda2415ebd83a523a8f05a 07-May-2013 Vic Yang <victoryang@chromium.org> Use TestNAError for not applicable FAFT tests

When a not applicable test is run, raise TestNAError so the user sees
something like:
TEST_NA: Nothing needs to be tested on this device

BUG=chrome-os-partner:19170
TEST=Run firmware_ECThermal on Spring and see TEST_NA.

Change-Id: Ib4a12f2224324ca0ad887a20ec565cc1e60d55d1
Signed-off-by: Vic Yang <victoryang@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/50233
Reviewed-by: Yusuf Mohsinally <mohsinally@chromium.org>
Commit-Queue: David James <davidjames@chromium.org>
/external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py
4d743049231ce20b9a2a8694b2e01fad78a12f3a 02-May-2013 Vic Yang <victoryang@chromium.org> Add more log message about skipped FAFT tests

The current warning message doesn't say that skipped tests are normal.
Let's add more descriptive messages so that people don't get confused.

BUG=chrome-os-partner:19170
TEST=None

Change-Id: Iad186d5dbde7d9458b1dac599376ac32b3951b13
Signed-off-by: Vic Yang <victoryang@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/49945
Reviewed-by: Yusuf Mohsinally <mohsinally@chromium.org>
/external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py
b09a04183cf7017eb4b30ad28f5f81f1204c36b3 06-Nov-2012 Vic Yang <victoryang@chromium.org> faft: Fix lint warning and error

This includes removing unnecessary 'pass' and unused argumenets.

BUG=chrome-os-partner:35902
TEST=Check tests still work.

Change-Id: Ifb9ac8348e7e26e2ea1a10b9c7e509f817e117b9
Signed-off-by: Vic Yang <victoryang@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/37424
Reviewed-by: Tom Wai-Hong Tam <waihong@chromium.org>
/external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py
8326bfff0058088de20a0a83b6244777753ff0cd 12-Oct-2012 Tom Wai-Hong Tam <waihong@chromium.org> faft: Move EC key-pressing methods to ChromeEC class

We want to re-use these EC key-pressing methods in other tests. Like the
Ctrl-U dev USB boot test needs to emulate CTRL and U via Chrome EC. So
move it to ChromeEC class.

BUG=chromium-os:35255
TEST=run_remote_tests.sh --board daisy --remote dut firmware_ECKeyboard

Change-Id: I303ca641e941d2d36f3669511ffc3d6313d944cc
Reviewed-on: https://gerrit.chromium.org/gerrit/35408
Reviewed-by: Vic Yang <victoryang@chromium.org>
Reviewed-by: Mike Truty <truty@chromium.org>
Commit-Ready: Tom Wai-Hong Tam <waihong@chromium.org>
Tested-by: Tom Wai-Hong Tam <waihong@chromium.org>
/external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py
6019a1a0c375bee346b183921d3edf5f098d3f68 12-Oct-2012 Tom Wai-Hong Tam <waihong@chromium.org> faft: Refactor Chrome EC related methods into a class

So far the Chrome EC related methods are quite distributed across tests.
We should have a single place to control all the Chrome EC functions.

This CL does the first step to refactor the UART command methods into
the ChromeEC class.

BUG=chromium-os:35254
TEST=run_remote_tests.sh --board link --remote dut suite:faft_ec

Change-Id: If148170bcf6801a2a33f666b8a5a7fadc47f3243
Reviewed-on: https://gerrit.chromium.org/gerrit/35403
Reviewed-by: Vic Yang <victoryang@chromium.org>
Reviewed-by: Mike Truty <truty@chromium.org>
Commit-Ready: Tom Wai-Hong Tam <waihong@chromium.org>
Tested-by: Tom Wai-Hong Tam <waihong@chromium.org>
/external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py
72bdc35b3d3fd35842b04f582bfaa4e5fb83bb16 18-Sep-2012 Tom Wai-Hong Tam <waihong@chromium.org> faft: Call the setup methods of their superclasses on EC FAFT

Should call the setup methods of their superclasses first. Otherwise,
some variables may not be initialized.

BUG=chrome-os-partner:14158,chrome-os-partner:14161,chrome-os-partner:14162
TEST=run_remote_tests.sh --remote=$IP suite:faft_ec

Change-Id: Ie93f3a6bba42a1f4bcaffd6020e2c23ef7214964
Reviewed-on: https://gerrit.chromium.org/gerrit/33480
Commit-Ready: Tom Wai-Hong Tam <waihong@chromium.org>
Tested-by: Tom Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: Mike Truty <truty@chromium.org>
/external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py
f3a35c4634ab88f12bee2d1bbd46c2c9a9806b62 14-Sep-2012 Vic Yang <victoryang@chromium.org> faft: run EC peripheral tests in normal mode

EC peripheral tests don't care about dev mode. Let's only run them in
normal mode.

BUG=chromium-os:34392
TEST=Passed the tests.

Change-Id: If68a153be5ae3dbe2bfac21012c3ab9573c5fa4d
Signed-off-by: Vic Yang <victoryang@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/33296
Reviewed-by: Tom Wai-Hong Tam <waihong@chromium.org>
/external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py
4d72cb65b05f9a535405c0dc90d6e24c75837c9c 24-Jul-2012 Vic Yang <victoryang@chromium.org> FAFT: Issue warning when running incompatible EC tests

Some EC tests require certain functinoality of EC which might not exists
on all devices. In this case, we should issue warning and let the test
pass so as not to raise false alarm.

BUG=chrome-os-partner:11629
TEST=Check tests are gated by EC attributes.

Change-Id: Ie518271e2a21094b1f8876c9520d95678a7474e5
Reviewed-on: https://gerrit.chromium.org/gerrit/28355
Reviewed-by: Tom Wai-Hong Tam <waihong@chromium.org>
Tested-by: Vic Yang <victoryang@chromium.org>
Commit-Ready: Vic Yang <victoryang@chromium.org>
/external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py
80eda6d23620343968dfc468b7eb36cbbb8347db 04-Jul-2012 Vic Yang <victoryang@chromium.org> firmware_ECKeyboard: Remove use of 'ectool kbpress'

We can now use UART console and thus don't need 'ectool kbpress'
anymore. Let's remove this since 'ectool kbpress' will be removed.

BUG=chrome-os-partner:10285
TEST=Test passed.

Change-Id: Ie67e74a50d43049041c620ecef1f4667e699c5c5
Reviewed-on: https://gerrit.chromium.org/gerrit/26761
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Commit-Ready: Vic Yang <victoryang@chromium.org>
Tested-by: Vic Yang <victoryang@chromium.org>
/external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py
e3c89ebe3f9d6db23fd2923d68c7b9074965a162 31-May-2012 Vic Yang <victoryang@chromium.org> Improve EC keyboard control code

Previously, we need to separate character string and special keys when
simulating key press. This CL modify the key control function to accept
mixed string.

BUG=chrome-os-partner:9188
TEST=firmware_ECKeyboard passed.

Change-Id: I3e5723c28ef960b003ab25b4a8bd98afd799b3e4
Reviewed-on: https://gerrit.chromium.org/gerrit/24163
Reviewed-by: Tom Wai-Hong Tam <waihong@chromium.org>
Commit-Ready: Vic Yang <victoryang@chromium.org>
Tested-by: Vic Yang <victoryang@chromium.org>
/external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py
39d68b3886ffd90339d28b3f2b387b5de436e205 30-May-2012 Vic Yang <victoryang@chromium.org> Add a test of EC sending keyboard event to host

This test simulates key press and check if EC can correctly send
keyboard event to host.

BUG=chrome-os-partner:9188
TEST=Test passed.

Change-Id: Ieab2e5e9e878a06743e3ba404d1115469741eac9
Reviewed-on: https://gerrit.chromium.org/gerrit/24012
Reviewed-by: Tom Wai-Hong Tam <waihong@chromium.org>
Tested-by: Vic Yang <victoryang@chromium.org>
Commit-Ready: Vic Yang <victoryang@chromium.org>
/external/autotest/server/site_tests/firmware_ECKeyboard/firmware_ECKeyboard.py