Skip to content

test(client): Assign tinylicious real service test ports dynamically#27701

Draft
Josmithr wants to merge 23 commits into
microsoft:mainfrom
Josmithr:client/use-updated-test-tools-for-tinylicious-tests
Draft

test(client): Assign tinylicious real service test ports dynamically#27701
Josmithr wants to merge 23 commits into
microsoft:mainfrom
Josmithr:client/use-updated-test-tools-for-tinylicious-tests

Conversation

@Josmithr

@Josmithr Josmithr commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Updates the tinylicious real service tests to leverage dynamic port assignment functionality to avoid needing to hard code workspace-globally unique port numbers (in multiple places per relevant package).

This PR updates both test-tools and tinylicious in order to accomplish this. As both of these live in separate release groups from the client, we'll need to publish updates to both libraries before we can actually enable this in client. Those changes have been extracted into separate PRs. This PR exists as an end-to-end example of how everything comes together.

tinylicious PR: #27709
test-tools PR: #27700

@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 (6845 lines, 21 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

Bundle size comparison

Base commit: e1dcf16db76b2c217bf3bd9e5de80eabc8e68ce8
Head commit: b0cb6c5a832229de62990de1409298108b7efa3c

⚠️ Comparison unavailable.

The PR's CI build failed — fix the build and the comment will update once the next run succeeds.

@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