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

CI: add NIT tests for nut-scanner/libupsclient #1814

Merged
merged 11 commits into from
Jan 16, 2023

Conversation

jimklimov
Copy link
Member

@jimklimov jimklimov commented Jan 15, 2023

While the test is rudimentary, it allows to check that nut-scanner tool basically works, and the C client library talks to upsd well. In practice during tests in build tree it often depends a lot on libtool wrapper preparing the environment that allows (or not) to find the library at run-time search, so the tests are skipped (not failed) when it says the library could not be found.

Also addresses a poorly-understood message on console when "NewNUT" (avahi) discovery fails.

Note: initially hoped this would be a test for sanity-checker from #1811, with dummy-ups configs providing some bogus or duplicate data, but NUT discovery only reports the client/relay connection point, not further details of the already running driver's config or data.

Closes: #1815

@jimklimov jimklimov added enhancement nut-scanner CI Entries related to continuous integration infrastructure (historically also recipes like Makefiles) labels Jan 15, 2023
@jimklimov jimklimov added this to the 2.8.1 milestone Jan 15, 2023
@AppVeyorBot

This comment was marked as resolved.

@jimklimov
Copy link
Member Author

AppVeyor: Cannot load NUT library (libupsclient.dll) : The specified module could not be found.. NUT search disabled. ugh :\

@jimklimov jimklimov merged commit cd9bd31 into networkupstools:master Jan 16, 2023
@jimklimov jimklimov deleted the issue-1810 branch January 16, 2023 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Entries related to continuous integration infrastructure (historically also recipes like Makefiles) enhancement nut-scanner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

nut-scanner should add non-standard NUT ports to "nutclient" configs
2 participants