more improvements

cypress/integration/article.spec.js

@@ -3,7 +3,9 @@ import article from '../selectors/article.sel'
 
 describe('Article', () => {
     beforeEach(() => {
-        cy.login()
+        cy.register().then((email) => {
+            cy.login(email)
+        })
         cy.visit('/editor/')
     })
 

cypress/integration/register.spec.js

@@ -0,0 +1,44 @@
+import registration from '../selectors/register.sel'
+import header from '../selectors/header.sel'
+
+describe('Register', () => {
+    // actually we should not use let here, check register_aliases.spec
+    // https://docs.cypress.io/guides/core-concepts/variables-and-aliases.html#Aliases
+    let username
+    let email
+
+    beforeEach(() => {
+        // we need random username and email each test
+        username = `cy${Math.random().toString().slice(2, 8)}`
+        email = `${username}@mailinator.com`
+        cy.visit('/register')
+    })
+
+    it('can register a new account', () => {
+        // added delay as sometimes it can make tests flaky if typing too fast (default is 10)
+        cy.get(registration.usernameField).type(username, { delay: 50 })
+        cy.get(registration.emailField).type(email)
+        cy.get(registration.passwordField).type('Testtest1')
+        cy.get(registration.signUpButton).click()
+        cy.get(header.settingsLink).should('be.visible')
+    })
+
+    it('check registration request body and response', () => {
+        cy.intercept('/api/users').as('loginRequest')
+        cy.get(registration.usernameField).type(username)
+        cy.get(registration.emailField).type(email)
+        cy.get(registration.passwordField).type('Testtest1{enter}')
+
+        cy.wait('@loginRequest').then((xhr) => {
+            // check request body
+            expect(xhr.request.body.user.email).to.eq(email)
+            expect(xhr.request.body.user.password).to.eq('Testtest1')
+            expect(xhr.request.body.user.username).to.eq(username)
+            // check response body
+            expect(xhr.response.body.user.email).to.eq(email)
+            expect(xhr.response.body.user.id).not.to.eq(null)
+            expect(xhr.response.body.user.token).not.to.eq(null)
+        })
+        cy.get(header.settingsLink).should('be.visible')
+    })
+})

cypress/integration/register_aliases.spec.js

@@ -0,0 +1,43 @@
+import registration from '../selectors/register.sel'
+import header from '../selectors/header.sel'
+
+describe('Register', () => {
+    beforeEach(() => {
+        // we need random username and email each test
+        const random = `cy${Math.random().toString().slice(2, 8)}`
+        // use alias instead of let
+        cy.wrap(random).as('username')
+        cy.wrap(`${random}@mailinator.com`).as('email')
+        cy.visit('/register')
+    })
+
+    // this is not an arrow function anymore =>
+    // https://docs.cypress.io/guides/core-concepts/variables-and-aliases.html#Avoiding-the-use-of-this
+    it('can register a new account', function () {
+        // added delay as sometimes it can make tests flaky if typing too fast (default is 10)
+        cy.get(registration.usernameField).type(this.username, { delay: 50 })
+        cy.get(registration.emailField).type(this.email)
+        cy.get(registration.passwordField).type('Testtest1')
+        cy.get(registration.signUpButton).click()
+        cy.get(header.settingsLink).should('be.visible')
+    })
+
+    it('check registration request body and response', function () {
+        cy.intercept('/api/users').as('loginRequest')
+        cy.get(registration.usernameField).type(this.username)
+        cy.get(registration.emailField).type(this.email)
+        cy.get(registration.passwordField).type('Testtest1{enter}')
+
+        cy.wait('@loginRequest').then((xhr) => {
+            // check request body
+            expect(xhr.request.body.user.email).to.eq(this.email)
+            expect(xhr.request.body.user.password).to.eq('Testtest1')
+            expect(xhr.request.body.user.username).to.eq(this.username)
+            // check response body
+            expect(xhr.response.body.user.email).to.eq(this.email)
+            expect(xhr.response.body.user.id).not.to.eq(null)
+            expect(xhr.response.body.user.token).not.to.eq(null)
+        })
+        cy.get(header.settingsLink).should('be.visible')
+    })
+})

cypress/selectors/header.sel.js

@@ -0,0 +1,3 @@
+module.exports = {
+    settingsLink: '[ui-sref*=settings]'
+}

cypress/selectors/register.sel.js

@@ -0,0 +1,6 @@
+module.exports = {
+    usernameField: '[ng-model*=username]',
+    emailField: '[ng-model*=email]',
+    passwordField: '[ng-model*=password]',
+    signUpButton: '.btn'
+}

cypress/support/index.js

@@ -19,6 +19,3 @@ import './register.cmd'
 
 // Alternatively you can use CommonJS syntax:
 // require('./commands')
-before(() => {
-    cy.register()
-})

cypress/support/register.cmd.js

@@ -14,7 +14,6 @@ Cypress.Commands.add('register', () => {
     })
         .then((response) => {
             expect(response.status).to.eq(200)
-            Cypress.env('email', email)
             return email
         })
 })