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.

Sign up for GitHub

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

Please don't use implicit variable declarations! #151

Merged
merged 2 commits into from
Oct 17, 2017
Merged

Please don't use implicit variable declarations! #151

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
ae98cd6
Please don't use implicit variable declarations!
fluffywaffles Oct 17, 2017
cb8d370
Merge branch 'develop' into patch-1
Oct 17, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions source/guides/getting-started/testing-your-app.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -270,7 +270,7 @@ describe('The Login Page', function(){

it('sets auth cookie when logging in via form submission', function(){
// destructuring assignment of the this.currentUser object
{ username, password } = this.currentUser
const { username, password } = this.currentUser

cy.visit('/login')

Expand Down Expand Up @@ -351,7 +351,7 @@ describe('The Dashboard Page', function(){

it('logs in programmatically without using the UI', function(){
// destructuring assignment of the this.currentUser object
{ username, password } = this.currentUser
const { username, password } = this.currentUser

// programmatically log us in without needing the UI
cy.request('POST', '/login', {
Expand Down