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.

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

"do not mock" test fails in interactive console #61

Closed
TWiStErRob opened this issue Dec 8, 2023 · 7 comments · Fixed by #69
Closed

"do not mock" test fails in interactive console #61

TWiStErRob opened this issue Dec 8, 2023 · 7 comments · Fixed by #69
Labels
bug Something isn't working tests

Comments

@TWiStErRob
Copy link
Contributor

TWiStErRob commented Dec 8, 2023 •
edited
Loading

To Reproduce

  1. Clone this repo
  2. npm test or npx jest

Expected behavior
Pass

Logs
image

Additional context
Notice that the expected 200 is yellow. That's part of the string somehow!
@TWiStErRob
Copy link
Contributor Author

TWiStErRob commented Dec 8, 2023 •
edited
Loading

Workaround: npm run test | cat on unix/linux/mac

@shubhbapna shubhbapna added bug Something isn't working tests labels Dec 27, 2023
@shubhbapna
Copy link
Collaborator

Interestingly, I am not able to reproduce it. Could you share some more details about the environment you were running this in? Maybe the OS and the node version?

@TWiStErRob
Copy link
Contributor Author

I'm away from work, but AFAIR:
Node 20.10.0
MacOS is latest as far as M1 Pro gets updated I guess it was macOS Sonoma | 14.1.2
and ZSH.

@shubhbapna
Copy link
Collaborator

I am not able to test it on a mac but at least when I am using node 20 I am not seeing this issue. Once you are back, can you test it again and see if this is still showing up?

@TWiStErRob
Copy link
Contributor Author

TWiStErRob commented Dec 28, 2023 •
edited
Loading

Exact same problem on Windows 10 with Node v18.17.1, and cmd shell:
image

Do you have colors in your output?

@shubhbapna
Copy link
Collaborator

I do have colors in my output

@shubhbapna
Copy link
Collaborator

Oh I was able to reproduce it now. Not sure what changed though

@shubhbapna shubhbapna mentioned this issue Jan 12, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore: fix "do not mock" unit test shubhbapna/act-js
2 participants
@TWiStErRob @shubhbapna