From d20c784218b7cb0d562c15f73082dc1adab4b55b Mon Sep 17 00:00:00 2001
From: BI-MarcB <93246302+BI-MarcB@users.noreply.github.com>
Date: Tue, 8 Aug 2023 16:45:59 +0200
Subject: [PATCH] Fix generic tests returning "TypeError: 'NoneType' object is
 not subscriptable" (#1)

Generic tests return an error "TypeError: 'NoneType' object is not subscriptable", because they are listed in the nodes, but not relevant for dbt-coverage.
---
 dbt_coverage/__init__.py | 21 +++++++++++----------
 1 file changed, 11 insertions(+), 10 deletions(-)

diff --git a/dbt_coverage/__init__.py b/dbt_coverage/__init__.py
index 2822c92..6a2b538 100644
--- a/dbt_coverage/__init__.py
+++ b/dbt_coverage/__init__.py
@@ -72,15 +72,15 @@ def from_node(node, manifest: Manifest) -> Table:
 
         if original_file_path is None:
             logging.warning("original_file_path value not found in manifest for %s", unique_id)
-
-        return Table(
-            unique_id,
-            # Take table name from manifest.json instead of catalog.json since in catalog.json the
-            # name is actually an alias in case it is defined.
-            manifest_table["name"].lower(),
-            original_file_path,
-            {col.name: col for col in columns},
-        )
+        else:
+            return Table(
+                unique_id,
+                # Take table name from manifest.json instead of catalog.json since in catalog.json the
+                # name is actually an alias in case it is defined.
+                manifest_table["name"].lower(),
+                original_file_path,
+                {col.name: col for col in columns},
+            )
 
     def get_column(self, column_name):
         return self.columns.get(column_name)
@@ -120,7 +120,8 @@ def filter_tables(self, model_path_filter: List[str]) -> Catalog:
 
     @staticmethod
     def from_nodes(nodes, manifest: Manifest):
-        tables = [Table.from_node(table, manifest) for table in nodes]
+        tables_tmp = [Table.from_node(table, manifest) for table in nodes]
+        tables = [table for table in tables_tmp if table is not None]
         return Catalog({table.unique_id: table for table in tables})
 
     def get_table(self, table_id):