Skip to content

Commit

Permalink
Refactor resource monitor code and remove unused methods
Browse files Browse the repository at this point in the history
- Removed `set_resource_log_enabled` and `is_resource_log_enabled` methods from `CoreResourceMonitor` class.
- Updated corresponding test case in `test_resource_monitor.py`.
- No functional changes made, only code cleanup.
  • Loading branch information
drew2a committed Nov 27, 2023
1 parent 5c309d3 commit 75a105e
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -55,12 +55,6 @@ def check_resources(self):
# Additionally, record the disk and notify on low disk space available.
self.record_disk_usage()

def set_resource_log_enabled(self, enabled):
self.resource_log_enabled = enabled

def is_resource_log_enabled(self):
return self.resource_log_enabled

def record_disk_usage(self, recorded_at=None):
recorded_at = recorded_at or time.time()

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -96,11 +96,6 @@ def test_get_disk_usage_exception(resource_monitor: CoreResourceMonitor):
assert not resource_monitor.get_disk_usage()


def test_enable_resource_log(resource_monitor):
resource_monitor.set_resource_log_enabled(True)
assert resource_monitor.is_resource_log_enabled()


def test_profiler(resource_monitor):
"""
Test the profiler start(), stop() methods.
Expand Down

0 comments on commit 75a105e

Please sign in to comment.