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

Broken Tests #369

Closed
peternewman opened this issue Mar 19, 2022 · 5 comments
Closed

Broken Tests #369

peternewman opened this issue Mar 19, 2022 · 5 comments
Labels

Comments

@peternewman
Copy link
Contributor

Describe the bug

Tests in 68fbd44 don't work:

2022-03-19T02:47:22.8357254Z testPrometheus
2022-03-19T02:47:23.3419424Z ./unit_tests.sh: 401: eval: assertContains: not found
2022-03-19T02:47:23.3419775Z ./unit_tests.sh: 402: eval: assertContains: not found
2022-03-19T02:47:23.3420078Z ./unit_tests.sh: 403: eval: assertContains: not found

They require shunit2 2.1.8:
https://github.com/kward/shunit2/blob/master/doc/CHANGES-2.1.md#changes-with-218

Which isn't yet in the latest Ubuntu:
https://packages.ubuntu.com/jammy/shunit2

Upstream issue raised here:
kward/shunit2#155

To Reproduce

Run tests with shunit2 2.1.7 or below:
https://github.com/matteocorti/check_ssl_cert/runs/5583298905?check_suite_focus=true

Expected behavior

Tests work

System (please complete the following information):

GitHub Actions

@peternewman
Copy link
Contributor Author

Obviously this new version is unlikely to ever be backported to 16.04 etc...

@williamdes
Copy link

Let me know what you need I will see if I can do something
I asked to take over the package maintenance

@matteocorti
Copy link
Owner

The tests are failing without consequences (no global fail). I would leave like that ...

@matteocorti
Copy link
Owner

I rewrote the tests using a grep so that it works even with older versions

@peternewman
Copy link
Contributor Author

The tests are failing without consequences (no global fail). I would leave like that ...

I was just a bit worried it goes from not running silently to actually failing (i.e. the test no longer passes on systems running the newest version).

I rewrote the tests using a grep so that it works even with older versions

Amazing, that seems like the perfect outcome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants