Skip to content

Conversation

tanyatrayanava
Copy link
Contributor

No description provided.

.meta({ rte: rte.standalone })
.after(async() => {
//Delete database
await deleteDatabase(ossStandaloneConfig.databaseName);

Choose a reason for hiding this comment

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

Do you aware it's overlapping the afterEach? Also, it's the same, why need it twice?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks, you are right
removed

//Delete database
await deleteDatabase(ossStandaloneConfig.databaseName);
})
('Verify that wheh popover Editor is displayed user can see it is populated with the script that was detected between the quotes or it is blank if quotes were empty', async t => {

Choose a reason for hiding this comment

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

You have a typo (wheh)
Also, IMO the names of the tests in this fixture are too long

Copy link
Contributor Author

Choose a reason for hiding this comment

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

reduced the names where possible

//Close popover editor and re-open with added script
await t.pressKey('esc');
await t.typeText(workbenchPage.queryInput, `${command} "${script}`, { replace: true });
await t.debug();

Choose a reason for hiding this comment

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

Redundant

Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed

@tanyatrayanava tanyatrayanava merged commit 13e988e into main Mar 21, 2022
@tanyatrayanava tanyatrayanava deleted the feature/e2e-cypher-tests branch March 21, 2022 06:34
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