Add a lint check for empty and missing environment files

docs/releases.rst

@@ -4,6 +4,12 @@
     Releases
 ======================
 
+tmt-1.38.0
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+``tmt lint`` now reports a failure if empty environment files are found.
+
+
 tmt-1.37.0
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 

tests/lint/plan/data/empty_env_file.fmf

@@ -0,0 +1,5 @@
+summary: Empty environment file plan
+environment-file:
+    - empty_env
+execute:
+    script: tmt --help

tests/lint/plan/data/env

@@ -0,0 +1 @@
+test=test

tests/lint/plan/data/good.fmf

@@ -2,3 +2,5 @@ summary: Basic smoke test
 execute:
     script: tmt --help
 id: 9c73e336-983e-4349-9586-b63c20ea2b89
+environment-file:
+    - env

tests/lint/plan/test.sh

@@ -50,6 +50,7 @@ rlJournalStart
 
         rlRun -s "$tmt plan lint invalid_url" 1
         rlAssertGrep "fail P005 remote fmf id in \"default-0\" is invalid, repo 'http://invalid-url' cannot be cloned" $rlRun_LOG
+        rlAssertGrep "skip P008 no environment files found" $rlRun_LOG
 
         rlRun -s "$tmt plan lint invalid_ref" 1
         rlAssertGrep "fail P005 remote fmf id in \"default-0\" is invalid, git ref 'invalid-ref-123456' is invalid" $rlRun_LOG
@@ -71,6 +72,9 @@ rlJournalStart
         rlRun -s "$tmt plan lint invalid-plugin-key" 0
         rlAssertGrep 'warn C000 key "wrong" not recognized by schema$' $rlRun_LOG
         rlAssertGrep 'warn C000 key "wrong" not recognized by schema /schemas/prepare/feature' $rlRun_LOG
+
+        rlRun -s "$tmt plan lint empty_env_file" 1
+        rlAssertGrep "fail P008 the file \"$(pwd)/empty_env\" is empty" $rlRun_LOG
     rlPhaseEnd
 
     rlPhaseStartTest "P007: step phases require existing guests and roles"

tmt/base.py

@@ -2332,6 +2332,23 @@ def _lint_step(step: str) -> LinterReturn:
         yield from _lint_step('execute')
         yield from _lint_step('finish')
 
+    def lint_empty_env_files(self) -> LinterReturn:
+        """ P008: env files are not empty """
+
+        env_files = self.node.get("environment-file") or []
+
+        if not env_files:
+            yield LinterOutcome.SKIP, 'no environment files found'
+            return
+
+        for env_file in env_files:
+            env_file = Path(env_file).resolve()
+            if not env_file.stat().st_size:
+                yield LinterOutcome.FAIL, f'the file "{env_file}" is empty'
+                return
+
+        yield LinterOutcome.PASS, 'no empty environment files'
+
     def wake(self) -> None:
         """ Wake up all steps """