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

Adapt ts-node in test automation program #60

Closed
samchon opened this issue Jan 22, 2020 · 2 comments
Closed

Adapt ts-node in test automation program #60

samchon opened this issue Jan 22, 2020 · 2 comments
Assignees
Projects

Comments

@samchon
Copy link
Owner

@samchon samchon commented Jan 22, 2020

Instead of compiling test automation program, using ts-node would be better.

Therefore, when compiling, src/test files would be ignored and *.js.map files would not be generated. Also, the configuration file for debugging defined .vscode/launch.json would be changed.

@samchon samchon added the enhancement label Jan 22, 2020
@samchon samchon self-assigned this Jan 22, 2020
@samchon samchon added this to To do in v2.4 Update via automation Jan 22, 2020
@samchon samchon moved this from To do to In progress in v2.4 Update Jan 22, 2020
@samchon samchon closed this in 5867c52 Jan 22, 2020
v2.4 Update automation moved this from In progress to Done Jan 22, 2020
@samchon

This comment has been minimized.

Copy link
Owner Author

@samchon samchon commented Feb 8, 2020

ts-node spends too much time when using dynamic import and it makes hard to find compile error on the test automation program until executes it.

Therefore, it would better to using source-map-support instead.

@samchon samchon reopened this Feb 8, 2020
v2.4 Update automation moved this from Done to In progress Feb 8, 2020
samchon added a commit that referenced this issue Feb 8, 2020
@samchon samchon added the help wanted label Feb 8, 2020
@samchon

This comment has been minimized.

Copy link
Owner Author

@samchon samchon commented Feb 8, 2020

Decided to provide both of them

  • npm run test
  • npm run ts-test
@samchon samchon closed this in 33f60db Feb 8, 2020
v2.4 Update automation moved this from In progress to Done Feb 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
v2.4 Update
  
Done
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.