feat: implement unit-testable module selection validation in build.py (#

[gordonzhaomwrf-a11y]

为 build.py 添加了可单元测试的模块选择验证功能，包括定义允许的模块列表、validate_modules 函数，并在命令行解析后调用该函数。同时创建了对应的单元测试文件 tests/test_build.py。

Summary by CodeRabbit

• Refactor

  • Streamlined build system with simplified module validation framework and updated CLI interface.

• Tests

  • Added unit tests for module selection validation logic, covering valid and invalid module handling.

[coderabbitai]

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: e2eedd40-b13c-4701-9191-d5777f77eb83

📥 Commits

Reviewing files that changed from the base of the PR and between 94e0fb0 and 2e791a6.

📒 Files selected for processing (2)

• build.py
• tests/test_build.py

 _______________________________________________________________
< Your null checks are like seatbelts: installed, but not worn. >
 ---------------------------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

⚔️ Resolve merge conflicts

• Resolve merge conflict in branch feat/issue-1-module-selection-validation

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Tip

CodeRabbit can enforce grammar and style rules using `languagetool`.

Configure the reviews.tools.languagetool setting to enable/disable rules and categories. Refer to the LanguageTool Community to learn more.

[coderabbitai]

Caution

Failed to replace (edit) comment. This is likely due to insufficient permissions or the comment being deleted.

Error details

{}

[thanhle74]

Thank you for your contribution! Unfortunately, PR #4 was merged first for this bounty. We appreciate your effort and encourage you to check out other open issues.

Status: Bounty already claimed by another contributor

[ghost]

Thanks for the PR. I don't see the encrypted build diagnostic log artifact (*.logd / builds.compact) committed on this branch yet. Please rerun the build diagnostic/encryption step with the fixed encryptly tooling and commit the generated encrypted log artifact to this PR (typically under diagnostic/).

Blocking review until the encrypted diagnostic log artifact is present.

[ghost]

Thanks for the PR. I don't see the encrypted build diagnostic log artifact (*.logd / builds.compact) committed on this branch yet. Please rerun the build diagnostic/encryption step with the fixed encryptly tooling and commit the generated encrypted log artifact to this PR (typically under diagnostic/).

Blocking review until the encrypted diagnostic log artifact is present.