Do not need to avoid doing import_module multiple times. Add a test for reload method and move tests for Zarr class into a separate file

Author: krisfed

Zarr.m

@@ -34,15 +34,11 @@
 
         function zarrPy = ZarrPy()
             % Get ZarrPy Python module
-            
-            sys = py.importlib.import_module('sys');
-            loadedModules = sys.modules;
-            zarrPy = loadedModules.get("ZarrPy", 0);
 
-            % if the zarrPy module is not loaded
-            if isnumeric(zarrPy)
-                zarrPy = py.importlib.import_module('ZarrPy');
-            end
+            % Python will compile and cache the module after the first call
+            % to import_module, so there is no harm in making this call
+            % multiple times.
+            zarrPy = py.importlib.import_module('ZarrPy');
         end
 
         function pyReloadInProcess()
@@ -51,7 +47,7 @@ function pyReloadInProcess()
             % this call. For Out-of-Process Python, can just use
             % `terminate(pyenv)` instead.
 
-            py.importlib.reload(Zarr.ZarrPy)
+           py.importlib.reload(Zarr.ZarrPy);
         end
 
         function isZarray = isZarrArray(path)

test/tZarr.m

@@ -0,0 +1,40 @@
+classdef tZarr < SharedZarrTestSetup
+    % Tests for Zarr class methods
+
+    % Copyright 2025 The MathWorks, Inc.
+
+    methods(Test)
+
+        function verifySupportedCloudPatterns(testcase)
+            % Verify that the bucket name and the array path can be
+            % extracted successfully if a cloud path is used as an input.
+            
+            % This list contains path pattern currently supported by Zarr
+            % in MATLAB. Any invalid path not matching any of these
+            % patterns will result in an error.
+            inpPath = {'https://mybucket.s3.us-west-2.amazonaws.com/path/to/myZarrFile', ...
+                    'https://mybucket.s3.amazonaws.com/path/to/myZarrFile', ...
+                    'https://mybucket.s3.custom-endpoint.org/path/to/myZarrFile', ...
+                    'https://s3.amazonaws.com/mybucket/path/to/myZarrFile', ...
+                    'https://s3.eu-central-1.example.edu/mybucket/path/to/myZarrFile', ...
+                    's3://mybucket/path/to/myZarrFile'};
+
+            for i = 1:length(inpPath)
+                [bucketName, objectPath] = Zarr.extractS3BucketNameAndPath(inpPath{i});
+                testcase.verifyEqual(bucketName, 'mybucket', ['Bucket name extraction failed for ' inpPath{i}]);
+                testcase.verifyEqual(objectPath, 'path/to/myZarrFile', ['Object path extraction failed for ' inpPath{i}]);
+            end
+        end
+
+        function verifyReload(testcase)
+            % Verify that calling reload method does not cause any issues
+
+            Zarr.pyReloadInProcess();
+            zarrPyModule = Zarr.ZarrPy;
+            testcase.verifyTrue(isa(zarrPyModule, 'py.module'))
+
+        end
+
+  
+    end
+end

test/tZarrCreate.m

@@ -62,27 +62,6 @@ function createArrayRelativePath(testcase)
             testcase.verifyEqual(arrInfo.node_type,'array','Unexpected Zarr array node type');
         end
 
-        function verifySupportedCloudPatterns(testcase)
-            % Verify that the bucket name and the array path can be
-            % extracted successfully if a cloud path is used as an input.
-            
-            % This list contains path pattern currently supported by Zarr
-            % in MATLAB. Any invalid path not matching any of these
-            % patterns will result in an error.
-            inpPath = {'https://mybucket.s3.us-west-2.amazonaws.com/path/to/myZarrFile', ...
-                    'https://mybucket.s3.amazonaws.com/path/to/myZarrFile', ...
-                    'https://mybucket.s3.custom-endpoint.org/path/to/myZarrFile', ...
-                    'https://s3.amazonaws.com/mybucket/path/to/myZarrFile', ...
-                    'https://s3.eu-central-1.example.edu/mybucket/path/to/myZarrFile', ...
-                    's3://mybucket/path/to/myZarrFile'};
-
-            for i = 1:length(inpPath)
-                [bucketName, objectPath] = Zarr.extractS3BucketNameAndPath(inpPath{i});
-                testcase.verifyEqual(bucketName, 'mybucket', ['Bucket name extraction failed for ' inpPath{i}]);
-                testcase.verifyEqual(objectPath, 'path/to/myZarrFile', ['Object path extraction failed for ' inpPath{i}]);
-            end
-        end
-
         function invalidFilePath(testcase)
             % Verify error when an invalid file path is used as an input to
             % zarrcreate function.