Skip to content

BDD 测试计划#1

Merged
Loongphy merged 3 commits intomainfrom
cursor/bdd-b172
Feb 10, 2026
Merged

BDD 测试计划#1
Loongphy merged 3 commits intomainfrom
cursor/bdd-b172

Conversation

@Loongphy
Copy link
Owner

Add BDD-style tests for registry and CLI to cover key behaviors and unhandled edge cases.

These BDD-style tests are implemented using Zig's native std.testing framework, leveraging "Given/When/Then" naming conventions and helper structures to achieve BDD's readability without introducing external dependencies. The new tests focus on critical business scenarios (e.g., auto-import, handling auth without email, account selection logic) and CLI argument parsing that were previously less covered by traditional unit tests, minimizing overlap with existing tests.


Open in Cursor Open in Web

Co-authored-by: Loongphy Wei <Loongphy@users.noreply.github.com>
@cursor
Copy link

cursor bot commented Feb 10, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@macroscopeapp
Copy link

macroscopeapp bot commented Feb 10, 2026

Adopt BDD test plan by adding BDD-style test modules, documenting conventions, and fixing cli.selectRemoveWithNumbers to start selected index population at 0

Add BDD-style tests for CLI parsing and registry sync, introduce shared BDD helpers, document BDD testing conventions, import new tests in the main test block, and set the selected indices array to start at index 0 in src/cli.zig.

📍Where to Start

Start with selectRemoveWithNumbers in src/cli.zig, then review the new helpers in src/tests/bdd_helpers.zig and the test entry points added in the test block of src/main.zig.


Macroscope summarized 91c86ac.

cursoragent and others added 2 commits February 10, 2026 14:06
Co-authored-by: Loongphy Wei <Loongphy@users.noreply.github.com>
Co-authored-by: Loongphy Wei <Loongphy@users.noreply.github.com>
@Loongphy Loongphy marked this pull request as ready for review February 10, 2026 14:17
@Loongphy
Copy link
Owner Author

@macroscope-app review

@macroscopeapp
Copy link

macroscopeapp bot commented Feb 10, 2026

Review complete — no issues found. View the results here:

@Loongphy Loongphy merged commit 894f492 into main Feb 10, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants