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

{log:false} of clearLocalStorage doesn't work. #6342

Open
sainthkh opened this issue Feb 6, 2020 · 3 comments · May be fixed by #6495
Open

{log:false} of clearLocalStorage doesn't work. #6342

sainthkh opened this issue Feb 6, 2020 · 3 comments · May be fixed by #6495

Comments

@sainthkh
Copy link
Collaborator

@sainthkh sainthkh commented Feb 6, 2020

Current behavior:

cy.clearLocalStorage({log:false}) should not show the command on the reporter. But it does.

Desired behavior:

It should not show the command.

Test code to reproduce

it('test', () => {
  cy.clearLocalStorage({log:false})
})

Result:

Screenshot from 2020-02-06 15-42-33

Versions

3.8.3

@Jelijah

This comment has been minimized.

Copy link

@Jelijah Jelijah commented Feb 9, 2020

Hello, I would like to work on this bug. Am I able to start working on this? What is required?

@sainthkh

This comment has been minimized.

Copy link
Collaborator Author

@sainthkh sainthkh commented Feb 10, 2020

Just go on and submit a PR. Here's the guide.

If you have anything to ask, feel free to do it.

maddhruv added a commit to maddhruv/cypress that referenced this issue Feb 18, 2020
@maddhruv maddhruv linked a pull request that will close this issue Feb 18, 2020
0 of 5 tasks complete
@maddhruv

This comment has been minimized.

Copy link

@maddhruv maddhruv commented Feb 18, 2020

@sainthkh raised a PR here - #6495

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants
You can’t perform that action at this time.