adding a couple more tests and changing location of icons

Author: omarArm

images/trace-and-live-dark.svg renamed to media/trace-and-live-dark.svg

@@ -52,7 +52,7 @@
         {
           "id": "cmsis-debugger",
           "title": "Trace and Live View",
-          "icon": "images/trace-and-live-dark.svg"
+          "icon": "media/trace-and-live-dark.svg"
         }
       ]
     },
@@ -61,7 +61,7 @@
         {
           "id": "cmsis-debugger.liveWatch",
           "name": "Live Watch",
-          "icon": "images/live-watch-dark.svg"
+          "icon": "media/trace-and-live-light.svg"
         }
       ]
     },

images/trace-and-live-light.svg renamed to media/trace-and-live-light.svg

@@ -210,6 +210,22 @@ describe('LiveWatchTreeDataProvider', () => {
             await (liveWatchTreeDataProvider as any).refresh(node);
             expect(node.value.result).toBe('new-value');
         });
+
+        it('refresh without argument evaluates each root and fires tree change once', async () => {
+            const nodeA = makeNode('node-A', { result: 'value-A', variablesReference: 0 }, 1);
+            const nodeB = makeNode('node-B', { result: 'value-B', variablesReference: 0 }, 2);
+            (liveWatchTreeDataProvider as any).roots = [nodeA, nodeB];
+            const evalMock = jest.spyOn(liveWatchTreeDataProvider as any, 'evaluate')
+                .mockImplementation(async (expr: unknown) => ({ result: String(expr) + '-updated', variablesReference: 0 }));
+            const fireSpy = jest.spyOn((liveWatchTreeDataProvider as any)._onDidChangeTreeData, 'fire');
+            await (liveWatchTreeDataProvider as any).refresh();
+            expect(evalMock).toHaveBeenCalledTimes(2);
+            expect(nodeA.value.result).toBe('node-A-updated');
+            expect(nodeB.value.result).toBe('node-B-updated');
+            expect(fireSpy).toHaveBeenCalledTimes(1);
+            // fire called with undefined (no specific node) per implementation
+            expect(fireSpy.mock.calls[0][0]).toBeUndefined();
+        });
     });
 
     describe('command registration', () => {
@@ -309,5 +325,35 @@ describe('LiveWatchTreeDataProvider', () => {
             expect(refreshSpy).toHaveBeenCalled();
         });
     });
+
+    describe('evaluate', () => {
+        it('returns No active session when none set', async () => {
+            const result = await (liveWatchTreeDataProvider as any).evaluate('myExpression');
+            expect(result.result).toBe('No active session');
+            expect(result.variablesReference).toBe(0);
+        });
+
+        it('maps string result into LiveWatchValue', async () => {
+            // mock active session with evaluateGlobalExpression returning a string
+            (liveWatchTreeDataProvider as any)._activeSession = {
+                evaluateGlobalExpression: jest.fn().mockResolvedValue('string-value'),
+                session: {}
+            };
+            const evalResult = await (liveWatchTreeDataProvider as any).evaluate('myExpression');
+            expect(evalResult.result).toBe('string-value');
+            expect(evalResult.variablesReference).toBe(0);
+        });
+
+        it('maps object result into LiveWatchValue', async () => {
+            const responseObj = { result: 'value', variablesReference: 1234 };
+            (liveWatchTreeDataProvider as any)._activeSession = {
+                evaluateGlobalExpression: jest.fn().mockResolvedValue(responseObj),
+                session: {}
+            };
+            const evalResult = await (liveWatchTreeDataProvider as any).evaluate('myExpression');
+            expect(evalResult.result).toBe('value');
+            expect(evalResult.variablesReference).toBe(1234);
+        });
+    });
 });
 /* eslint-enable @typescript-eslint/no-explicit-any */