Skip to content

test: add final scorecard composed e2e#1203

Merged
chhoumann merged 1 commit into
masterfrom
scorecard/17-final-scorecard-e2e
May 14, 2026
Merged

test: add final scorecard composed e2e#1203
chhoumann merged 1 commit into
masterfrom
scorecard/17-final-scorecard-e2e

Conversation

@chhoumann
Copy link
Copy Markdown
Owner

@chhoumann chhoumann commented May 13, 2026

Add final composed e2e coverage for the completed scorecard remediation stack.

This final stack PR adds Obsidian-backed composed-flow coverage that exercises the remediated areas together rather than only as isolated unit tests. It is intended as the acceptance layer for the preceding remediation PRs.

Review focus:

  • scorecard-composed-flows.test.ts coverage of macro/template/capture/multi flows.
  • Whether the e2e assertions reflect realistic QuickAdd usage in the dev vault.
  • Any fragility in the composed Obsidian CLI-backed flow setup.

Stack position: 17/17, based on scorecard/16-decision-bucket.

Validated as part of the completed scorecard mission with bun run build-with-lint, bun run test, bun run build, bun run test:e2e, and Obsidian dev vault reload/smoke checks.

Summary by CodeRabbit

  • Tests
    • Added comprehensive end-to-end test suite for scorecard template and capture workflows, including macro composition and multi-choice routing scenarios with automated failure artifact capture for improved debugging and test reliability.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 13, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 5c248e6e-622f-415a-a556-a389e37c1bca

📥 Commits

Reviewing files that changed from the base of the PR and between cf9d8dd and 0b36cb6.

📒 Files selected for processing (1)
  • tests/e2e/scorecard-composed-flows.test.ts

📝 Walkthrough

Walkthrough

Adds a Vitest E2E test module that sets up vault-run locking and a sandbox, provides QuickAdd choice builders and execution/teardown helpers, patches QuickAdd with composed macro and routed multi-child choices, and runs two acceptance tests validating template output and routed multi-capture behavior.

Changes

Scorecard Composition E2E Tests

Layer / File(s) Summary
Module setup and lifecycle management
tests/e2e/scorecard-composed-flows.test.ts
Module imports vitest and Obsidian e2e helpers, defines vault/plugin/test constants and shared wait options, and provides module-level beforeAll/afterAll that acquire/publish the vault run lock, initialize the sandbox API, and perform resilient teardown.
Test utilities and choice helpers
tests/e2e/scorecard-composed-flows.test.ts
Adds builder functions for QuickAdd Template, Capture, Macro, and Multi choices, a helper to remove prior test choices by __qa-scorecard- prefix, runChoice to execute choices, and runTeardownStep to aggregate teardown errors.
Suite-level setup and per-test failure handling
tests/e2e/scorecard-composed-flows.test.ts
Suite beforeAll writes scorecard-template.md, deletes previous sandbox outputs/targets, patches QuickAdd plugin data with the composed macro (template → capture) and routed multi-child capture, reloads QuickAdd, and beforeEach captures failure artifacts on test failures.
Acceptance tests for composed flows
tests/e2e/scorecard-composed-flows.test.ts
First test runs the composed macro and asserts created template-output and capture-target files with expected bodies. Second test verifies routing state via quickadd:list, runs the routed multi-capture child, and asserts the capture target contains the expected multi child body.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

🐇 A scorecard test hops into view,
Composing choices, old and new,
Locks held tight while templates grow,
Macro runs and routed paths flow.
Two tests confirm the outputs true.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch scorecard/17-final-scorecard-e2e

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

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

@chhoumann chhoumann marked this pull request as ready for review May 13, 2026 15:30
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 07fcac7e48

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread tests/e2e/scorecard-composed-flows.test.ts
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 13, 2026

Deploying quickadd with  Cloudflare Pages  Cloudflare Pages

Latest commit: 0b36cb6
Status:⚡️  Build in progress...

View logs

@chhoumann chhoumann force-pushed the scorecard/17-final-scorecard-e2e branch from 07fcac7 to 4ae0463 Compare May 14, 2026 09:35
@chhoumann chhoumann force-pushed the scorecard/16-decision-bucket branch 2 times, most recently from 3cec176 to fbb6557 Compare May 14, 2026 09:54
@chhoumann chhoumann force-pushed the scorecard/17-final-scorecard-e2e branch from 4ae0463 to dc7ce23 Compare May 14, 2026 09:54
@chhoumann chhoumann force-pushed the scorecard/16-decision-bucket branch from fbb6557 to bb391b3 Compare May 14, 2026 10:20
@chhoumann chhoumann force-pushed the scorecard/17-final-scorecard-e2e branch from dc7ce23 to 51d2398 Compare May 14, 2026 10:21
@chhoumann chhoumann force-pushed the scorecard/16-decision-bucket branch from bb391b3 to b3618f3 Compare May 14, 2026 10:28
@chhoumann chhoumann force-pushed the scorecard/17-final-scorecard-e2e branch from 51d2398 to 08e0c4b Compare May 14, 2026 10:28
@chhoumann chhoumann force-pushed the scorecard/16-decision-bucket branch from b3618f3 to 255c2ad Compare May 14, 2026 10:45
@chhoumann chhoumann force-pushed the scorecard/17-final-scorecard-e2e branch from 08e0c4b to bafb30d Compare May 14, 2026 10:45
@chhoumann chhoumann force-pushed the scorecard/16-decision-bucket branch from 255c2ad to 0340967 Compare May 14, 2026 10:50
@chhoumann chhoumann force-pushed the scorecard/17-final-scorecard-e2e branch from bafb30d to dfca303 Compare May 14, 2026 10:50
@chhoumann chhoumann force-pushed the scorecard/16-decision-bucket branch from 0340967 to 795ad92 Compare May 14, 2026 10:51
@chhoumann chhoumann force-pushed the scorecard/17-final-scorecard-e2e branch from dfca303 to 69a7ebf Compare May 14, 2026 10:51
@chhoumann chhoumann force-pushed the scorecard/16-decision-bucket branch from 795ad92 to e31d421 Compare May 14, 2026 10:52
@chhoumann chhoumann force-pushed the scorecard/17-final-scorecard-e2e branch from 69a7ebf to f5a91a3 Compare May 14, 2026 10:52
@chhoumann chhoumann force-pushed the scorecard/16-decision-bucket branch from e31d421 to 2fddda9 Compare May 14, 2026 10:53
@chhoumann chhoumann force-pushed the scorecard/17-final-scorecard-e2e branch from f5a91a3 to 082064c Compare May 14, 2026 10:53
@chhoumann chhoumann force-pushed the scorecard/16-decision-bucket branch from 2fddda9 to 24ddb96 Compare May 14, 2026 10:54
@chhoumann chhoumann force-pushed the scorecard/17-final-scorecard-e2e branch from 082064c to b7f1f64 Compare May 14, 2026 10:54
@chhoumann chhoumann force-pushed the scorecard/16-decision-bucket branch from 24ddb96 to ecdd896 Compare May 14, 2026 10:54
@chhoumann chhoumann force-pushed the scorecard/17-final-scorecard-e2e branch from b7f1f64 to 3271cf1 Compare May 14, 2026 10:55
@chhoumann chhoumann force-pushed the scorecard/16-decision-bucket branch from ecdd896 to 9f3abac Compare May 14, 2026 10:55
@chhoumann chhoumann force-pushed the scorecard/17-final-scorecard-e2e branch from 3271cf1 to 0a65a2f Compare May 14, 2026 10:55
@chhoumann chhoumann force-pushed the scorecard/16-decision-bucket branch from 9f3abac to a871916 Compare May 14, 2026 10:56
@chhoumann chhoumann force-pushed the scorecard/17-final-scorecard-e2e branch from 0a65a2f to f1b450c Compare May 14, 2026 10:56
@chhoumann chhoumann force-pushed the scorecard/16-decision-bucket branch from a871916 to 489c0fb Compare May 14, 2026 10:56
@chhoumann chhoumann force-pushed the scorecard/17-final-scorecard-e2e branch from f1b450c to 9efc0c3 Compare May 14, 2026 10:56
@chhoumann chhoumann force-pushed the scorecard/16-decision-bucket branch from 489c0fb to a01d773 Compare May 14, 2026 10:56
@chhoumann chhoumann force-pushed the scorecard/17-final-scorecard-e2e branch from 9efc0c3 to ddb6207 Compare May 14, 2026 10:57
@chhoumann chhoumann force-pushed the scorecard/16-decision-bucket branch from a01d773 to 1b1dfce Compare May 14, 2026 10:57
@chhoumann chhoumann force-pushed the scorecard/17-final-scorecard-e2e branch from ddb6207 to 996e682 Compare May 14, 2026 10:57
@chhoumann chhoumann force-pushed the scorecard/16-decision-bucket branch from 1b1dfce to c1eb845 Compare May 14, 2026 10:57
@chhoumann chhoumann force-pushed the scorecard/17-final-scorecard-e2e branch from 996e682 to fc24024 Compare May 14, 2026 10:57
@chhoumann chhoumann force-pushed the scorecard/17-final-scorecard-e2e branch from fc24024 to 0b36cb6 Compare May 14, 2026 10:59
@chhoumann chhoumann changed the base branch from scorecard/16-decision-bucket to master May 14, 2026 10:59
@chhoumann chhoumann merged commit 9592b84 into master May 14, 2026
2 of 4 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.

1 participant