fix: Ansible sanity test fixes

Signed-off-by: Mahendra Paipuri <[email protected]>
mahendrapaipuri · Feb 24, 2024 · 4c0f33c · 4c0f33c
1 parent 0471da0
commit 4c0f33c
Show file tree

Hide file tree

Showing 6 changed files with 5 additions and 5 deletions.
diff --git a/roles/ceems_api_server/molecule/alternative/prepare.yml b/roles/ceems_api_server/molecule/alternative/prepare.yml
@@ -29,7 +29,7 @@
   tasks:
     - name: Create sacct executable file
       ansible.builtin.copy:
-        src: "../sacct"
+        src: "../default/files/sacct"
         dest: /usr/bin/sacct
         owner: root
         group: root

diff --git a/roles/ceems_api_server/molecule/sacct → ...s_api_server/molecule/default/files/sacct b/roles/ceems_api_server/molecule/sacct → ...s_api_server/molecule/default/files/sacct
diff --git a/roles/ceems_api_server/molecule/default/prepare.yml b/roles/ceems_api_server/molecule/default/prepare.yml
@@ -5,7 +5,7 @@
   tasks:
     - name: Create sacct executable file
       ansible.builtin.copy:
-        src: "../sacct"
+        src: "files/sacct"
         dest: /usr/bin/sacct
         owner: root
         group: root

diff --git a/roles/ceems_api_server/molecule/latest/prepare.yml b/roles/ceems_api_server/molecule/latest/prepare.yml
@@ -5,7 +5,7 @@
   tasks:
     - name: Create sacct executable file
       ansible.builtin.copy:
-        src: "../sacct"
+        src: "../default/files/sacct"
         dest: /usr/bin/sacct
         owner: root
         group: root

diff --git a/roles/ceems_exporter/molecule/latest/tests/test_latest.py b/roles/ceems_exporter/molecule/latest/tests/test_latest.py
@@ -26,6 +26,7 @@ def test_directories(host):
         assert d.is_directory
         assert d.exists
 
+
 def test_capabilities_property(host):
     s = host.service("ceems_exporter")
     p = s.systemd_properties
@@ -54,4 +55,4 @@ def test_collectors(host):
     exporter_out = host.check_output('curl http://localhost:9010/metrics').strip()
     assert "ceems_scrape_collector_success{collector=\"ipmi_dcmi\"}" not in exporter_out
     assert "ceems_scrape_collector_success{collector=\"rapl\"}" not in exporter_out
-    assert "ceems_scrape_collector_success{collector=\"emissions\"}" in exporter_out
+    assert "ceems_scrape_collector_success{collector=\"emissions\"}" in exporter_out
diff --git a/roles/nvidia_dcgm_exporter/molecule/alternative/tests/test_alternative.py b/roles/nvidia_dcgm_exporter/molecule/alternative/tests/test_alternative.py
@@ -3,7 +3,6 @@
 
 import os
 import testinfra.utils.ansible_runner
-import pytest
 
 testinfra_hosts = testinfra.utils.ansible_runner.AnsibleRunner(
     os.environ['MOLECULE_INVENTORY_FILE']).get_hosts('all')