Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/lit 2921 js sdk migrate to jest #549

Open
wants to merge 152 commits into
base: master
Choose a base branch
from

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 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
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.

4 participants