pytests: Merge all *_probe into the new probe test |
|||
Issue descriptionThe new pytests/probe is now more powerful that I'd recommend to merge other "probe" tests to it, including: - usb_probe - i2c_probe - hwmon_probe Meanwhile, I'd also recommend to improve probe test itself so that: - There can be a simple UI listing components it is checking and have found or not. For example | Category| Name | Status | Info | +---------+------+--------+---------------+ | Battery | AXX | Pass | | | Touch | Syn1 | Failed |Mismatch: manid| | FPC | FXX | Failed |Not found |
,
Jan 23 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform/factory/+/3c275b6e0e7a5096cfbbd452016b80a3aa2755c2 commit 3c275b6e0e7a5096cfbbd452016b80a3aa2755c2 Author: Chih-Yu Huang <akahuang@google.com> Date: Mon Jan 16 10:09:56 2017 pytest: add UI in probe pytest. We generate a table to show the probed component for each category and passes the rule or not for better debugging. BUG= chromium:677867 TEST=manually test at device Change-Id: Ie82b332548bc3ee14a24ae210903b22c4ccf94ca Reviewed-on: https://chromium-review.googlesource.com/428613 Commit-Ready: Chih-Yu Huang <akahuang@chromium.org> Tested-by: Chih-Yu Huang <akahuang@chromium.org> Reviewed-by: Hung-Te Lin <hungte@chromium.org> Reviewed-by: Pi-Hsun Shih <pihsun@chromium.org> [modify] https://crrev.com/3c275b6e0e7a5096cfbbd452016b80a3aa2755c2/py/test/pytests/probe/probe.py
,
Mar 28 2017
The following revision refers to this bug: https://chrome-internal.googlesource.com/chromeos/overlays/overlay-eve-private/+/72c9af04579b478e5bc85a86f8e48ecb6c81d8d6 commit 72c9af04579b478e5bc85a86f8e48ecb6c81d8d6 Author: Chih-Yu Huang <akahuang@google.com> Date: Tue Mar 28 07:44:44 2017
,
Mar 28 2017
The following revision refers to this bug: https://chrome-internal.googlesource.com/chromeos/overlays/overlay-eve-private/+/e453165eb5034657c6a24f09384ad8f064d6af89 commit e453165eb5034657c6a24f09384ad8f064d6af89 Author: Chih-Yu Huang <akahuang@google.com> Date: Tue Mar 28 10:47:46 2017
,
Jul 26 2017
Assign to yhong. I think this is roughly done, and we should not change test lists to only use the single probe, removing individual probe_* tests.
,
Aug 9 2017
The following revision refers to this bug: https://chrome-internal.googlesource.com/chromeos/overlays/overlay-poppy-private/+/6a65e971b9578e0cb1c1c75c1a4eaf59b4591c39 commit 6a65e971b9578e0cb1c1c75c1a4eaf59b4591c39 Author: Yong Hong <yhong@google.com> Date: Wed Aug 09 12:18:05 2017
,
Sep 19 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform/factory/+/8ae05062bb55221c101d2ec740084de021b4f1fb commit 8ae05062bb55221c101d2ec740084de021b4f1fb Author: Yong Hong <yhong@chromium.org> Date: Tue Sep 19 12:25:10 2017 probe: Load the probe statement file by `config_utils.LoadConfig`. This CL modifies `probe_cmdline.py` to load the given config file by config_utils.LoadConfig instead of type_utils + json loader so that the features such as inheritance and overridden can be applied in probe statement file. BUG= chromium:677867 TEST=manually test on DUT Change-Id: I7d1931cfd5d915c5a316f1eb4aa34e5366abe56a Reviewed-on: https://chromium-review.googlesource.com/666856 Commit-Ready: Yong Hong <yhong@google.com> Tested-by: Yong Hong <yhong@google.com> Reviewed-by: Chih-Yu Huang <akahuang@chromium.org> [modify] https://crrev.com/8ae05062bb55221c101d2ec740084de021b4f1fb/py/probe/statement.schema.json [modify] https://crrev.com/8ae05062bb55221c101d2ec740084de021b4f1fb/py/probe/probe_cmdline.py [add] https://crrev.com/8ae05062bb55221c101d2ec740084de021b4f1fb/py/probe/generic_statement.schema.json [modify] https://crrev.com/8ae05062bb55221c101d2ec740084de021b4f1fb/py/probe/common.py [modify] https://crrev.com/8ae05062bb55221c101d2ec740084de021b4f1fb/py/test/utils/goofy_plugin_utils.py [modify] https://crrev.com/8ae05062bb55221c101d2ec740084de021b4f1fb/py/utils/testdata/config_utils_unittest.json [modify] https://crrev.com/8ae05062bb55221c101d2ec740084de021b4f1fb/py/utils/config_utils.py [modify] https://crrev.com/8ae05062bb55221c101d2ec740084de021b4f1fb/py/test/test_lists/manager.py [modify] https://crrev.com/8ae05062bb55221c101d2ec740084de021b4f1fb/py/test/pytests/blocking_charge.py [add] https://crrev.com/8ae05062bb55221c101d2ec740084de021b4f1fb/py/utils/testdata/extra_dir/config_utils_unittest_extra_dir.json [modify] https://crrev.com/8ae05062bb55221c101d2ec740084de021b4f1fb/py/utils/config_utils_unittest.py
,
Sep 19 2017
The following revision refers to this bug: https://chrome-internal.googlesource.com/chromeos/overlays/overlay-soraka-private/+/a42e9bdae50d350430e3c074bea10d7efdcf22d2 commit a42e9bdae50d350430e3c074bea10d7efdcf22d2 Author: Yong Hong <yhong@google.com> Date: Tue Sep 19 14:11:07 2017
,
Sep 27 2017
pytest 'verify_touch_device_fw' should be fine to use the new probe framework as well.
,
Sep 27 2017
The following revision refers to this bug: https://chrome-internal.googlesource.com/chromeos/overlays/overlay-coral-private/+/4460e1d3c0aa6df6471cda16838ea91d901b66d0 commit 4460e1d3c0aa6df6471cda16838ea91d901b66d0 Author: Yong Hong <yhong@google.com> Date: Wed Sep 27 08:47:52 2017
,
Sep 28 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform/factory/+/addd4c89ae04d705567b72de0e9c6111bce846fc commit addd4c89ae04d705567b72de0e9c6111bce846fc Author: Yong Hong <yhong@chromium.org> Date: Thu Sep 28 10:59:26 2017 probe: Removes the legacy *_probe pytests. This CL deprecates the legacy probe pytests: * i2c_probe * usb_probe In the future, all probing related tasks should be implemented by the new probe framework. BUG= chromium:677867 TEST=manually test on DUT Change-Id: I90343de794968f23f3a2c6d2e4d92497fb9255b9 Reviewed-on: https://chromium-review.googlesource.com/689414 Commit-Ready: Yong Hong <yhong@google.com> Tested-by: Yong Hong <yhong@google.com> Reviewed-by: Hung-Te Lin <hungte@chromium.org> [modify] https://crrev.com/addd4c89ae04d705567b72de0e9c6111bce846fc/po/zh-CN.po [modify] https://crrev.com/addd4c89ae04d705567b72de0e9c6111bce846fc/py/test/test_lists/generic_common.test_list.json [delete] https://crrev.com/e74783dc9831f136f5cf901e1ab98d80034b29e2/py/test/pytests/probe/probe_sample_i2c.json [delete] https://crrev.com/e74783dc9831f136f5cf901e1ab98d80034b29e2/py/test/pytests/usb_probe.py [delete] https://crrev.com/e74783dc9831f136f5cf901e1ab98d80034b29e2/py/test/pytests/i2c_probe.py
,
Oct 11 2017
The following revision refers to this bug: https://chrome-internal.googlesource.com/chromeos/overlays/overlay-coral-private/+/a356de783e02e5a83be92fea8aa5dee5867386de commit a356de783e02e5a83be92fea8aa5dee5867386de Author: zgh <guohui.zhou@bitland.corp-partner.google.com> Date: Wed Oct 11 07:57:43 2017
,
Apr 13 2018
re #11, "verify_touch_device_fw" has already been listed in chromium:740513 so let's mark this bug fixed first and remove that pytest in that bug. |
|||
►
Sign in to add a comment |
|||
Comment 1 by bugdroid1@chromium.org
, Jan 20 2017