Skip to content

Commit 92836de

Browse files
committed
fix: trying class property report path
1 parent ef7eef2 commit 92836de

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

tests/tests.py

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -34,12 +34,14 @@ def get_executor(self) -> str:
3434
return "slurm"
3535

3636
def get_executor_settings(self) -> Optional[ExecutorSettingsBase]:
37+
self.REPORT_PATH = Path.cwd() / "efficiency_report_test"
38+
3739
return ExecutorSettings(
38-
efficiency_report=True,
39-
init_seconds_before_status_checks=5,
40-
efficiency_report_path=Path.cwd() / "efficiency_report_test",
41-
# seconds_between_status_checks=5,
42-
)
40+
efficiency_report=True,
41+
init_seconds_before_status_checks=5,
42+
efficiency_report_path=self.REPORT_PATH,
43+
# seconds_between_status_checks=5,
44+
)
4345

4446
def test_simple_workflow(self, tmp_path):
4547
# for an unkown reason, the efficiency report is not created
@@ -60,12 +62,9 @@ def test_simple_workflow(self, tmp_path):
6062
report_found = False
6163

6264
report_path = None
63-
# we do not need to ensure the path exists, as it is created by the
64-
# executor
65-
expected_path = Path.cwd() / "efficiency_report_test"
6665

6766
# Check if the efficiency report file exists - based on the regex pattern
68-
for fname in os.listdir(expected_path):
67+
for fname in os.listdir(self.REPORT_PATH):
6968
if pattern.match(fname):
7069
report_found = True
7170
report_path = os.path.join(expected_path, fname)

0 commit comments

Comments
 (0)