Skip to content

add: e2e test automation#11

Open
Rink9 wants to merge 1 commit into
xaviranik:developfrom
Rink9:automation/e2e
Open

add: e2e test automation#11
Rink9 wants to merge 1 commit into
xaviranik:developfrom
Rink9:automation/e2e

Conversation

@Rink9
Copy link
Copy Markdown
Collaborator

@Rink9 Rink9 commented Apr 1, 2022

Added e2e test automation for boostimer

Comment thread tests/e2e/package.json
"homepage": "https://github.com/rink9/boostimer-e2e#readme",
"devDependencies": {
"@wordpress/e2e-test-utils": "^4.11.0",
"@wordpress/e2e-tests": "^1.20.0",
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

kindly fix the indention here.

import { URLS } from '../utils/urls';

describe( 'Sale timer test cases', () => {
it( 'should set sale timer', async () => {
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kindly adhere to WordPress JavaScript Coding Standards in terms of spacing and indentions. Write doc-block if necessary.

Comment thread tests/e2e/src/index.php
@@ -0,0 +1,2 @@
<?php
/* mimicing the dev files */ No newline at end of file
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't need this file here. Kindly remove the src directory.

}
}

export async function isElementExists( element ){
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider this approach:

export async function isElementExists( element ){
   return await page.$( element ) !== null;
} 

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants