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 test devoted to ctest #4186

Open
wants to merge 59 commits into
base: main
Choose a base branch
from

Conversation

hippo91
Copy link
Contributor

@hippo91 hippo91 commented Nov 24, 2024

This changes test system

The following changes are proposed:

This PR is a proposal to add end to end test devoted to ctest part of the extension.

It adds a directory named single-root-ctest in the end-to-end-tests directory.

It also changes slightly the source directory in order:

  • to avoid early return in the refreshTests method (ctest.ts file) when in testing mode;
  • to change the return code of the ctest method (cmakeProject.ts) so that successfull test return 0 and failure returns -1.

The purpose of this change

When doing change to the ctest part of the extension, i did not feel very comfortable not being backed up by tests.
This PR tries to circumvent this.

Guillaume Peillex and others added 30 commits November 24, 2024 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant