Skip to content

feat(test-tools): Add with-test-port command to support tinylicious real service tests#27700

Draft
Josmithr wants to merge 16 commits into
microsoft:mainfrom
Josmithr:test-tools/support-tinylicious-scenarios
Draft

feat(test-tools): Add with-test-port command to support tinylicious real service tests#27700
Josmithr wants to merge 16 commits into
microsoft:mainfrom
Josmithr:test-tools/support-tinylicious-scenarios

Conversation

@Josmithr

@Josmithr Josmithr commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Adds with-test-port command, and updates getTestPort function so that they can be used to dynamically assign ports to tinylicious real service tests. Currently, packages that leverage these tests are required to hard-code repo-globally unique ports (in multiple locations, which must match) for their tinylicious service instances. This functionality will allow them to get their ports dynamically in a way that guarantees no conflicts between packages.

This is part of a multi-PR effort to allow our repo's tinylicious real service tests to get port assignments dynamically.

This library update needs to be published before the changes can be used in the client packages. But the whole story can be previewed here: #27701.

@github-actions

github-actions Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Hi! Thank you for opening this PR. Want me to review it?

Based on the diff (311 lines, 9 files), I've queued these reviewers:

  • Correctness — logic errors, race conditions, lifecycle issues
  • Security — vulnerabilities, secret exposure, injection
  • API Compatibility — breaking changes, release tags, type design
  • Performance — algorithmic regressions, memory leaks
  • Testing — coverage gaps, hollow tests

How this works

  • Adjust the reviewer set by ticking/unticking boxes above. Reviewer toggles alone don't trigger anything.

  • Tick Start review below to dispatch the review fleet.

  • After review finishes, tick Start review again to request another run — it auto-resets after each dispatch.

  • This comment updates as new commits land; your reviewer selections are preserved.

  • Start review

@github-actions

Copy link
Copy Markdown
Contributor

🔗 No broken links found! ✅

Your attention to detail is admirable.

linkcheck output

1: starting server using command "npm run serve -- --no-open"
and when url "[ 'http://127.0.0.1:3000' ]" is responding with HTTP status code 200
running tests using command "npm run check-links"


> fluid-framework-website@0.0.0 serve
> docusaurus serve --no-open

[SUCCESS] Serving "build" directory at: http://localhost:3000/

> fluid-framework-website@0.0.0 check-links
> linkcheck http://localhost:3000 --skip-file skipped-urls.txt

Crawling...

Stats:
  298156 links
    1937 destination URLs
    2187 URLs ignored
       0 warnings
       0 errors


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