search

HEPCloud / decisionengine_modules

Apache License 2.0
2 stars 19 forks source link

Added unit tests for dem_htcondor_slots_status_count, dem_htcondor_cores_count, dem_htcondor_memory_count metrics #497

Closed IlyaBaburashvili closed 4 months ago

IlyaBaburashvili commented 5 months ago

Unit tests use cs.fixture to mock Condor and compare the metric values to values in expected_metric_values.json.

pep8speaks commented 4 months ago

Hello @IlyaBaburashvili! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:

Comment last updated at 2024-04-12 16:25:58 UTC
mambelli commented 4 months ago

This PR was superseeded by #498 Closing without merging