Skip to content

test: Add missing test:realsvc:tinylicious:report test scripts to packages to ensure tests get run by CI#27667

Open
Josmithr wants to merge 9 commits into
microsoft:mainfrom
Josmithr:tinylicious-client/fix-tests
Open

test: Add missing test:realsvc:tinylicious:report test scripts to packages to ensure tests get run by CI#27667
Josmithr wants to merge 9 commits into
microsoft:mainfrom
Josmithr:tinylicious-client/fix-tests

Conversation

@Josmithr

@Josmithr Josmithr commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Add missing test script to ensure the package's tests are actually run by our pipelines. Also cleans up some related infra/code and documents current assumptions / limitations of the Tinylicious tests.

@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

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

Based on the diff (163 lines, 12 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

@Josmithr Josmithr changed the title test: Add missing test script test: Add missing test scripts Jul 10, 2026
@Josmithr Josmithr changed the title test: Add missing test scripts test: Add missing test:realsvc:tinylicious:report test scripts to packages to ensure tests get run by CI Jul 11, 2026
@Josmithr Josmithr marked this pull request as ready for review July 11, 2026 00:15
@Josmithr Josmithr requested review from a team, CraigMacomber and Copilot July 11, 2026 00:15

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot reviewed 11 out of 12 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

@github-actions

Copy link
Copy Markdown
Contributor

Bundle size comparison

Base commit: 9c7c1cbf75d76d635aa7cdd33c02a3522a86e97c
Head commit: 0111136892ec8fdd2ba61b7cfddf77cb1ffc5b03

Notable changes

  • 🔴 fluidFramework.js: parsed 392807 → 394163 (+1356), gzip 111526 → 111941 (+415)
  • 🔴 sharedTree.js: parsed 382194 → 383543 (+1349), gzip 108914 → 109321 (+407)
Per-bundle deltas

@fluid-example/bundle-size-tests

  • azureClient.js: parsed 619098 → 619154 (+56), gzip 164828 → 164870 (+42)
  • odspClient.js: parsed 591824 → 591880 (+56), gzip 158938 → 158980 (+42)
  • aqueduct.js: parsed 525587 → 525622 (+35), gzip 140703 → 140733 (+30)
  • 🔴 fluidFramework.js: parsed 392807 → 394163 (+1356), gzip 111526 → 111941 (+415)
  • 🔴 sharedTree.js: parsed 382194 → 383543 (+1349), gzip 108914 → 109321 (+407)
  • containerRuntime.js: parsed 303937 → 303951 (+14), gzip 83200 → 83206 (+6)
  • sharedString.js: parsed 175984 → 175991 (+7), gzip 49445 → 49453 (+8)
  • experimentalSharedTree.js: parsed 160798 → 160798 (0), gzip 45804 → 45804 (0)
  • matrix.js: parsed 159845 → 159852 (+7), gzip 45411 → 45418 (+7)
  • loader.js: parsed 145256 → 145270 (+14), gzip 39063 → 39077 (+14)
  • odspDriver.js: parsed 104329 → 104350 (+21), gzip 32625 → 32635 (+10)
  • directory.js: parsed 66616 → 66623 (+7), gzip 18532 → 18541 (+9)
  • 748.js: parsed 58793 → 58793 (0), gzip 17827 → 17827 (0)
  • map.js: parsed 46709 → 46716 (+7), gzip 14310 → 14318 (+8)
  • odspPrefetchSnapshot.js: parsed 45642 → 45656 (+14), gzip 15277 → 15285 (+8)
  • 985.js: parsed 44491 → 44491 (0), gzip 13726 → 13726 (0)
  • summarizerDelayLoadedModule.js: parsed 30749 → 30749 (0), gzip 7753 → 7753 (0)
  • socketModule.js: parsed 26476 → 26483 (+7), gzip 7887 → 7895 (+8)
  • createNewModule.js: parsed 12480 → 12480 (0), gzip 4786 → 4786 (0)
  • summaryModule.js: parsed 3797 → 3797 (0), gzip 1860 → 1860 (0)
  • connectionState.js: parsed 724 → 724 (0), gzip 429 → 429 (0)
  • sharedTreeAttributes.js: parsed 666 → 673 (+7), gzip 432 → 441 (+9)
  • debugAssert.js: parsed 429 → 429 (0), gzip 299 → 299 (0)
  • FluidFramework-HashFallback.js: parsed 422 → 422 (0), gzip 316 → 316 (0)

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.

2 participants