Skip to content

Commit

Permalink
tes(monitor): pylint improvements
Browse files Browse the repository at this point in the history
  • Loading branch information
jacksonpradolima committed Dec 22, 2024
1 parent 50fcdfc commit 8f8198e
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 3 deletions.
1 change: 1 addition & 0 deletions .pylintrc
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ disable=
too-few-public-methods, # enum classes don't normally have methods
too-many-instance-attributes,
too-many-arguments,
too-many-positional-arguments,
import-error,
attribute-defined-outside-init,
redefined-outer-name
14 changes: 11 additions & 3 deletions coleman4hcs/tests/utils/test_monitor.py
Original file line number Diff line number Diff line change
@@ -1,9 +1,17 @@
"""
Test cases for the MonitorCollector utility in the coleman4hcs package.
These tests cover functionality including data collection, handling of temporary
buffers, and performance benchmarks.
"""
import os
from unittest.mock import MagicMock

import pandas as pd
import pytest
from unittest.mock import MagicMock
from coleman4hcs.utils.monitor import MonitorCollector

from coleman4hcs.evaluation import EvaluationMetric
from coleman4hcs.utils.monitor import MonitorCollector


@pytest.fixture
Expand Down Expand Up @@ -104,7 +112,7 @@ def test_create_file(tmp_path, monitor_collector):
monitor_collector.create_file(file_path)

assert os.path.exists(file_path)
with open(file_path, 'r') as f:
with open(file_path, 'r', encoding='utf-8') as f:
header = f.readline().strip()
assert header == ";".join(monitor_collector.col_names)

Expand Down

0 comments on commit 8f8198e

Please sign in to comment.