Improve Reducer component resources. Add more tests. Add custom rules for the Code Analyzer. (#43)

Author: isaacito12

BEV/BEV_system_model.mdl

@@ -1,6 +1,6 @@
 function plan = buildfile
-%% Define tasks for buildtool to check code and run tests
-% In the Editor, use Run Build button to start task.
+% Define tasks for the buildtool to check code and run tests.
+% In the Editor, use the "Run Build" button to start a task.
 
 % Overview of MATLAB Build Tool
 % https://www.mathworks.com/help/matlab/matlab_prog/overview-of-matlab-build-tool.html

BEV/Model-Basic/SimulationCases/buildfile.m

@@ -1,6 +1,6 @@
 function plan = buildfile
-%% Define tasks for buildtool to check code and run tests
-% In the Editor, use Run Build button to start task.
+% Define tasks for the buildtool to check code and run tests.
+% In the Editor, use the "Run Build" button to start a task.
 
 % Overview of MATLAB Build Tool
 % https://www.mathworks.com/help/matlab/matlab_prog/overview-of-matlab-build-tool.html

BEV/Model-Thermal/SimulationCases/buildfile.m

@@ -1,6 +1,6 @@
 function plan = buildfile
-%% Define tasks for buildtool to check code and run tests
-% In the Editor, use Run Build button to start task.
+% Define tasks for the buildtool to check code and run tests.
+% In the Editor, use the "Run Build" button to start a task.
 
 % Overview of MATLAB Build Tool
 % https://www.mathworks.com/help/matlab/matlab_prog/overview-of-matlab-build-tool.html

BEV/buildfile.m

@@ -51,153 +51,6 @@ function PassingTest_4(~)
       evalin("base", "BEV_main_script");
     end  % function
 
-    %% Link tests
-
-    function LinkTest_1(testcase)
-      % Check all the Callback Button blocks in a model.
-      % Different tests are done depending on the name of the block.
-      % - "Plot ..." makes a visualization/plot using some block parameters.
-      % - "Open ... app" opens an app.
-      % - "Open ... script" opens the script in the Editor.
-      % - Something else, which is assumed to be in a script call style, not in a function call style.
-      %
-      % Get the ClickFcn property of the Callback Button block and validate it.
-      % ClickFcn should call one main action, optionally with the disp command, comment lines, or empty lines.
-
-      model_name = "BEV_system_model";
-
-      load_system(model_name)
-
-      block_paths = string(getfullname(Simulink.findBlocksOfType(model_name, "CustomCallbackButton")));
-
-      num_blocks = numel(block_paths);
-      for idx = 1 : num_blocks
-        target_block_path = block_paths(idx);
-
-        disp("Found a Custom Callback Button: " + target_block_path)
-
-        [system_path, block_name, ~] = fileparts(target_block_path);
-
-        find_options = Simulink.FindOptions(SearchDepth = 1);
-        all_found_blocks = string(getfullname(Simulink.findBlocks(system_path, find_options)));
-
-        ClickFcn_text = string(get_param(target_block_path, "ClickFcn"));
-
-        lines = splitlines(ClickFcn_text); 
-
-        if height(lines) > 1
-          % Remove comment lines.
-          logical_index = not(startsWith(lines, "%"));
-          lines = lines(logical_index);
-
-          % Remove empty lines.
-          logical_index = not(lines == "");
-          lines = lines(logical_index);
-
-          % Remove lines containing the disp command.
-          % This assumes that the target text is not in the same line as disp.
-          logical_index = not(contains(lines, "disp("));
-          lines = lines(logical_index);
-        end  % if
-
-        if height(lines) > 1
-          % Skip testing if there are 2 or more lines remaining at this point.
-          % !todo: Test the callback code even when it has 2 or more lines.
-
-          continue
-
-        end  % if
-
-        target_line = lines;
-        disp("Target: " + target_line)
-
-        if startsWith(block_name, "Plot")
-          % Plot button.
-          %
-          % The text lines must be in the following style.
-          %   plotFunction(gcs + "/Target block")
-          %
-          % Check the plotFunction and the target block.
-
-          % Use which to get the full path to the plot function.
-          target_function_name = extractBefore(target_line, "(");
-          disp("Plot: " + target_function_name)
-          target_function_fullpath = string(which(target_function_name));  % !test-target
-          verifyTrue(testcase, target_function_fullpath ~= "")
-
-          % Get
-          %   /Target block
-          % from
-          %   (gcs + "/Target block", ...)
-          sp = optionalPattern(whitespacePattern);
-          extracted_block_name = extractBetween(target_line, "("+sp+"gcs"+sp+"+"+sp+"""", """"+optionalPattern(sp+","+wildcardPattern("Except",""""))+sp+")");
-          % Construct a full block path and check it.
-          constructed_block_path = system_path + extracted_block_name;
-          logical_index = constructed_block_path == all_found_blocks;  % !test-target
-          verifyEqual(testcase, nnz(logical_index), 1)
-
-        elseif startsWith(block_name, "Open") && endsWith(block_name, whitespacePattern+("A"|"a")+"pp")
-          % App button.
-          %
-          % The text lines must be in the following style.
-          %   SomeApp
-          % or
-          %   SomeApp(gcs + "/Target block")
-          %
-          % Check the SomeApp and the target block.
-          % - The app name must end with "App".
-
-          verifyTrue(testcase, contains(target_line, "App" + alphanumericBoundary))
-
-          if contains(target_line, "App(")
-            target_app_name = extractBefore(target_line, "(");
-          else
-            target_app_name = target_line;
-          end  % if
-          disp("App: " + target_app_name)
-
-          % Use which to get the full path to the plot function.
-          target_app_fullpath = string(which(target_app_name));  % !test-target
-          verifyTrue(testcase, target_app_fullpath ~= "")
-
-          if contains(target_line, "App(")
-            % Get
-            %   /Target block
-            % from
-            %   (gcs + "/Target block", ...)
-            sp = optionalPattern(whitespacePattern);
-            extracted_block_name = extractBetween(target_line, "("+sp+"gcs"+sp+"+"+sp+"""", """"+optionalPattern(sp+","+wildcardPattern("Except",""""))+sp+")");
-            % Construct a full block path and check it.
-            constructed_block_path = system_path + extracted_block_name;
-            logical_index = constructed_block_path == all_found_blocks;  % !test-target
-            verifyEqual(testcase, nnz(logical_index), 1)
-          end  % if
-
-        elseif startsWith(block_name, "Open") && endsWith(block_name, whitespacePattern+("S"|"s")+"cript")
-          % Open a script in the Editor.
-          %   edit("script_name")
-          %
-          % Check that the script_name exists.
-
-          verifyTrue(testcase, startsWith(target_line, "edit("))
-
-          sp = optionalPattern(whitespacePattern);
-          script_name = extractBetween(target_line, "("+sp+"""", """"+sp+")");
-          disp("Script: " + script_name)
-
-          target_fullpath = string(which(script_name));  % !test-target
-          verifyTrue(testcase, target_fullpath ~= "")
-
-        else
-          disp("Not plot, not app, not edit.")
-          % Assume that the target text is in a script style, not a function style.
-          target_fullpath = string(which(target_line));  % !test-target
-          verifyTrue(testcase, target_fullpath ~= "")
-
-        end  % if
-      end  % for
-    end  % function
-
   end  % methods
 
 end  % classdef