Skip to content

Added test cases for prepublishOnly script #563

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.

Sign up for GitHub

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

Jump to bottom
Open
suresh-gangumalla wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Added test cases for prepublishOnly script #563

suresh-gangumalla wants to merge 1 commit into from

Conversation

@suresh-gangumalla
Copy link
Collaborator

@suresh-gangumalla suresh-gangumalla commented Dec 31, 2025 •
edited
Loading

  1. resovled issue with including .test.js files inside node-modules directory in windows OS
  2. Exporting pre publish functions to add unit test cases
  3. Using Sinon Stubs to mimic different function returns instead of depending on original functions implementation which results to reading, copying, writing .org.js files of src/components. With stubs mimic the behaviour to match the scope of test case

@suresh-gangumalla
Added test cases for prepublishOnly script
e3816b2 
1. resovled issue with including .test.js files inside
node-modules directory in windows OS
2. Exporting pre publish functions to add unit test cases
3. Using Sinon Stubs to mimic different functions returns instead
of depending on original functions implementation which results to
reading, copying, writing .org.js files of src/components.
With stubs mimic the behaviour to match the scope of test case

Signed-off-by: Suresh Kumar Gangumalla <sureshkumar.gangumalla@invecas.com>
@suresh-gangumalla suresh-gangumalla requested review from michielvandergeest and removed request for michielvandergeest December 31, 2025 05:16
@github-actions
Copy link

github-actions bot commented Dec 31, 2025

Test Results: ✅ PASSED

Run at: 2025-12-31T05:22:52.100Z

Summary:
passed: 960 failed: 0 of 960 tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@michielvandergeest michielvandergeest Awaiting requested review from michielvandergeest michielvandergeest is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@suresh-gangumalla