Skip to content

test(app): await missing checks validation - #119

Closed
albert20260301 wants to merge 1 commit into
altertable-ai:fc/data-app-clifrom
albert20260301:fix/pr118-check-test-lint
Closed

albert20260301 wants to merge 1 commit into
altertable-ai:fc/data-app-clifrom
albert20260301:fix/pr118-check-test-lint

Conversation

@albert20260301

Copy link
Copy Markdown
Contributor

Fix the CI-only lint failure in the data-app contract test.

The assertion now awaits and captures the rejection explicitly, matching the prior failure assertion in the same test. No production behavior or public API changes.

Validation:

  • bun run lint
  • bun test src/commands/app/check.test.ts

@albert20260301

Copy link
Copy Markdown
Contributor Author

@francoischalifour could you review this?

I picked you because you authored the affected data-app contract test and own the target branch. CI is pending; the review focus is preserving the awaited rejection assertion while resolving the type-aware lint failure.

@albert20260301

Copy link
Copy Markdown
Contributor Author

Closing this now-redundant stacked repair: the current target branch already contains the lint-safe assertion and PR #118 has a fully green required workflow.

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