Handles keyinterrupt and add func tests to CI

canonical / prometheus-juju-exporter

prometheus-juju-exporter snap collects machines' running status in all models under a Juju controller.

GNU General Public License v3.0

2 stars 8 forks source link

Handles keyinterrupt and add func tests to CI #15

Closed agileshaw closed 1 year ago

agileshaw commented 1 year ago

This PR removes the **kwargs in Exporter.trigger() and Exporter.run() function because it only serves testing purposes. Instead, gracefully shutdown with key interrupt is implemented.

This PR also adds functional tests to the CI workflow for a more complete validation.

Fixes: #8

agileshaw commented 1 year ago

Functional tests configured and passed: https://github.com/canonical/prometheus-juju-exporter/actions/runs/3906158572/jobs/6674011154