Merge pull request #13 from stackhpc/fix/empty-values

Return empty dict when values are empty

Author: Matt Pryor

pyhelm3/client.py

@@ -98,7 +98,7 @@ async def pull_chart(
         devel: bool = False,
         repo: t.Optional[str] = None,
         version: t.Optional[str] = None
-    ) -> contextlib.AbstractAsyncContextManager[pathlib.Path]:
+    ) -> t.AsyncIterator[pathlib.Path]:
         """
         Context manager that pulls the specified chart and yields a chart object
         whose ref is the unpacked chart directory.
@@ -323,7 +323,7 @@ async def should_install_or_upgrade_release(
             if revision_chart.version != chart.metadata.version:
                 return True
             # If the values have changed from the deployed release, we should redeploy
-            revision_values = (await current_revision.values()) or {}
+            revision_values = await current_revision.values()
             if revision_values != values:
                 return True
             # If the chart and values are the same, there is nothing to do

pyhelm3/command.py

@@ -474,7 +474,7 @@ async def get_values(
             command.extend(["--revision", revision])
         if namespace:
             command.extend(["--namespace", namespace])
-        return json.loads(await self.run(command))
+        return json.loads(await self.run(command)) or {}
 
     async def history(
         self,