Add tests - Githubissues

purescript / spago

🍝 PureScript package manager and build tool

BSD 3-Clause "New" or "Revised" License

793 stars 132 forks source link

Add tests #31

Closed f-f closed 5 years ago

f-f commented 5 years ago

There should be some tests so that we have some kind of assurance that we don't break stuff.

Anything is good to start, even just a bash script that installs the binary, runs some commands and checks that the return codes are all right. This should prevent simple breakage, like what happened when https://github.com/justinwoo/spacchetti-cli/commit/4e9c20e93d92b18168678dd28a070e901adc3428 lead to #29

justinwoo commented 5 years ago

These tests should probably also take into account the difference between Psc-Package projects using local-setup, insdhall and Spacchetti projects using whatever else

f-f commented 5 years ago

Yeah, probably best to change the names of the psc-package-related commands, so like

spacchetti insdhall → spacchetti psc-package-install and
spacchetti local-setup → spacchetti psc-package-local-setup

justinwoo commented 5 years ago

psc-package-insdhall, it's a big selling point for this tool

justinwoo commented 5 years ago

It also needs some deprecation warnings since many of my other projects will break

f-f commented 5 years ago

Right, then probably best to do the renaming of commands together with #23 so we don't break things