Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(codegen): testIdAttributeName configuration added to recorder #17

Merged
merged 1 commit into from
Mar 11, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 16 additions & 0 deletions examples/recorder-crx/options.html
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
<!DOCTYPE html>
<html lang="en">
<head>
<meta charset="UTF-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<title>Playwright CRX - Configuration</title>
</head>
<body>
<form id="options-form">
<label for="test-id">TestID Attribute Name:</label>
<input type="text" id="test-id" name="test-id" placeholder="Enter Attribute Name" pattern="[a-zA-Z][\w\-]*" title="Must be a valid attribute name">
<button id="submit" type="submit">Save</button>
</form>
<script type="module" src="/src/options.ts"></script>
</body>
</html>
6 changes: 5 additions & 1 deletion examples/recorder-crx/public/manifest.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,5 +19,9 @@
},
"default_title": "Record"
},
"permissions": ["debugger", "tabs", "contextMenus"]
"options_ui": {
"page": "options.html",
"open_in_tab": false
},
"permissions": ["debugger", "tabs", "contextMenus", "storage"]
}
14 changes: 12 additions & 2 deletions examples/recorder-crx/src/background.ts
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
*/

import type { CrxApplication } from 'playwright-crx';
import { crx, _debug, _setUnderTest } from 'playwright-crx';
import playwright, { crx, _debug, _setUnderTest } from 'playwright-crx';

type Mode = 'none' | 'recording' | 'inspecting' | 'assertingText' | 'recording-inspecting' | 'standby' | 'assertingVisibility' | 'assertingValue';

Expand Down Expand Up @@ -106,6 +106,10 @@ async function attach(tab: chrome.tabs.Tab) {
}
}

async function setTestIdAttributeName(testIdAttributeName: string) {
playwright.selectors.setTestIdAttribute(testIdAttributeName);
}

chrome.action.onClicked.addListener(attach);

chrome.contextMenus.create({
Expand All @@ -118,5 +122,11 @@ chrome.contextMenus.onClicked.addListener(async (_, tab) => {
if (tab) await attach(tab);
});

chrome.storage.sync.onChanged.addListener(async ({ testIdAttributeName }) => {
if (!testIdAttributeName) return;

await setTestIdAttributeName(testIdAttributeName.newValue);
});

// for testing
Object.assign(self, { attach, _debug, _setUnderTest });
Object.assign(self, { attach, setTestIdAttributeName, _debug, _setUnderTest });
51 changes: 51 additions & 0 deletions examples/recorder-crx/src/options.css
Original file line number Diff line number Diff line change
@@ -0,0 +1,51 @@
body {
font-family: sans-serif;
margin: 20px;
}

h1 {
text-align: center;
margin-bottom: 15px;
}

form {
display: flex;
flex-direction: column;
gap: 10px;
}

label {
font-weight: bold;
}

input[type="text"] {
padding: 5px;
border: 1px solid #ccc;
border-radius: 4px;
width: 100%;
}

button[type="submit"] {
background-color: #4CAF50;
border: none;
color: white;
padding: 10px 20px;
text-align: center;
text-decoration: none;
display: inline-block;
font-size: 16px;
margin: 4px 2px;
cursor: pointer;
border-radius: 4px;
transition: background-color 0.2s ease-in-out;
}

button:hover {
background-color: #45A049;
}

button[type="submit"]:disabled {
background-color: #ccc;
color: #666;
cursor: not-allowed;
}
31 changes: 31 additions & 0 deletions examples/recorder-crx/src/options.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
import './options.css';

async function initialize() {
const formElem = document.getElementById('options-form') as HTMLFormElement;
const testIdAttributeNameElem = document.getElementById('test-id') as HTMLInputElement;
const submitElem = document.getElementById('submit') as HTMLButtonElement;

if (!testIdAttributeNameElem || !formElem || !submitElem) return;

testIdAttributeNameElem.addEventListener('input', () => {
submitElem.disabled = false;
});

formElem.addEventListener('submit', (e) => {
if (!formElem.reportValidity()) return;

e.preventDefault();

submitElem.disabled = true;
const testIdAttributeName = testIdAttributeNameElem.value;
chrome.storage.sync.set({ testIdAttributeName }).catch(() => {});

return false;
});

submitElem.disabled = true;
const { testIdAttributeName } = await chrome.storage.sync.get('testIdAttributeName');
testIdAttributeNameElem.value = testIdAttributeName ?? 'data-testid';
}

initialize();
1 change: 1 addition & 0 deletions examples/recorder-crx/vite.config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ export default defineConfig({
plugins: [sourcemaps()],
input: {
'background': path.resolve(__dirname, 'src/background.ts'),
'options': path.resolve(__dirname, 'options.html'),
},
output: {
entryFileNames: '[name].js',
Expand Down
39 changes: 38 additions & 1 deletion tests/crx/recorder.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -204,7 +204,7 @@ test('should record with all supported actions and assertions', async ({ context
});
const context = await browser.newContext();
const page = await context.newPage();
await page.goto('http://127.0.0.1:3000/root.html');
await page.goto('${baseURL}/root.html');
await page.getByRole('checkbox').check();
await page.getByRole('button', { name: 'button' }).click();
await page.getByRole('checkbox').uncheck();
Expand All @@ -227,3 +227,40 @@ test('should record with all supported actions and assertions', async ({ context

await expect(recorderPage.locator('.CodeMirror-line')).toHaveText(code.split('\n'));
});

test('should record with custom testid', async ({ page, attachRecorder, recordAction, baseURL, extensionServiceWorker }) => {
const recorderPage = await attachRecorder(page);
await recordAction(() => page.goto(`${baseURL}/empty.html`));
await page.setContent(`
<button data-testid='btn-testid'>Button</button>
<button data-foobar='btn-foobar'>Button</button>
`);
await recordAction(() => page.locator('button').first().click());
await extensionServiceWorker.evaluate(async () => {
await (globalThis as any).setTestIdAttributeName('data-foobar');
});
// injected recorder poll period
await page.waitForTimeout(1000);
await recordAction(() => page.locator('button').nth(1).click());

await recorderPage.getByTitle('Record').click();

const code = `const { chromium } = require('playwright');

(async () => {
const browser = await chromium.launch({
headless: false
});
const context = await browser.newContext();
const page = await context.newPage();
await page.goto('${baseURL}/empty.html');
await page.getByTestId('btn-testid').click();
await page.getByTestId('btn-foobar').click();

// ---------------------
await context.close();
await browser.close();
})();`;

await expect(recorderPage.locator('.CodeMirror-line')).toHaveText(code.split('\n'));
});
Loading