Skip to content

Remove mockedFunction for Node.js test runner mock method #201

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

Merged
merged 1 commit into from
Jan 14, 2024
Merged

Remove mockedFunction for Node.js test runner mock method #201

merged 1 commit into from
Jan 14, 2024

Conversation

jean-michelet
Copy link
Contributor

@jean-michelet jean-michelet commented Jan 14, 2024

This pull request addresses issue #188: "Remove mockedFunction for Node.js test runner mock fn".

I observed that the issue was specifically related to the mocking of object methods. To address this, I have implemented the use of the mock.method() function instead of mock.fn.

I noticed that the file isArrayExpression.js was excluded from the test suite due to a missing .spec extension. I have corrected this by renaming the file to isArrayExpression.spec.js, ensuring that it is now included in the test suite.

Looking forward to your feedback and any further suggestions.

@fraxken fraxken merged commit 0f88b48 into NodeSecure:master Jan 14, 2024
@fraxken
Copy link
Member

fraxken commented Jan 14, 2024

@allcontributors please add @jean-michelet for test

Copy link
Contributor

@fraxken

I've put up a pull request to add @jean-michelet! 🎉

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