fix: remove targets as a class variable in interface

kapitan/inventory/inventory.py

@@ -27,7 +27,6 @@ class InventoryTarget:
 
 class Inventory(ABC):
     path: str = "inventory"
-    targets: dict = {}
 
     def __init__(self, path: str = path, compose_target_name: bool = False):
         self.inventory_path = path
@@ -37,6 +36,8 @@ def __init__(self, path: str = path, compose_target_name: bool = False):
         # config
         self.compose_target_name = compose_target_name
 
+        self.targets = {}
+
     @property
     def inventory(self) -> dict:
         if not self.targets:

tests/test_compile.py

@@ -17,10 +17,10 @@
 import yaml
 import toml
 from kapitan.cli import main
+from kapitan.resources import get_inventory
 from kapitan.utils import directory_hash
 from kapitan.cached import reset_cache
 from kapitan.targets import validate_matching_target_name
-from kapitan.inventory import Inventory
 from kapitan.errors import InventoryError
 
 
@@ -136,8 +136,6 @@ def setUp(self):
     def test_compile(self):
         sys.argv = ["kapitan", "compile", "-c"]
         main()
-        # Compile again to verify caching works as expected
-        main()
         os.remove("./compiled/.kapitan_cache")
         compiled_dir_hash = directory_hash(os.getcwd() + "/compiled")
         test_compiled_dir_hash = directory_hash(os.getcwd() + "/../../tests/test_kubernetes_compiled")
@@ -175,7 +173,7 @@ def test_compile_not_matching_targets(self):
     def test_compile_vars_target_missing(self):
         inventory_path = "inventory"
         target_filename = "minikube-es"
-        target_obj = Inventory(inventory_path).inventory["nodes"][target_filename]["parameters"]["kapitan"]
+        target_obj = get_inventory(inventory_path).get_target(target_filename)["kapitan"]
         # delete vars.target
         del target_obj["vars"]["target"]