-
Notifications
You must be signed in to change notification settings - Fork 34
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.
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
base: dev
Are you sure you want to change the base?
Added test cases for prepublishOnly script #563
Conversation
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>
Test Results: ✅ PASSEDRun at: 2025-12-31T05:22:52.100Z Summary: |
michielvandergeest
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the tests run correctly MacOS, however there is an issue with the release command
Test Results: ✅ PASSEDRun at: 2026-01-06T06:38:51.333Z Summary: |
Signed-off-by: Suresh Kumar Gangumalla <sureshkumar.gangumalla@invecas.com>
Test Results: ✅ PASSEDRun at: 2026-01-06T06:49:09.691Z Summary: |
michielvandergeest
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Uh oh!
There was an error while loading. Please reload this page.