HERA-Team / hera_mc

The HERA monitor and control (M&C) system.
BSD 2-Clause "Simplified" License
3 stars 3 forks source link

Fix bugs in code and tests that were causing code and tests to fail onsite #531

Closed bhazelton closed 3 years ago

bhazelton commented 3 years ago

These are fixes following the merge of PR #518 which led to both code errors and testing errors onsite.

Tests pass with these fixes.

codecov[bot] commented 3 years ago

Codecov Report

Merging #531 (4289855) into main (f97d685) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #531   +/-   ##
=======================================
  Coverage   96.85%   96.85%           
=======================================
  Files          33       33           
  Lines        4707     4707           
=======================================
  Hits         4559     4559           
  Misses        148      148           
Impacted Files Coverage Δ
hera_mc/mc_session.py 98.36% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f97d685...4289855. Read the comment docs.