Skip to content
This repository was archived by the owner on Jan 18, 2026. It is now read-only.

chore: convert unit test to ts#50

Merged
xiaozhenliu-gg5 merged 15 commits intomasterfrom
unit_test_ts
Oct 23, 2019
Merged

chore: convert unit test to ts#50
xiaozhenliu-gg5 merged 15 commits intomasterfrom
unit_test_ts

Conversation

@xiaozhenliu-gg5
Copy link
Contributor

@xiaozhenliu-gg5 xiaozhenliu-gg5 commented Oct 15, 2019

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 15, 2019
@alexander-fenster
Copy link
Contributor

Should we also change the npm test script to run a script from build/test, not from test?

@xiaozhenliu-gg5
Copy link
Contributor Author

In package.json of generated client library, npm test runs mocha build/test now.

Copy link
Contributor

@alexander-fenster alexander-fenster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove cp -r test build/ from the generated package.json. Other than that, it looks good!

@xiaozhenliu-gg5 xiaozhenliu-gg5 merged commit 5aaa5d1 into master Oct 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

generated unit test should be in TypeScript

3 participants