setting highlighted label to undefined when not needed

Author: omarArm

src/views/live-watch/live-watch.test.ts

@@ -141,7 +141,7 @@ describe('LiveWatchTreeDataProvider', () => {
             (tracker as any)._onWillStopSession.fire(gdbtargetDebugSession);
             // Allow async handlers to complete
             await new Promise(resolve => setTimeout(resolve, 0));
-            expect(node.value.highlightedLabel).toEqual({ label: 'myVar = ' });
+            expect(node.value.highlightedLabel).toBeUndefined();
         });
 
         it('reassigns IDs sequentially for restored nodes on construction', () => {
@@ -413,9 +413,7 @@ describe('LiveWatchTreeDataProvider', () => {
                 session: {}
             };
             await (liveWatchTreeDataProvider as any).refresh();
-            expect(node.value.highlightedLabel).toEqual({
-                label: 'myVar = '
-            });
+            expect(node.value.highlightedLabel).toBeUndefined();
         });
     });
 

src/views/live-watch/live-watch.ts

@@ -34,7 +34,7 @@ export interface LiveWatchValue {
     variablesReference: number;
     type?: string;
     evaluateName?: string;
-    highlightedLabel?: vscode.TreeItemLabel;
+    highlightedLabel?: vscode.TreeItemLabel | undefined;
 }
 
 interface SessionLiveWatchState {
@@ -403,7 +403,7 @@ export class LiveWatchTreeDataProvider implements vscode.TreeDataProvider<LiveWa
         if (node.value.result !== result.result) {
             node.value.highlightedLabel = { label: node.expression + ' = ', highlights: [[0, node.expression.length]] };
         } else {
-            node.value.highlightedLabel = { label: node.expression + ' = ' };
+            node.value.highlightedLabel = undefined;
         }
         node.value.result = result.result;
         node.value.variablesReference = result.variablesReference;
@@ -493,7 +493,7 @@ export class LiveWatchTreeDataProvider implements vscode.TreeDataProvider<LiveWa
     private async save() {
         // Clear highlighted labels before saving, as they are only relevant for the current state of the tree and can cause confusion when reloading
         for (const node of this.roots) {
-            node.value.highlightedLabel = { label: node.expression + ' = ' };
+            node.value.highlightedLabel = undefined;
         }
         await this.context.workspaceState.update(this.STORAGE_KEY, this.roots);
     }