Add tests for processing step documentation properties

resources/ReleaseNotes.md

@@ -9,3 +9,4 @@
 - Make sure converted values in `mag.process.SignedInteger` are returned as `double`
 - Fix [#10](https://github.com/ImperialCollegeLondon/MAG-Data-Visualization-Toolbox/issues/10)
 - Fix [#19](https://github.com/ImperialCollegeLondon/MAG-Data-Visualization-Toolbox/issues/19)
+- Add tests for processing step documentation properties

src/analyze/+mag/+process/Range.m

@@ -34,7 +34,7 @@
             value = "Apply Range-Based Scaling";
         end
 
-        function value = get.Description(~)
+        function value = get.Description(this)
             value = "Apply scale factor to " + join(compose("""%s""", this.Variables), ", ") + " based on range """ + this.RangeVariable + """.";
         end
 

src/analyze/+mag/+process/SignedInteger.m

@@ -28,7 +28,7 @@
             value = "Change Signedness";
         end
 
-        function value = get.Description(~)
+        function value = get.Description(this)
             value = "Convert variables " + join(compose("""%s""", this.Variables), ", ") + " from unsigned to signed int16.";
         end
 

tests/unit/analyze/tStep.m

@@ -0,0 +1,39 @@
+classdef tStep < MAGAnalysisTestCase
+% TSTEP Unit tests for "mag.process.Step" classes.
+
+    properties (TestParameter)
+        StepClass = tStep.retrieveProcessingSteps()
+        PropertyName = {"Name", "Description", "DetailedDescription"}
+    end
+
+    methods (Test)
+
+        % Test that calling documentation properties on processing steps
+        % does not throw.
+        function documentationProperty(testCase, StepClass, PropertyName)
+
+            % Set up.
+            stepClass = feval(StepClass);
+
+            % Exercise.
+            documentation = stepClass.(PropertyName);
+
+            % Verify.
+            testCase.assertClass(documentation, "string", "Documentation property type should be ""string"".")
+            testCase.verifyNotEmpty(documentation, "Documentation property value should not be empty.");
+        end
+    end
+
+    methods (Static, Access = private)
+
+        function classNames = retrieveProcessingSteps()
+
+            metaPackage = meta.package.fromName("mag.process");
+
+            metaClass = metaPackage.ClassList;
+            metaClass([metaClass.Abstract]) = [];
+
+            classNames = {metaClass.Name};
+        end
+    end
+end