ci/increase-coverage into main

.github/workflows/matlab-tests.yml

@@ -26,10 +26,11 @@ jobs:
         uses: matlab-actions/setup-matlab@v2
 
       - name: Run tests
-        uses: matlab-actions/run-tests@v2
+        uses: matlab-actions/run-command@v2
         with:
-            source-folder: toolbox
-            code-coverage-cobertura: code-coverage/coverage.xml
+          command: |
+            openProject("uihtml-debugger.prj");
+            runTests;
 
       - name: Upload coverage to Codecov
         uses: codecov/codecov-action@v4

README.md

@@ -7,7 +7,14 @@
 The UIHTML Debugger is a non-intrusive toolkit designed to bridge the gap between MATLAB and JavaScript development. It provides two essential tools to streamline web-based UI development within MATLAB:
 
 1.  **Console Error Rerouter**: Forwards console.log, warn, and error messages directly to the MATLAB Command Window.
-2.  **UIHTML DevTools**: Injects Eruda—a full-featured mobile-style console—directly into your UI for deep inspection without leaving the MATLAB environment.
+1.  **UIHTML DevTools**: Injects Eruda—a full-featured mobile-style console—directly into your UI for deep inspection without leaving the MATLAB environment.
+
+---
+
+## Requirements
+
+*   **MATLAB R2023a** or later.
+*   **Base MATLAB** (no additional toolboxes required).
 
 ---
 

resources/project/0qs9rsEL3yBSqUmUWZjEFIhMQPo/2rAGxcYfPv6icDMAq1Qlbz9L_I0d.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info Name="Toolbox Image File" ReadOnly="WRITABLE"/>

resources/project/0qs9rsEL3yBSqUmUWZjEFIhMQPo/2rAGxcYfPv6icDMAq1Qlbz9L_I0p.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info location="Deployment_ToolboxImageFile_e879122a-6b93-4d3b-9a10-49a600882ccb" type="Label"/>

resources/project/0qs9rsEL3yBSqUmUWZjEFIhMQPo/SLdGz8tUgmpJ67GHyRyb0tvRNuUd.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info Name="App Gallery File" ReadOnly="WRITABLE"/>

resources/project/0qs9rsEL3yBSqUmUWZjEFIhMQPo/SLdGz8tUgmpJ67GHyRyb0tvRNuUp.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info location="Deployment_ToolboxAppGalleryFiles_e879122a-6b93-4d3b-9a10-49a600882ccb" type="Label"/>

resources/project/0qs9rsEL3yBSqUmUWZjEFIhMQPo/gY9EjcEXQx_03t0CxCXVb3wLivEd.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info Name="Getting Started Guide" ReadOnly="WRITABLE"/>

resources/project/0qs9rsEL3yBSqUmUWZjEFIhMQPo/gY9EjcEXQx_03t0CxCXVb3wLivEp.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info location="Deployment_GettingStartedGuideFile_e879122a-6b93-4d3b-9a10-49a600882ccb" type="Label"/>

resources/project/fjRQtWiSIy7hIlj-Kmk87M7s21k/0qs9rsEL3yBSqUmUWZjEFIhMQPod.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info DataType="None" Name="Toolbox - Toolbox1 (1)" ReadOnly="0" SingleValued="0"/>

resources/project/fjRQtWiSIy7hIlj-Kmk87M7s21k/0qs9rsEL3yBSqUmUWZjEFIhMQPop.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info location="Deployment_Category_e879122a-6b93-4d3b-9a10-49a600882ccb" type="Category"/>

resources/project/fjRQtWiSIy7hIlj-Kmk87M7s21k/lJpff9HwDURjOood0D_4QIQyyZod.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info DataType="None" Name="Toolbox - Toolbox1 (2)" ReadOnly="0" SingleValued="0"/>

resources/project/fjRQtWiSIy7hIlj-Kmk87M7s21k/lJpff9HwDURjOood0D_4QIQyyZop.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info location="Deployment_Category_9f79ed46-75ab-4c44-81b9-9b322b0cd885" type="Category"/>

resources/project/lJpff9HwDURjOood0D_4QIQyyZo/M2j0tNMQnrX0MAixgunqSH6O1cUd.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info Name="Getting Started Guide" ReadOnly="WRITABLE"/>

resources/project/lJpff9HwDURjOood0D_4QIQyyZo/M2j0tNMQnrX0MAixgunqSH6O1cUp.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info location="Deployment_GettingStartedGuideFile_9f79ed46-75ab-4c44-81b9-9b322b0cd885" type="Label"/>

resources/project/lJpff9HwDURjOood0D_4QIQyyZo/O-y4x_Fk_elHFgVfW5CS2Gg4bVEd.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info Name="App Gallery File" ReadOnly="WRITABLE"/>

resources/project/lJpff9HwDURjOood0D_4QIQyyZo/O-y4x_Fk_elHFgVfW5CS2Gg4bVEp.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info location="Deployment_ToolboxAppGalleryFiles_9f79ed46-75ab-4c44-81b9-9b322b0cd885" type="Label"/>

resources/project/lJpff9HwDURjOood0D_4QIQyyZo/PBlyKlej6TlbKRuFfxG9YYP9NA8d.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info Name="Toolbox Image File" ReadOnly="WRITABLE"/>

resources/project/lJpff9HwDURjOood0D_4QIQyyZo/PBlyKlej6TlbKRuFfxG9YYP9NA8p.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info location="Deployment_ToolboxImageFile_9f79ed46-75ab-4c44-81b9-9b322b0cd885" type="Label"/>

resources/project/qaw0eS1zuuY1ar9TdPn1GMfrjbQ/TCsOjEr3rMe6CtDnuQLgASe-kAEd.xml

@@ -0,0 +1,6 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info>
+    <Category UUID="FileClassCategory">
+        <Label UUID="design"/>
+    </Category>
+</Info>

resources/project/qaw0eS1zuuY1ar9TdPn1GMfrjbQ/TCsOjEr3rMe6CtDnuQLgASe-kAEp.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info location="runTests.m" type="File"/>

resources/project/root/amwE3LmoG--0pRWluRol6WgE4ZYd.xml

@@ -1,2 +1,2 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<Info configMapData="{&quot;uihtml-dubugger&quot;:&quot;MathWorksDeploymentConfig{3b515abd-47e6-464e-adf2-ddaf8b81973e}&quot;}"/>
+<Info configMapData="{&quot;uihtml-dubugger&quot;:&quot;MathWorksDeploymentConfig{3b515abd-47e6-464e-adf2-ddaf8b81973e}&quot;,&quot;uihtml_debugger&quot;:&quot;MathWorksDeploymentConfig{21b24f2e-5ece-47b0-a968-d291eb125bc4}&quot;}"/>

resources/project/root/xRfC_Iq2vKr7PkzRJo-T0mI2Drkd.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info ToolboxConfigModel="{&quot;entries&quot;:[{&quot;content&quot;:{&quot;appGalleryLabel&quot;:{&quot;content&quot;:{&quot;categoryID&quot;:&quot;Deployment_Category_e879122a-6b93-4d3b-9a10-49a600882ccb&quot;,&quot;labelID&quot;:&quot;Deployment_ToolboxAppGalleryFiles_e879122a-6b93-4d3b-9a10-49a600882ccb&quot;},&quot;type&quot;:&quot;deployment.toolbox.model.ProjectLabel&quot;,&quot;uuid&quot;:&quot;253e8a7c-20f9-4ebf-add4-32713f0e617b&quot;},&quot;authorName&quot;:&quot;Austin Decker&quot;,&quot;description&quot;:&quot;The UIHTML Debugger is a non-intrusive toolkit designed to bridge the gap between MATLAB and JavaScript development. It provides two essential tools to streamline web-based UI development within MATLAB.&quot;,&quot;gettingStartedGuideLabel&quot;:{&quot;content&quot;:{&quot;categoryID&quot;:&quot;Deployment_Category_e879122a-6b93-4d3b-9a10-49a600882ccb&quot;,&quot;labelID&quot;:&quot;Deployment_GettingStartedGuideFile_e879122a-6b93-4d3b-9a10-49a600882ccb&quot;},&quot;type&quot;:&quot;deployment.toolbox.model.ProjectLabel&quot;,&quot;uuid&quot;:&quot;a2bd07ad-eeaa-4e6a-949a-c75564bf3dd4&quot;},&quot;maxRelease&quot;:&quot;R2025b&quot;,&quot;minRelease&quot;:&quot;R2014b&quot;,&quot;outputFileName&quot;:&quot;uihtml_debugger.mltbx&quot;,&quot;outputFolder&quot;:&quot;release&quot;,&quot;summary&quot;:&quot;MATLAB UIHTML Debugger&quot;,&quot;supportedPlatforms&quot;:[&quot;Windows&quot;,&quot;Linux&quot;,&quot;Mac&quot;,&quot;MatlabOnline&quot;],&quot;toolboxFolder&quot;:&quot;toolbox&quot;,&quot;toolboxImgLabel&quot;:{&quot;content&quot;:{&quot;categoryID&quot;:&quot;Deployment_Category_e879122a-6b93-4d3b-9a10-49a600882ccb&quot;,&quot;labelID&quot;:&quot;Deployment_ToolboxImageFile_e879122a-6b93-4d3b-9a10-49a600882ccb&quot;},&quot;type&quot;:&quot;deployment.toolbox.model.ProjectLabel&quot;,&quot;uuid&quot;:&quot;95c9122e-4ee1-4cf1-be4e-b586d3f56cea&quot;},&quot;toolboxName&quot;:&quot;uihtml_debugger&quot;,&quot;toolboxVersion&quot;:&quot;1.1.3&quot;,&quot;uniqueId&quot;:&quot;35da3f7c-30f8-467f-81c6-e62ba3206be6&quot;},&quot;type&quot;:&quot;deployment.toolbox.model.ToolboxConfiguration&quot;,&quot;uuid&quot;:&quot;0d280020-8b56-44d2-aa8f-04ed650919e6&quot;},{&quot;content&quot;:{},&quot;type&quot;:&quot;deployment.toolbox.model.ProjectStateValidationIssueList&quot;,&quot;uuid&quot;:&quot;46190ffa-3b95-44dd-a9ad-fbc0a96b5b05&quot;}],&quot;packageUris&quot;:[&quot;http://schema.mathworks.com/mf0/deployment_toolbox_model/1.0.0&quot;],&quot;version&quot;:&quot;1.0&quot;}" displayName="uihtml_debugger" lastAccessed="1773989182439" type="Toolbox" uniqueId="e879122a-6b93-4d3b-9a10-49a600882ccb"/>

resources/project/root/xRfC_Iq2vKr7PkzRJo-T0mI2Drkp.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info location="MathWorksDeploymentConfig{21b24f2e-5ece-47b0-a968-d291eb125bc4}" type="Files"/>

resources/project/xRfC_Iq2vKr7PkzRJo-T0mI2Drk/yBqJXaEP8Uc9sR9kNtkek_ebn80d.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info/>

resources/project/xRfC_Iq2vKr7PkzRJo-T0mI2Drk/yBqJXaEP8Uc9sR9kNtkek_ebn80p.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Info location="1" type="DIR_SIGNIFIER"/>

runTests.m

@@ -0,0 +1,29 @@
+import matlab.unittest.TestRunner;
+import matlab.unittest.TestSuite;
+import matlab.unittest.plugins.CodeCoveragePlugin;
+import matlab.unittest.plugins.codecoverage.CoberturaFormat;
+
+% Create suite
+suite = TestSuite.fromFolder('tests', 'IncludingSubfolders', true);
+
+% Create runner
+runner = TestRunner.withTextOutput;
+
+% Configure coverage
+sourceFiles = { ...
+    fullfile(pwd, 'toolbox', 'ConsoleErrorRerouter.m'), ...
+    fullfile(pwd, 'toolbox', 'UIHTMLDevTools.m') ...
+};
+
+mkdir('code-coverage');
+coverageFile = fullfile(pwd, 'code-coverage', 'coverage.xml');
+plugin = CodeCoveragePlugin.forFile(sourceFiles, ...
+    'Producing', CoberturaFormat(coverageFile));
+runner.addPlugin(plugin);
+
+% Run tests
+results = runner.run(suite);
+
+% Output results for CI
+display(results);
+assertSuccess(results);

tests/MockComponent.m

@@ -1,4 +1,4 @@
-classdef MockComponent < handle
+classdef MockComponent < handle & dynamicprops
     properties
         HTMLSource
     end

tests/tConsoleErrorRerouter.m

@@ -60,6 +60,13 @@ function testConstructorInvalidComponent(testCase)
                 "ConsoleErrorRerouter:InvalidComponent");
         end % function testConstructorInvalidComponent
 
+        function testConstructorNoSource(testCase)
+            testCase.Component.HTMLSource = "";
+            rerouter = ConsoleErrorRerouter(testCase.Component);
+            testCase.addTeardown(@() delete(rerouter));
+            testCase.verifyEqual(string(testCase.Component.HTMLSource), "");
+        end % function testConstructorNoSource
+
         function testUrlSourceMock(testCase)
             mockComp = MockComponent();
             mockComp.HTMLSource = "http://example.com";
@@ -216,8 +223,115 @@ function testPayloadEdgeCases(testCase)
                 'HTMLEventData', struct('level', 'error', 'message', 'No stack'));
             rerouter.onHTMLEventReceived([], goodData);
             testCase.verifySubstring(rerouter.LastMessage, "No stack");
+
+            % Object payload (instead of struct)
+            mockPayload = struct('level', 'error', 'message', 'Object message');
+            objData = struct('HTMLEventName', "ConsoleError", ...
+                'HTMLEventData', mockPayload);
+            rerouter.onHTMLEventReceived([], objData);
+            testCase.verifySubstring(rerouter.LastMessage, "Object message");
+
+            % Missing level
+            badData = struct('HTMLEventName', "ConsoleError", ...
+                'HTMLEventData', struct('message', 'No level'));
+            % We can't set LastMessage, but we know it should NOT be "No level"
+            % It currently is "Object message" from previous step
+            rerouter.onHTMLEventReceived([], badData);
+            testCase.verifyEqual(rerouter.LastMessage, "Object message");
+
+            % Other event name
+            otherData = struct('HTMLEventName', "OtherEvent", ...
+                'HTMLEventData', struct('level', 'error', 'message', 'Ignored'));
+            rerouter.onHTMLEventReceived([], otherData);
+            testCase.verifyEqual(rerouter.LastMessage, "Object message");
         end % function testPayloadEdgeCases
+
+        function testListenerSetupFailure(testCase)
+            tempDir = tempname;
+            mkdir(tempDir);
+            testCase.addTeardown(@() rmdir(tempDir, 's'));
+            htmlFile = fullfile(tempDir, "test.html");
+            writelines("<html></html>", htmlFile);
+
+            % Create a struct that doesn't support listeners but has valid HTMLSource
+            mock = struct('HTMLSource', htmlFile);
+            rerouter = ConsoleErrorRerouter(mock);
+            testCase.addTeardown(@() delete(rerouter));
+            testCase.verifyClass(rerouter, "ConsoleErrorRerouter");
+        end % function testListenerSetupFailure
+
+        function testDeleteInvalidComponent(testCase)
+            rerouter = ConsoleErrorRerouter(testCase.Component);
+            testCase.Rerouter = rerouter;
+            delete(testCase.Figure); % Deletes component too
+            delete(rerouter); % Should not error in removeShim
+            testCase.Rerouter = [];
+        end % function testDeleteInvalidComponent
+
+        function testOriginalSourceDeleted(testCase)
+            tempDir = tempname;
+            mkdir(tempDir);
+            testCase.addTeardown(@() rmdir(tempDir, 's'));
+            htmlFile = fullfile(tempDir, "temp.html");
+            writelines("<html><body></body></html>", htmlFile);
+
+            mockComp = MockComponent();
+            mockComp.HTMLSource = htmlFile;
+            rerouter = ConsoleErrorRerouter(mockComp);
+
+            % Delete original source before rerouter is deleted
+            delete(htmlFile);
+
+            % removeShim should handle this gracefully
+            delete(rerouter);
+            testCase.verifyEqual(string(mockComp.HTMLSource), string(htmlFile));
+        end % function testOriginalSourceDeleted
 
+        function testCleanupWarnings(testCase)
+            if isunix
+                tempDir = tempname;
+                mkdir(tempDir);
+                testCase.addTeardown(@() rmdir(tempDir, 's'));
+                htmlFile = fullfile(tempDir, "test.html");
+                writelines("<html><body></body></html>", htmlFile);
+
+                mockComp = MockComponent();
+                mockComp.HTMLSource = htmlFile;
+                rerouter = ConsoleErrorRerouter(mockComp);
+
+                % On Linux, to make delete fail, we make the directory read-only
+                tempFile = string(mockComp.HTMLSource);
+                fileattrib(tempDir, '-w');
+                testCase.addTeardown(@() fileattrib(tempDir, '+w'));
+
+                testCase.verifyWarning(@() delete(rerouter), "ConsoleErrorRerouter:FailedCleanup");
+            end
+        end % function testCleanupWarnings
+
+        function testTempWriteFailure(testCase)
+            if isunix
+                % On Unix, we can easily make a directory read-only
+                tempDir = tempname;
+                mkdir(tempDir);
+                testCase.addTeardown(@() rmdir(tempDir, 's'));
+                htmlFile = fullfile(tempDir, "test.html");
+                writelines("<html></html>", htmlFile);
+
+                % Make dir read-only
+                fileattrib(tempDir, '-w');
+                testCase.addTeardown(@() fileattrib(tempDir, '+w'));
+
+                mockComp = MockComponent();
+                mockComp.HTMLSource = htmlFile;
+                testCase.verifyError(@() ConsoleErrorRerouter(mockComp), ...
+                    "ConsoleErrorRerouter:TempWriteFailure");
+            else
+                % On Windows, it's harder to reliably trigger this via attributes,
+                % so we might skip or use a different trick if needed.
+                % For now, we'll assume unix coverage in CI is enough or skip.
+            end
+        end % function testTempWriteFailure
+
         function testEmptyTargetDir(testCase)
             origDir = pwd;
             tempDir = tempname;