Update alcunit setter

Signed-off-by: pem70 <[email protected]>
zowe · Jun 20, 2024 · 160274e · 160274e
1 parent 0c5eb3c
commit 160274e
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/src/zos_files/zowe/zos_files_for_zowe_sdk/datasets.py b/src/zos_files/zowe/zos_files_for_zowe_sdk/datasets.py
@@ -51,8 +51,8 @@ def __init__(
         self.dirblk = dirblk
         self.avgblk = avgblk
         self.recfm = recfm
-        self.blksize = blksize
         self.lrecl = lrecl
+        self.blksize = blksize
         self.storclass = storclass
         self.mgntclass = mgntclass
         self.dataclass = dataclass
@@ -92,7 +92,7 @@ def alcunit(self) -> Optional[str]:
     def alcunit(self, alcunit: Optional[str]):
         if alcunit is None:
             if self.like is not None:
-                self.alcunit = None
+                self.__alcunit = None
             else:
                 self.__alcunit = "TRK"
         elif alcunit not in ("CYL", "TRK"):

diff --git a/tests/unit/files/datasets/test_create.py b/tests/unit/files/datasets/test_create.py
@@ -218,3 +218,12 @@ def test_create_data_set_without_options(self):
         with self.assertRaises(ValueError) as error:
             Files(self.test_profile).create_data_set("DSNAME123")
         self.assertTrue("dataset options" in str(error.exception))
+
+    def test_DatasetOption_alcunit_setter(self):
+        options = DatasetOption(like="test")
+        self.assertEqual(options.alcunit, None)
+        options = DatasetOption()
+        self.assertEqual(options.alcunit, "TRK")
+        with self.assertRaises(KeyError) as error:
+            options.alcunit = "test"
+        self.assertTrue("'alcunit' must be 'CYL' or 'TRK'" in str(error.exception))