Skip to content

Commit e3d65a8

Browse files
committed
test: Removes unnecessary Path()s
Changes `Path(snapshot_artifacts_dir)` to `snapshot_artifacts_dir`, since `snapshot_artifacts_dir` is already a `Path` object. Signed-off-by: Takahiro Itazuri <itazur@amazon.com>
1 parent f36927e commit e3d65a8

File tree

1 file changed

+6
-12
lines changed

1 file changed

+6
-12
lines changed

tests/integration_tests/functional/test_cpu_features.py

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -538,12 +538,8 @@ def test_cpu_wrmsr_restore(
538538
dump_msr_state_to_file(msrs_after_fname, vm.ssh, shared_names)
539539

540540
# Compare the two lists of MSR values and assert they are equal
541-
before_df = pd.read_csv(
542-
Path(snapshot_artifacts_dir) / shared_names["msrs_before_fname"]
543-
)
544-
after_df = pd.read_csv(
545-
Path(snapshot_artifacts_dir) / shared_names["msrs_after_fname"]
546-
)
541+
before_df = pd.read_csv(snapshot_artifacts_dir / shared_names["msrs_before_fname"])
542+
after_df = pd.read_csv(snapshot_artifacts_dir / shared_names["msrs_after_fname"])
547543
check_msrs_are_equal(
548544
before_df,
549545
after_df,
@@ -606,9 +602,7 @@ def test_cpu_cpuid_snapshot(
606602
)
607603
clean_and_mkdir(snapshot_artifacts_dir)
608604

609-
cpuid_before_fname = (
610-
Path(snapshot_artifacts_dir) / shared_names["cpuid_before_fname"]
611-
)
605+
cpuid_before_fname = snapshot_artifacts_dir / shared_names["cpuid_before_fname"]
612606

613607
dump_cpuid_to_file(cpuid_before_fname, vm.ssh)
614608

@@ -705,13 +699,13 @@ def test_cpu_cpuid_restore(
705699
)
706700

707701
# Dump CPUID to a file for further comparison
708-
cpuid_after_fname = Path(snapshot_artifacts_dir) / shared_names["cpuid_after_fname"]
702+
cpuid_after_fname = snapshot_artifacts_dir / shared_names["cpuid_after_fname"]
709703
dump_cpuid_to_file(cpuid_after_fname, vm.ssh)
710704

711705
# Compare the two lists of MSR values and assert they are equal
712706
check_cpuid_is_equal(
713-
Path(snapshot_artifacts_dir) / shared_names["cpuid_before_fname"],
714-
Path(snapshot_artifacts_dir) / shared_names["cpuid_after_fname"],
707+
snapshot_artifacts_dir / shared_names["cpuid_before_fname"],
708+
snapshot_artifacts_dir / shared_names["cpuid_after_fname"],
715709
guest_kernel.base_name(), # this is to annotate the assertion output
716710
)
717711

0 commit comments

Comments
 (0)