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.

Sign up for GitHub

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

Jump to bottom

restructure and addition of integration-tests in the CI #96

Merged
merged 7 commits into from
Nov 11, 2024
Merged

restructure and addition of integration-tests in the CI #96

merged 7 commits into from
Nov 11, 2024

Conversation

christophkloeffel
Copy link
Contributor

@christophkloeffel christophkloeffel commented Oct 12, 2024 •
edited
Loading

  • renames and restructures all test folders consistently to the ones in the TRLC repo
  • adds the integrations-tests run to the CI
  • adds make targets make test (system-tests, unit-tests) and make test-all (integration-tests, system-tests, unit-tests) and make clang-tidythat is now used in the CI to build the dependency clang-tidy for the lobster-cpp tool
  • it caches clang-tidy for subsequent runs of the CI. retention period of the cache is 7 days
  • fixes invoking tailwith an outdated syntax that lead to an error on a newer version of GNU tail

@christophkloeffel christophkloeffel requested a review from a team as a code owner October 12, 2024 03:54
@christophkloeffel christophkloeffel force-pushed the feature/add-integration-tests-to-ci branch 3 times, most recently from d8c82cf to f3b5358 Compare October 12, 2024 04:05
@christophkloeffel christophkloeffel added the internal Affects the CI, tests or refactorings only, not relevant to the end-user label Oct 12, 2024
@christophkloeffel christophkloeffel mentioned this pull request Oct 12, 2024
Closed
phiwuu
phiwuu previously approved these changes Oct 14, 2024
View reviewed changes
phiwuu
phiwuu previously approved these changes Oct 18, 2024
View reviewed changes
@christophkloeffel christophkloeffel force-pushed the feature/add-integration-tests-to-ci branch 4 times, most recently from 2fa8950 to 403f994 Compare October 21, 2024 10:24
@christophkloeffel christophkloeffel linked an issue Oct 23, 2024 that may be closed by this pull request
integration tests use deprecated TRLC check file #101
Closed
@christophkloeffel christophkloeffel force-pushed the feature/add-integration-tests-to-ci branch 2 times, most recently from 49ddba9 to 7d9795d Compare October 23, 2024 15:10
@christophkloeffel christophkloeffel force-pushed the feature/add-integration-tests-to-ci branch 3 times, most recently from 72b9582 to 9d1346d Compare November 8, 2024 21:00
@christophkloeffel christophkloeffel force-pushed the feature/add-integration-tests-to-ci branch 2 times, most recently from 9e9d5dc to 261302b Compare November 9, 2024 20:51
@phiwuu phiwuu force-pushed the feature/add-integration-tests-to-ci branch from 261302b to 9d79f3b Compare November 11, 2024 17:50
@phiwuu phiwuu merged commit f69c82b into main Nov 11, 2024
21 checks passed
@phiwuu phiwuu deleted the feature/add-integration-tests-to-ci branch November 11, 2024 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@phiwuu phiwuu phiwuu approved these changes

Assignees
No one assigned
Labels
internal Affects the CI, tests or refactorings only, not relevant to the end-user
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

integration tests use deprecated TRLC check file
2 participants
@christophkloeffel @phiwuu