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

Add end-to-end tests for CLI commands #370

Open
4 of 9 tasks
antoineco opened this issue Apr 16, 2023 · 1 comment
Open
4 of 9 tasks

Add end-to-end tests for CLI commands #370

antoineco opened this issue Apr 16, 2023 · 1 comment
Labels
area/test Issue or PR is related to test(s) kind/enhancement New feature or request priority/low Nice to have issues thatare not urgent, but can be nagging.

Comments

@antoineco
Copy link
Contributor

antoineco commented Apr 16, 2023

Feature request summary

There is a lot to be done in terms of testing. For a start, we could focus on high level "e2e" tests and execute each command of the CLI tool against a sample unikernel project.

We could start with one project + one test case per command, then expand as we see fit.

This doesn't provide a feedback loop as quick as unit tests, but would already allow us to catch major regressions in the CI.

E2E tests

  1. priority/low
    craciunoiuc
  2. priority/low
  3. priority/low
    craciunoiuc
  4. priority/low
    craciunoiuc
  5. area/test kind/test
    nderjung
  6. area/test kind/enhancement
    nderjung
  7. craciunoiuc
  8. craciunoiuc
  9. priority/none
@antoineco
Copy link
Contributor Author

antoineco commented May 2, 2023

I think our next focus should be on kraft build.

It would be very nice to have an initial test that just

  • initializes a temporary project from a few fixture files (helloworld.c might even be enough?)
  • writes a kraft.yaml in that project
  • pulls the dependencies
  • builds a few targets

That would cover several subsystems at once. An ideal case for an e2e test.

@craciunoiuc craciunoiuc added the priority/low Nice to have issues thatare not urgent, but can be nagging. label Feb 29, 2024
@antoineco antoineco removed their assignment Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/test Issue or PR is related to test(s) kind/enhancement New feature or request priority/low Nice to have issues thatare not urgent, but can be nagging.
Projects
Status: Ready
Status: 🧊 Icebox
Development

No branches or pull requests

3 participants