Merge pull request #114 from mathworks/85-list-selector-changing-items-or-itemsdata-sets-value-arbitrarily

85 list selector changing items or itemsdata sets value arbitrarily

Author: rjackey

test/+wt/+test/ListSelector.m

@@ -1,7 +1,7 @@
 classdef ListSelector < wt.test.BaseWidgetTest
     % Implements a unit test for a widget or component
 
-%   Copyright 2020-2025 The MathWorks Inc.
+    %   Copyright 2020-2025 The MathWorks Inc.
 
     %% Properties
     properties
@@ -19,10 +19,10 @@ function createFigure(testCase)
             testCase.createFigure@wt.test.BaseWidgetTest();
 
             % Adjust grid size
-            testCase.Figure.Position(3:4) = [500 700];
-            testCase.Grid.RowHeight = repmat({175},1,3);
+            testCase.Figure.Position(3:4) = [800 700];
+            testCase.Grid.RowHeight = repmat({175},1,4);
             testCase.Grid.ColumnWidth = {'1x','1x','1x'};
-
+            
         end %function
 
     end %methods
@@ -47,7 +47,7 @@ function setup(testCase)
 
     %% Unit Test
     methods (Test)
-            
+        
         function testProgrammaticItemsValueSelection(testCase)
 
             % Get the listbox
@@ -67,7 +67,7 @@ function testProgrammaticItemsValueSelection(testCase)
             testCase.verifyEqual(testCase.Widget.SelectedIndex, newSelIdx);
 
         end %function
-            
+        
 
         function testProgrammaticItemsDataValueSelection(testCase)
 
@@ -92,7 +92,7 @@ function testProgrammaticItemsDataValueSelection(testCase)
 
         end %function
 
-            
+        
 
         function testHighlightedValue(testCase)
 
@@ -116,7 +116,49 @@ function testHighlightedValue(testCase)
         end %function
 
 
+        
+        function testHighlightedValueOnListRemoval(testCase)
+            
+            % Get the listbox
+            listControl = testCase.Widget.ListBox;
+            
+            % Add items to the list
+            newSelIdx = [1 2 4];
+            newValue = testCase.ItemNames(newSelIdx);
+            testCase.verifySetProperty("Value", newValue);
+            testCase.verifyEqual(testCase.Widget.SelectedIndex, newSelIdx);
+            
+            % Highlight a value in the list
+            newHiliteIdx = [1 2 4];
+            newHilite = testCase.ItemNames(newHiliteIdx);
+            testCase.verifySetProperty("HighlightedValue", newHilite);
+            
+            % Remove items from list
+            testCase.Widget.Items(5) = [];
 
+            % List selection value should match
+            expValue = testCase.ItemNames([1 2 4]);
+            testCase.verifyEqual(string(listControl.Items), expValue);
+            
+            % List selection highlight should match
+            expHighlight = testCase.ItemNames([1 2 4]);
+            testCase.verifyEqual(testCase.Widget.HighlightedValue, expHighlight);
+            
+            % Remove items from list
+            testCase.Widget.Items(2) = [];
+            
+            % List selection value should match
+            expValue = testCase.ItemNames([1 4]);
+            testCase.verifyEqual(string(listControl.Items), expValue);
+            
+            % List selection highlight should match
+            expHighlight = testCase.ItemNames([1 4]);
+            testCase.verifyEqual(testCase.Widget.HighlightedValue, expHighlight);
+            
+        end %function
+        
+        
+        
         function testInteractiveSelection(testCase)
 
             % Get the listbox
@@ -146,7 +188,7 @@ function testInteractiveSelection(testCase)
         end %function
 
 
-            
+        
         function testButtonEnables(testCase)
 
             % Get the listbox and button grid
@@ -205,7 +247,7 @@ function testButtonEnables(testCase)
         end %function
 
 
-            
+        
         function testButtonFunctions(testCase)
 
             % Get the listbox and button grid
@@ -265,22 +307,22 @@ function testButtonFunctions(testCase)
 
             % Verify new highlight
             testCase.verifyEqual(w.HighlightedValue, testCase.ItemNames(1));
-           
+            
             % Verify button enables
             testCase.verifyEquality(buttonGrid.ButtonEnable(2:4), [1 0 1]);
 
         end %function
 
 
-            
+        
         function testUserButtons(testCase)
 
             % Get the widget
             w = testCase.Widget;
 
             % Add User buttons
             w.UserButtons.Icon = ["plot_24.png","play_24.png"];
-            drawnow            
+            drawnow
 
             % Press the buttons
             b = w.UserButtons.Button;
@@ -291,7 +333,7 @@ function testUserButtons(testCase)
         end %function
 
 
-            
+        
         function testStyleProperties(testCase)
 
             % Set ButtonWidth