vis: remove empty shares from updates

novem/vis/shared.py

@@ -46,10 +46,12 @@ def set(self, share: Union[str, List[str]]) -> None:
         rms = set(es) - set(shares)
         adds = set(shares) - set(es)
 
-        for r in rms:
+        # Delete non-empty items
+        for r in filter(None, rms):
             self.api.api_delete(f"/shared/{r}")
 
-        for a in adds:
+        # Add non-empty items
+        for a in filter(None, adds):
             self.api.api_create(f"/shared/{a}")
 
     def __iadd__(self, share: str) -> List[str]:

tests/test_share.py

@@ -77,6 +77,18 @@ def get_shares(request, context):
         text=partial(del_share, "+novem_demo~novem_test"),
     )
 
+    requests_mock.register_uri(
+        "delete",
+        f"{api_root}vis/plots/{plot_id}/shared/+novem_demo~novem_demo",
+        text=partial(del_share, "+novem_demo~novem_test"),
+    )
+
+    requests_mock.register_uri(
+        "delete",
+        f"{api_root}vis/plots/{plot_id}/shared/public",
+        text=partial(del_share, "public"),
+    )
+
     # create a novem api object
     n = Plot(plot_id, type=plot_type, config_path=config_file)  # config location
 
@@ -91,3 +103,5 @@ def get_shares(request, context):
 
     n.shared -= "+novem_demo~novem_test"
     # print(n.shared.get())
+
+    n.shared = ""