Skip to content

Conversation

@joshLong145
Copy link

@joshLong145 joshLong145 commented Jul 22, 2024

  • Migration of local-tests from custom test runner to Jest for standardized test runner implementations.
  • Creates a package for tinny which has been migrated from the setup directory in local-tests
  • Has migrated explicit test cases for each permutation of session generation to an implicit implementations for code reuse.

At this time we still rely on the global installation of packages into node_modules for binding to other local packages through package import statements.

To reduce the scope of this PR the migration away from global module installation has not been implemented. This will be done in a followup PR.

New Test Command

For running new jest test suite the test:e2e command can be used. CI has been migrated to this command in this PR.

@joshLong145 joshLong145 requested a review from Ansonhkg as a code owner July 22, 2024 18:19
@joshLong145 joshLong145 marked this pull request as draft July 22, 2024 18:20
@joshLong145 joshLong145 removed the splitting pr breaking down pr into smaller prs for review label Sep 9, 2024
@Ansonhkg Ansonhkg added 🚧 In Progress - Don’t Merge pull request is still under active development and should not be merged into the main branch. It sig and removed 🚧 In Progress - Don’t Merge pull request is still under active development and should not be merged into the main branch. It sig labels Oct 2, 2024
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ Ansonhkg
❌ Josh Long


Josh Long seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@Ansonhkg Ansonhkg closed this Aug 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants