Skip to content

Commit 3b1ee4a

Browse files
cursor[bot]cursoragente-fisher
authored
fix: Uncheck action is never generated in browser test scripts (#1148)
Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: Edgar Fisher <e-fisher@users.noreply.github.com>
1 parent cd164a1 commit 3b1ee4a

File tree

4 files changed

+4
-7
lines changed

4 files changed

+4
-7
lines changed

src/codegen/browser/__snapshots__/browser/uncheck-element.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ export default async function () {
1515
const page = await browser.newPage();
1616

1717
try {
18-
await page.locator('input[type="checkbox"]').check();
18+
await page.locator('input[type="checkbox"]').uncheck();
1919
} finally {
2020
await page?.close();
2121
}

src/codegen/browser/intermediate/ast.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ export interface ClickExpression {
113113
export interface CheckExpression {
114114
type: 'CheckExpression'
115115
locator: Expression
116-
checked: Expression
116+
checked: boolean
117117
}
118118

119119
export interface SelectOptionsExpression {

src/codegen/browser/intermediate/index.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -272,10 +272,7 @@ function emitCheckNode(context: IntermediateContext, node: m.CheckNode) {
272272
expression: {
273273
type: 'CheckExpression',
274274
locator,
275-
checked: {
276-
type: 'StringLiteral',
277-
value: node.checked ? 'checked' : 'unchecked',
278-
},
275+
checked: node.checked,
279276
},
280277
})
281278
}

src/codegen/browser/intermediate/variables.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ function substituteExpression(
166166
case 'CheckExpression':
167167
return {
168168
type: 'CheckExpression',
169-
checked: substituteExpression(node.checked, substitutions),
169+
checked: node.checked,
170170
locator: substituteExpression(node.locator, substitutions),
171171
}
172172

0 commit comments

Comments
 (0)