Skip to content

fix(roblox): isolate co-located suite registration - #68

Merged
cayasde merged 2 commits into
mainfrom
fix/isolated-colocated-roblox-suites
Jul 21, 2026
Merged

fix(roblox): isolate co-located suite registration#68
cayasde merged 2 commits into
mainfrom
fix/isolated-colocated-roblox-suites

Conversation

@cayasde

@cayasde cayasde commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Load discovered Roblox suites once with an owner-bound registration context.
  • Keep co-located dependency tests attributed to their own modules.
  • Preserve normal top-level module side effects and exports.
  • Add a patch changeset.

Why

Prevents a consumer suite from registering tests belonging to a co-located dependency that it requires first.

Close #65

Validation

  • lutest test (150 passed)
  • mise run typecheck
  • Built and smoke-tested C:\Users\cayasde\Projects\ok\lutest.exe.

@cayasde
cayasde marked this pull request as ready for review July 21, 2026 15:16
@cayasde
cayasde merged commit 9eacd32 into main Jul 21, 2026
6 checks passed
@cayasde
cayasde deleted the fix/isolated-colocated-roblox-suites branch July 21, 2026 15:16
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.

Co-located tests register under a consumer suite when the module is required first

1 participant