Issue#5: Avoid exception on invalid projects

Author: matsoftware

swift_code_metrics/_analyzer.py

@@ -4,18 +4,27 @@
 from ._parser import SwiftFileParser
 from ._metrics import Framework, Metrics
 from functional import seq
-
+from ._helpers import Log
 
 class Inspector:
-    def __init__(self, directory, artifacts, tests_default_suffixes, exclude_paths=None):
-        if exclude_paths is None:
-            exclude_paths = []
+    def __init__(self, directory, artifacts, tests_default_suffixes, exclude_paths=[]):
+        self.exclude_paths = exclude_paths
+        self.directory = directory
+        self.artifacts = artifacts
+        self.tests_default_suffixes = tests_default_suffixes
         self.frameworks = []
-        if directory is not None:
+        self.report = None
+
+    def analyze(self) -> bool:
+        if self.directory is not None:
             # Initialize report
-            self.__analyze_directory(directory, exclude_paths, tests_default_suffixes)
-            self.report = self._generate_report()
-            self._save_report(artifacts)
+            self.__analyze_directory(self.directory, self.exclude_paths, self.tests_default_suffixes)
+            if len(self.frameworks) > 0:
+                self.report = self._generate_report()
+                self._save_report(self.artifacts)
+                return True
+        return False
+
 
     def filtered_frameworks(self, is_test=False):
         return seq(self.frameworks) \

swift_code_metrics/scm.py

@@ -2,7 +2,7 @@
 
 from argparse import ArgumentParser
 
-from ._helpers import ReportingHelpers
+from ._helpers import Log,ReportingHelpers
 from ._analyzer import Inspector
 from ._presenter import GraphPresenter
 from .version import VERSION
@@ -66,6 +66,10 @@ def main():
     # Inspects the provided directory
     analyzer = Inspector(directory, artifacts, default_tests_paths, exclude)
 
+    if not analyzer.analyze():
+        Log.warn('No valid swift files found in the project')
+        sys.exit(0)
+
     if not should_generate_graphs:
         sys.exit(0)
 

swift_code_metrics/tests/test_integration.py

@@ -28,12 +28,32 @@ def test_sample_app(self):
         generated_json = IntegrationTest.read_json_file(output_file)
         self.assertEqual(generated_json, expected_json)
 
-
     @staticmethod
     def read_json_file(path):
         with open(path, 'r') as fp:
             return json.load(fp)
 
 
+class IntegrationUnhappyTest(unittest.TestCase):
+
+    def setUp(self):
+        self.maxDiff = None
+        sys.argv.clear()
+        sys.argv.append(os.path.dirname(os.path.realpath(__file__)))
+        sys.argv.append("--source")
+        sys.argv.append("any")
+        sys.argv.append("--artifacts")
+        sys.argv.append("any")
+
+    def tearDown(self):
+        sys.argv.clear()
+
+    def test_sample_app(self):
+        with self.assertRaises(SystemExit) as cm:
+            scm.main()  # should not throw exception and return 0
+
+        self.assertEqual(cm.exception.code, 0)
+
+
 if __name__ == '__main__':
     unittest.main()