feat: libvirt mapper recoded using metadata

powerapi-ng · Nov 16, 2021 · eac86ea · eac86ea
1 parent 0e68edf
commit eac86ea
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/powerapi/report_modifier/libvirt_mapper.py b/powerapi/report_modifier/libvirt_mapper.py
@@ -55,7 +55,7 @@ def modify_report(self, report):
             domain_name = result.groups(0)[0]
             try:
                 domain = self.libvirt.lookupByName(domain_name)
-                report.target = domain.UUIDString()
+                report.metadata["domain_id"] = domain.UUIDString()
             except libvirtError:
                 pass
         return report
diff --git a/tests/unit/report_modifier/test_libvirt_mapper.py b/tests/unit/report_modifier/test_libvirt_mapper.py
@@ -54,15 +54,16 @@ def test_modify_report_that_not_match_regexp_musnt_modify_report(libvirt_mapper)
     report = Report(0, 'sensor', BAD_TARGET)
     new_report = libvirt_mapper.modify_report(report)
     assert new_report.target == BAD_TARGET
+    assert new_report.metadata == {}
 
 
 def test_modify_report_that_match_regexp_must_modify_report(libvirt_mapper):
     report = Report(0, 'sensor', LIBVIRT_TARGET_NAME1)
     new_report = libvirt_mapper.modify_report(report)
-    assert new_report.target == UUID_1
+    assert new_report.metadata["domain_id"] == UUID_1
 
 
 def test_modify_report_that_match_regexp_but_with_wrong_domain_name_musnt_modify_report(libvirt_mapper):
     report = Report(0, 'sensor', LIBVIRT_TARGET_NAME2)
     new_report = libvirt_mapper.modify_report(report)
-    assert new_report.target == LIBVIRT_TARGET_NAME2
+    assert new_report.metadata["domain_id"] == LIBVIRT_TARGET_NAME2