[CCA Diversity] better integrations test coverage

cyclejs-community / create-cycle-app

Create Cycle.js apps with no build configuration.

ISC License

237 stars 21 forks source link

[CCA Diversity] better integrations test coverage #104

Open nickbalestra opened 7 years ago

nickbalestra commented 7 years ago

We could now rely on installScripts.js to pass prompt answers and test multiple combinations by snapshotting results, or simply add similar tests to the cycle-script repo as well. This make #101 it a good point for keeping questions also within the CLI as will make testing this easier due to init() being able to accept those params.

nickbalestra commented 7 years ago

Now that we got the typescript files in we should probably add tests as next thing. What you think @jvanbruegge ?

jvanbruegge commented 7 years ago

yeah, integration tests would be good however i would wait with #94 until #103

nickbalestra commented 7 years ago

Yes, perhaps after #103 but before we move forward after that (#94 goes with #93 i guess).

jvanbruegge commented 7 years ago

the integration tests can be written before #103 Just not the tests for the template