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

fix Clear out the localStorage after test (close #1546) #1590

Merged
merged 2 commits into from
Jul 11, 2017

Conversation

LavrovArtem
Copy link
Contributor

No description provided.

@testcafe-build-bot
Copy link
Collaborator

❌ Tests for the commit f8f1a9e have failed. See details:

@LavrovArtem
Copy link
Contributor Author

@testcafe-build-bot \retest

@testcafe-build-bot
Copy link
Collaborator

❌ Tests for the commit f8f1a9e have failed. See details:

@LavrovArtem
Copy link
Contributor Author

@testcafe-build-bot \retest

@testcafe-build-bot
Copy link
Collaborator

✅ Tests for the commit f8f1a9e have passed. See details:

fixture `Storages`
.page `http://localhost:3000/fixtures/driver/pages/set-value-to-storages.html`;

const saveNativeStorageKey = ClientFunction(storageName => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's call if setNativeStorageKey .
And we can do not return value from this method

@testcafe-build-bot
Copy link
Collaborator

✅ Tests for the commit 3d13a96 have passed. See details:

@AlexanderMoskovkin AlexanderMoskovkin merged commit b11b2fb into DevExpress:master Jul 11, 2017
@LavrovArtem LavrovArtem deleted the i1546 branch July 14, 2017 08:11
AlexanderMoskovkin pushed a commit to AndreyBelym/testcafe that referenced this pull request Jul 20, 2017
…evExpress#1590)

* fix `Clear out the localStorage after test` (close DevExpress#1546)

* fix review's issues
kirovboris pushed a commit to kirovboris/testcafe-phoenix that referenced this pull request Dec 18, 2019
…evExpress#1590)

* fix `Clear out the localStorage after test` (close DevExpress#1546)

* fix review's issues
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.

4 participants