Skip to content

test: cover Builder Circle pool modes - #43

Merged
FrankezeCode merged 1 commit into
BetterDevOrg:mainfrom
lntutor:feat/builder-circle-pool-mode-tests
Aug 2, 2026
Merged

test: cover Builder Circle pool modes#43
FrankezeCode merged 1 commit into
BetterDevOrg:mainfrom
lntutor:feat/builder-circle-pool-mode-tests

Conversation

@lntutor

@lntutor lntutor commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds coverage for parseBuilderCirclePoolMode across supported modes,
invalid and missing values, and mixed-case input.

Related issue

Fixes #39

Type of change

  • Bug fix
  • New feature
  • Documentation
  • Other: tests

How to test

  1. Run npx tsx --test test/BuilderCircleMatching.ts.
  2. Confirm all nine tests pass.

Checklist

  • I ran npm run build (the current main branch has unrelated lint errors in meetup pages)
  • I did not commit secrets or .env.local
  • I read CONTRIBUTING.md

@lntutor

lntutor commented Aug 1, 2026

Copy link
Copy Markdown
Contributor Author

Current head 6edf0ef is mergeable, and I do not see any unresolved review threads or contributor-side follow-up remaining on the current PR state. This looks ready for maintainer review when someone has a chance.

@FrankezeCode FrankezeCode left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @Intutor for the clean test coverage for parseBuilderCirclePoolMode.
Merged! ✅
This closes the pool-mode parser testing for hybrid Builder Circles.

@FrankezeCode
FrankezeCode merged commit 2d1afea into BetterDevOrg:main Aug 2, 2026
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.

[Good first issue]: Add unit tests for parseBuilderCirclePoolMode

2 participants