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
|