mozilla/mozci-tools

refactor `dump_failures` script to have an excluded manifest list instead of harcoded in the middle of the logic

Open

#9 创建于 2021年9月29日

在 GitHub 查看
 (6 评论) (0 反应) (1 负责人)Python (11 fork)auto 404
good first issue

仓库指标

Star
 (7 star)
PR 合并指标
 (PR 指标待抓取)

描述

in dump_failures.py, there is some code like this:

            if manifest in ("dom/canvas/test/mochitest.ini",):
                continue

the reason for this is the format of test names and output is not supported yet (due to being a sub test suite) in dump_failures, so until we have logic to handle these sub test suites it is simpler to skip these as seen in the current code.

To solve this issue here are some recommendations:

  1. build a list of manifests that are defined at the top of the file in a constant (SKIPPED_MANIFESTS = ...)
  2. replace the current code to use the new list of defined tests
  3. add a comment in code to help out with this, update any documentation as needed

贡献者指南