Only test if test samples found

Author: luis-camero

clearpath_generator_common/test/generate_samples.py

@@ -138,16 +138,26 @@ def error_log(name: str, sample: str, error: Exception) -> str:
     return f'{name} failed for sample "{sample}" with error: \n{error}'
 
 
-def generate_test_samples(root_dir: str):
-    """Generate all files from common generator."""
-    # Iterate through all samples in clearpath_config
+def get_test_samples():
+    """Return all test samples from Clearpath Config package."""
+    samples = []
     share_dir = get_package_share_directory('clearpath_config')
     sample_dir = os.path.join(share_dir, 'sample')
-    sample_errors = []
     for sample in os.listdir(sample_dir):
         # Filter for Test Samples
         if 'test' not in sample:
             continue
+        samples.append(samples)
+    return samples
+
+
+def generate_test_samples(root_dir: str):
+    """Generate all files from common generator."""
+    # Iterate through all samples in clearpath_config
+    share_dir = get_package_share_directory('clearpath_config')
+    sample_dir = os.path.join(share_dir, 'sample')
+    sample_errors = []
+    for sample in get_test_samples():
         print(f'Generating {sample}'.center(100, '-'))
         # Create Clearpath Directory
         src = os.path.join(sample_dir, sample)

clearpath_generator_common/test/test_samples.py

@@ -8,7 +8,7 @@
 
 from ament_index_python.packages import get_package_share_directory
 
-from .generate_samples import generate_test_samples
+from .generate_samples import generate_test_samples, get_test_samples
 
 
 class MissingSampleException(Exception):
@@ -55,6 +55,7 @@ class TestSamples:
                     os.path.join(installed_sample_dir, sample, 'robot.yaml'))))
         break
     installed_sample_dir = real_installed_sample_dir
+    found_samples = len(get_test_samples()) > 0
 
     def filter_lines(self, lines: List[str], filepath: str) -> str:
         """Filter line files to prevent comparing lines that are expected to be different."""
@@ -137,6 +138,8 @@ def test_generate_samples(self):
 
     def test_number_of_samples_match(self):
         """Validate number of samples matches."""
+        if not self.found_samples:
+            return
         errors = self.diff_dir_trees(
             self.new_sample_dir,
             self.installed_sample_dir,
@@ -151,6 +154,8 @@ def test_number_of_samples_match(self):
 
     def test_samples_match(self):
         """Validate contents of generated sample directory match."""
+        if not self.found_samples:
+            return
         dirs_cmp = filecmp.dircmp(
             self.new_sample_dir,
             self.installed_sample_dir