feat(codegen): include framepPath in jsonl format

Fixes: #34157
microsoft · Jan 12, 2025 · ffccfff · ffccfff
1 parent 6179b5b
commit ffccfff
Show file tree

Hide file tree

Showing 2 changed files with 33 additions and 1 deletion.
diff --git a/packages/playwright-core/src/server/codegen/jsonl.ts b/packages/playwright-core/src/server/codegen/jsonl.ts
@@ -28,7 +28,7 @@ export class JsonlLanguageGenerator implements LanguageGenerator {
     const locator = (actionInContext.action as any).selector ? JSON.parse(asLocator('jsonl', (actionInContext.action as any).selector)) : undefined;
     const entry = {
       ...actionInContext.action,
-      pageAlias: actionInContext.frame.pageAlias,
+      ...actionInContext.frame,
       locator,
     };
     return JSON.stringify(entry);

diff --git a/tests/library/inspector/cli-codegen-3.spec.ts b/tests/library/inspector/cli-codegen-3.spec.ts
@@ -55,6 +55,14 @@ test.describe('cli codegen', () => {
     expect.soft(sources.get('C#')!.text).toContain(`
 await page.GetByRole(AriaRole.Button, new() { Name = "Submit" }).First.ClickAsync();`);
 
+    const clickAction = sources.get('JSON')!.actions.map(l => JSON.parse(l)).find(a => a.name === 'click');
+    expect.soft(clickAction).toMatchObject({
+      name: 'click',
+      selector: 'internal:role=button[name="Submit"i] >> nth=0',
+      framePath: [],
+      pageAlias: 'page',
+    });
+
     expect(message.text()).toBe('click1');
   });
 
@@ -116,6 +124,14 @@ await page.GetByRole(AriaRole.Button, new() { Name = "Submit" }).Nth(1).ClickAsy
 
     expect.soft(sources.get('C#')!.text).toContain(`
 await page.Locator("#frame1").ContentFrame.GetByText("Hello1").ClickAsync();`);
+
+    const clickAction = sources.get('JSON')!.actions.map(l => JSON.parse(l)).find(a => a.name === 'click');
+    expect.soft(clickAction).toMatchObject({
+      name: 'click',
+      selector: 'internal:text="Hello1"i',
+      framePath: ['#frame1'],
+      pageAlias: 'page',
+    });
   });
 
   test('should generate frame locators (2)', async ({ openRecorder, server }) => {
@@ -141,6 +157,14 @@ await page.Locator("#frame1").ContentFrame.GetByText("Hello1").ClickAsync();`);
 
     expect.soft(sources.get('C#')!.text).toContain(`
 await page.Locator("#frame1").ContentFrame.Locator("iframe").ContentFrame.GetByText("Hello2").ClickAsync();`);
+
+    const clickAction = sources.get('JSON')!.actions.map(l => JSON.parse(l)).find(a => a.name === 'click');
+    expect.soft(clickAction).toMatchObject({
+      name: 'click',
+      selector: 'internal:text="Hello2"i',
+      framePath: ['#frame1', 'iframe'],
+      pageAlias: 'page',
+    });
   });
 
   test('should generate frame locators (3)', async ({ openRecorder, server }) => {
@@ -166,6 +190,14 @@ await page.Locator("#frame1").ContentFrame.Locator("iframe").ContentFrame.GetByT
 
     expect.soft(sources.get('C#')!.text).toContain(`
 await page.Locator("#frame1").ContentFrame.Locator("iframe").ContentFrame.Locator("iframe").Nth(2).ContentFrame.GetByText("HelloNameAnonymous").ClickAsync();`);
+
+    const clickAction = sources.get('JSON')!.actions.map(l => JSON.parse(l)).find(a => a.name === 'click');
+    expect.soft(clickAction).toMatchObject({
+      name: 'click',
+      selector: 'internal:text="HelloNameAnonymous"i',
+      framePath: ['#frame1', 'iframe', 'iframe >> nth=2'],
+      pageAlias: 'page',
+    });
   });
 
   test('should generate frame locators (4)', async ({ openRecorder, server }) => {