TeselaGen / fsml.org

A BioMADE Collaboration Project
https://fsml.org
13 stars 3 forks source link

Implement e2e CI tests for CLI tool #56

Open tgadam opened 2 years ago

tgadam commented 2 years ago

Implement e2e CI tests for CLI tool using GitHub Actions.

tgadam commented 2 years ago

@tgreen7 is this working yet? Or are just the unit tests working?

eabeliuk commented 2 years ago

We could at least test the version command asap, and then add the other tests.

eabeliuk commented 2 years ago

@tnrich we might need some help from you on this one, at least adding the first CI test.

tgadam commented 2 years ago

@tnrich @tgreen7 can you take a look at this?

tnrich commented 2 years ago

@tgadam what do we want to use for the e2e tests? Also is this about testing the compiled executable files or just testing the uncompiled cli tool?

tgadam commented 2 years ago

@tnrich ideally we can use the Deno test and coverage tools with the compiled executable files... however I'm not sure that will actually work so we may need to use them with the uncompiled tool and then have a few sanity tests against the compiled tool that won't have coverage.

tgadam commented 2 years ago

@tnrich do you have any more questions about this?

tnrich commented 2 years ago

@tgadam are we still sticking with Deno for now?

tgadam commented 2 years ago

Yeah... for now

tgadam commented 2 years ago

@tnrich for now we want e2e tests but not with the compiled version of the tool because we're not sure how we're going to compile it.

eabeliuk commented 2 years ago

We don't have any plans to generate a compiled version of the CLI. @tgadam 's previous comment still applies: for now, we need e2e tests. We have some installation packages using oclif. @tnrich , could you look into this? Thanks!