Add complex data type support to live watch

__mocks__/vscode.js

@@ -36,6 +36,8 @@ const MockTreeItemCollapsibleState = {
     Expanded: 2 
 };
 
+const registerTreeDataProviderMock = jest.fn(() => ({ dispose: jest.fn() }));
+
 class MockTreeItem {
     label;
     description;
@@ -71,6 +73,7 @@ module.exports = {
             warn: jest.fn(),
             error: jest.fn(),
         })),
+        registerTreeDataProvider: registerTreeDataProviderMock,
         showWarningMessage: jest.fn(),
         createStatusBarItem: jest.fn(),
         showQuickPick: jest.fn(),

package.json

@@ -52,15 +52,16 @@
         {
           "id": "cmsis-debugger",
           "title": "CMSIS Debug",
-          "icon": "$(rocket)"
+          "icon": "images/trace-and-live-dark.svg"
         }
       ]
     },
     "views": {
       "cmsis-debugger": [
         {
           "id": "cmsis-debugger.liveWatch",
-          "name": "Live Watch"
+          "name": "Live Watch",
+          "icon": "images/live-watch-dark.svg"
         }
       ]
     },

src/debug-session/gdbtarget-debug-session.test.ts

@@ -59,18 +59,18 @@ describe('GDBTargetDebugSession', () => {
 
     it('evaluates a global expression without active stack frame and returns a value', async () => {
         // Only mock relevant properties, return value is body of EvaluateResponse
-        (debugSession.customRequest as jest.Mock).mockReturnValueOnce({ result: '1234567' });
+        (debugSession.customRequest as jest.Mock).mockReturnValueOnce({ result: '1234567', variableReference: 0 });
         const result = await gdbTargetSession.evaluateGlobalExpression('myGlobalVariable');
-        expect(result).toEqual('1234567');
+        expect(result).toEqual({ result: '1234567', variableReference: 0 });
         expect(debugSession.customRequest as jest.Mock).toHaveBeenCalledWith('evaluate', { expression: 'myGlobalVariable', frameId: 0, context: 'hover' });
     });
 
     it('evaluates a global expression with active stack frame and returns a value', async () => {
         // Only mock relevant properties, return value is body of EvaluateResponse
-        (debugSession.customRequest as jest.Mock).mockReturnValueOnce({ result: '1234567' });
+        (debugSession.customRequest as jest.Mock).mockReturnValueOnce({ result: '1234567', variableReference: 0 });
         (vscode.debug.activeStackItem as unknown) = { session: debugSession, threadId: 1, frameId: 2 };
         const result = await gdbTargetSession.evaluateGlobalExpression('myGlobalVariable');
-        expect(result).toEqual('1234567');
+        expect(result).toEqual({ result: '1234567', variableReference: 0 });
         expect(debugSession.customRequest as jest.Mock).toHaveBeenCalledWith('evaluate', { expression: 'myGlobalVariable', frameId: 2, context: 'hover' });
         // restore default
         (vscode.debug.activeStackItem as unknown) = undefined;

src/debug-session/gdbtarget-debug-session.ts

@@ -54,7 +54,7 @@ export class GDBTargetDebugSession {
         this._cbuildRunParsePromise = undefined;
     }
 
-    public async evaluateGlobalExpression(expression: string, context = 'hover'): Promise<string> {
+    public async evaluateGlobalExpression(expression: string, context = 'hover'): Promise<DebugProtocol.EvaluateResponse['body'] | string> {
         try {
             const frameId = (vscode.debug.activeStackItem as vscode.DebugStackFrame)?.frameId ?? 0;
             const args: DebugProtocol.EvaluateArguments = {
@@ -63,7 +63,7 @@ export class GDBTargetDebugSession {
                 context: context
             };
             const response = await this.session.customRequest('evaluate', args) as DebugProtocol.EvaluateResponse['body'];
-            return response.result;
+            return response;
         } catch (error: unknown) {
             const errorMessage = (error as Error)?.message;
             logger.debug(`Session '${this.session.name}': Failed to evaluate global expression '${expression}' - '${errorMessage}'`);

src/features/cpu-states/cpu-states.ts

@@ -215,7 +215,10 @@ export class CpuStates {
 
     protected async getFrequency(): Promise<number|undefined> {
         const result = await this.activeSession?.evaluateGlobalExpression('SystemCoreClock');
-        const frequencyString = result?.match(/\d+/) ? result : undefined;
+        if (typeof result == 'string') {
+            return undefined;
+        }
+        const frequencyString = result?.result.match(/\d+/) ? result.result : undefined;
         if (!frequencyString) {
             return undefined;
         }