From 9deb427afcdd4446680e367afb67397a728d5d88 Mon Sep 17 00:00:00 2001
From: MatheusRich <matheusrichardt@gmail.com>
Date: Mon, 12 Feb 2018 23:05:26 -0200
Subject: [PATCH 1/8] Adding login command to cypress

This allows use the login function in every place, avoiding code duplication
---
 cypress/integration/authorization_spec.js | 37 ++++++++---------------
 cypress/integration/projects_spec.js      | 32 +++-----------------
 cypress/integration/users_spec.js         | 25 +--------------
 cypress/support/commands.js               | 32 ++++++++++++++++++++
 4 files changed, 51 insertions(+), 75 deletions(-)

diff --git a/cypress/integration/authorization_spec.js b/cypress/integration/authorization_spec.js
index 9307c9ce..5dee08e2 100644
--- a/cypress/integration/authorization_spec.js
+++ b/cypress/integration/authorization_spec.js
@@ -1,7 +1,5 @@
 describe('Authorization tests', () => {
   beforeEach(() => {
-    cy.visit('localhost:8080/#/');
-
     cy.server();
 
     // Stubing server response
@@ -14,6 +12,8 @@ describe('Authorization tests', () => {
   });
 
   it('should access homepage', () => {
+    cy.visit('localhost:8080/#/');
+
     cy.title().should('include', 'Falko');
 
     cy.get('#loginRegisterComponent').within(() => {
@@ -28,16 +28,7 @@ describe('Authorization tests', () => {
   });
 
   it('should log in user', () => {
-    cy.get('form').within(() => {
-      cy.get('input:first').eq(0).should('have.attr', 'placeholder', 'Email')
-        .type('carla@gmail.com')
-        .should('have.value', 'carla@gmail.com');
-
-      cy.get('input:last').eq(0).should('have.attr', 'placeholder', 'Password')
-        .type('123456789')
-        .should('have.value', '123456789');
-    });
-    cy.get('#loginButton').click();
+    cy.login()
 
     cy.url().should('eq', 'http://localhost:8080/#/projects');
   });
@@ -55,6 +46,8 @@ describe('Authorization tests', () => {
         },
       },
     }).as('invalidLogin');
+    
+    cy.visit('localhost:8080/#/');
 
     cy.get('form').within(() => {
       cy.get('input:first').eq(0).should('have.attr', 'placeholder', 'Email')
@@ -94,6 +87,8 @@ describe('Authorization tests', () => {
       response: 'fixture:login.json',
     }).as('register');
 
+    cy.visit('localhost:8080/#/');
+
     cy.get('#pills-register-tab').click();
 
     cy.get('form').within(() => {
@@ -127,6 +122,8 @@ describe('Authorization tests', () => {
       response: {},
     }).as('invalidRegister');
 
+    cy.visit('localhost:8080/#/');
+
     cy.get('#pills-register-tab').click();
 
     cy.get('form').within(() => {
@@ -153,17 +150,7 @@ describe('Authorization tests', () => {
   });
 
   it('should logout user', () => {
-    cy.get('form').within(() => {
-      cy.get('input:first').eq(0).should('have.attr', 'placeholder', 'Email')
-        .type('carla@gmail.com')
-        .should('have.value', 'carla@gmail.com');
-
-      cy.get('input:last').eq(0).should('have.attr', 'placeholder', 'Password')
-        .type('123456789')
-        .should('have.value', '123456789');
-    });
-
-    cy.get('#loginButton').click();
+    cy.login();
 
     cy.get('#noProjects');
 
@@ -183,7 +170,9 @@ describe('Authorization tests', () => {
   })
 
   it('should not access any page with user logged out', function(){
-    cy.visit('localhost:8080/#/asdfasdf')
+    cy.visit('localhost:8080/#/');
+    
+    cy.visit('localhost:8080/#/invalidURL')
 
     cy.get('#loginComponent')
   })
diff --git a/cypress/integration/projects_spec.js b/cypress/integration/projects_spec.js
index 11fd08d7..ec7072b5 100644
--- a/cypress/integration/projects_spec.js
+++ b/cypress/integration/projects_spec.js
@@ -1,27 +1,5 @@
 describe('Projects tests', function(){
-
-  function login(){
-    cy.route({
-      method: 'POST',
-      url: '/authenticate',
-      status: 200,
-      response: 'fixture:login.json'
-    }).as('login')
-
-    cy.get('form').within(function () {
-      cy.get('input:first').eq(0).should('have.attr', 'placeholder', 'Email')
-        .type('carla@gmail.com').should('have.value', 'carla@gmail.com')
-
-      cy.get('input:last').eq(0).should('have.attr', 'placeholder', 'Password')
-        .type('123456789').should('have.value', '123456789')
-    })
-
-    cy.get('.falko-button').eq(0).click()
-  }
-
   beforeEach(function(){
-    cy.visit('localhost:8080/#/')
-
     cy.server()
 
     cy.route({
@@ -34,7 +12,7 @@ describe('Projects tests', function(){
 
   it('should get projects', function(){
 
-    login()
+    cy.login()
 
     cy.get('.card-header').eq(0).contains('Owla')
     cy.get('.card-body').within(function(){
@@ -74,7 +52,7 @@ describe('Projects tests', function(){
       ]
     }).as('addProject')
 
-    login()
+    cy.login()
 
     cy.get('.falko-button').eq(0).click()
 
@@ -126,7 +104,7 @@ describe('Projects tests', function(){
 
   it('should cancel add project process', function () {
 
-    login()
+    cy.login()
 
     cy.get('#addButton').contains('Add a Project').click()
 
@@ -141,7 +119,7 @@ describe('Projects tests', function(){
   })
 
   it('should edit a project', function(){
-    login()
+    cy.login()
 
     cy.route({
       method: 'GET',
@@ -219,7 +197,7 @@ describe('Projects tests', function(){
   })
 
   it('should delete project', function(){
-    login()
+    cy.login()
 
     cy.route({
       method: 'GET',
diff --git a/cypress/integration/users_spec.js b/cypress/integration/users_spec.js
index 688d34a1..3e053be8 100644
--- a/cypress/integration/users_spec.js
+++ b/cypress/integration/users_spec.js
@@ -1,28 +1,5 @@
 describe('Users tests', () => {
-  function login() {
-    cy.route({
-      method: 'POST',
-      url: '/authenticate',
-      status: 200,
-      response: 'fixture:login.json',
-    }).as('login');
-
-    cy.get('form').within(() => {
-      cy.get('input:first').eq(0).should('have.attr', 'placeholder', 'Email')
-        .type('carla@gmail.com')
-        .should('have.value', 'carla@gmail.com');
-
-      cy.get('input:last').eq(0).should('have.attr', 'placeholder', 'Password')
-        .type('123456789')
-        .should('have.value', '123456789');
-    });
-
-    cy.get('#loginButton').click();
-  }
-
   beforeEach(() => {
-    cy.visit('localhost:8080/#/');
-
     cy.server();
 
     cy.route({
@@ -32,7 +9,7 @@ describe('Users tests', () => {
       response: 'fixture:projects.json',
     }).as('getProjects');
 
-    login();
+    cy.login();
 
     cy.route({
       method: 'GET',
diff --git a/cypress/support/commands.js b/cypress/support/commands.js
index c1f5a772..5f6cb5be 100644
--- a/cypress/support/commands.js
+++ b/cypress/support/commands.js
@@ -23,3 +23,35 @@
 //
 // -- This is will overwrite an existing command --
 // Cypress.Commands.overwrite("visit", (originalFn, url, options) => { ... })
+
+Cypress.Commands.add('login', () => {
+  const user = {
+    email: 'carla@email.com',
+    password: '123456789',
+  }
+
+  cy.visit('localhost:8080/#/');
+
+  cy.server();
+
+  cy.route({
+    method: 'POST',
+    url: '/authenticate',
+    status: 200,
+    response: 'fixture:login.json',
+  }).as('login');
+
+  cy.get('form').within(() => {
+    cy.get('input:first').eq(0).should('have.attr', 'placeholder', 'Email')
+      .type(user.email)
+      .should('have.value', 'carla@email.com');
+
+    cy.get('input:last').eq(0).should('have.attr', 'placeholder', 'Password')
+      .type(user.password)
+      .should('have.value', '123456789');
+  });
+
+  cy.get('#loginButton').click();
+
+  cy.wait('@login')
+})
\ No newline at end of file

From 62a5220ce9aaa15c042444010c3deb70c6e63e59 Mon Sep 17 00:00:00 2001
From: MatheusRich <matheusrichardt@gmail.com>
Date: Wed, 14 Feb 2018 22:19:36 -0200
Subject: [PATCH 2/8] Adding project.json fixture

---
 cypress/fixtures/project.json        |  9 +++++++++
 cypress/integration/projects_spec.js | 22 ++--------------------
 2 files changed, 11 insertions(+), 20 deletions(-)
 create mode 100644 cypress/fixtures/project.json

diff --git a/cypress/fixtures/project.json b/cypress/fixtures/project.json
new file mode 100644
index 00000000..db32d647
--- /dev/null
+++ b/cypress/fixtures/project.json
@@ -0,0 +1,9 @@
+{
+  "description": "Agile Projects Manager",
+  "github_slug": "fga-gpp-mds/owla",
+  "id": 2,
+  "is_project_from_github": true,
+  "is_scoring": false,
+  "name": "Falko",
+  "user_id": 1
+}
\ No newline at end of file
diff --git a/cypress/integration/projects_spec.js b/cypress/integration/projects_spec.js
index ec7072b5..74588879 100644
--- a/cypress/integration/projects_spec.js
+++ b/cypress/integration/projects_spec.js
@@ -125,16 +125,7 @@ describe('Projects tests', function(){
       method: 'GET',
       url: '/projects\/2',
       status: 200,
-      response:
-        {
-          "description":"Agile Projects Manager" ,
-          "github_slug": "fga-gpp-mds/owla",
-          "id":2,
-          "is_project_from_github":true,
-          "is_scoring":false,
-          "name":"Falko",
-          "user_id":1,
-        }
+      response: 'fixture:project.json'
     }).as('getProject')
 
     cy.get('.card-body').eq(1).click()
@@ -203,16 +194,7 @@ describe('Projects tests', function(){
       method: 'GET',
       url: '/projects\/1',
       status: 200,
-      response:
-        {
-          "description": "Agile Projects Manager",
-          "github_slug": "fga-gpp-mds/owla",
-          "id": 2,
-          "is_project_from_github": true,
-          "is_scoring": false,
-          "name": "Falko",
-          "user_id": 1,
-        }
+      response: 'fixture:project.json'
     }).as('getProject')
 
     cy.route({

From 92b8a2c1093c4e5f518ef0dff34453bd8e87c9dc Mon Sep 17 00:00:00 2001
From: MatheusRich <matheusrichardt@gmail.com>
Date: Wed, 14 Feb 2018 22:52:49 -0200
Subject: [PATCH 3/8] Adding dynamic  ids in Projects.vue

---
 src/components/Projects/Projects.vue | 28 ++++++++++++++--------------
 1 file changed, 14 insertions(+), 14 deletions(-)

diff --git a/src/components/Projects/Projects.vue b/src/components/Projects/Projects.vue
index f53fe3cc..03ce009e 100644
--- a/src/components/Projects/Projects.vue
+++ b/src/components/Projects/Projects.vue
@@ -5,12 +5,12 @@
     </div>
     <tr> <td colspan="2" bgcolor="#FFFFFF" height="30">&nbsp;</td> </tr>
     <div class="row justify-content-around" v-for="i in Math.ceil(projects.length / 2)">
-      <div v-for="project in projects.slice((i-1) * 2,i*2)" class="col-5">
+      <div v-for="project in projects.slice((i-1)*2, i*2)" :key="project.id" class="col-5">
         <div align="center">
-          <div class="card" id="projectCard">
+          <div class="card project-card" :id="`projectCard${project.id}`">
             <router-link v-bind:to="'/projects/'+project.id">
-              <div class="card-header" id="projectHeader">
-                <div class="row align-itens-around" id="projectTitle">
+              <div class="card-header project-header" :id="`projectHeader${project.id}`">
+                <div class="row align-itens-around project-title" :id="`projectTitle${project.id}`">
                   <div class="col">
                     <h4 class="no-margin float-left"> {{project.name}}</h4>
                   </div>
@@ -26,7 +26,7 @@
                     </p>
                   </div>
                   <div class="col-md-4 justify-content-center">
-                    <grade id="grade" align="center" v-bind:project="project.id"></grade>
+                    <grade class="grade" :id="`grade${project.id}`" align="center" v-bind:project="project.id"></grade>
                   </div>
                 </div>
               </div>
@@ -151,21 +151,21 @@ div a {
   color: #28639A;
 }
 
-#projectCard:hover {
-  box-shadow: 0 4px 12px 0 rgba(0,0,0,0.2);
-  border-color: #5D6A6F;
-}
-
-#projectCard {
+.project-card {
   box-shadow: 0 2px 4px 0 rgba(0,0,0,0.3);
   transition: 0.2s;
 }
 
-#projectHeader {
+.project-card:hover {
+  box-shadow: 0 4px 12px 0 rgba(0,0,0,0.2);
+  border-color: #5D6A6F;
+}
+
+.project-header {
   background-color: #5D6A6F;
 }
 
-#projectTitle {
+.project-title {
   margin: 0;
   color: white;
 }
@@ -175,7 +175,7 @@ div a {
   font-style: italic;
 }
 
-#grade {
+.grade {
   margin-left: 1.5em;
 }
 </style>

From 54ef3632bae84dfcad28adf084c063cda523759a Mon Sep 17 00:00:00 2001
From: MatheusRich <matheusrichardt@gmail.com>
Date: Wed, 14 Feb 2018 22:53:16 -0200
Subject: [PATCH 4/8] Adding ids to buttons in Project.vue

---
 src/components/Projects/Project.vue | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/components/Projects/Project.vue b/src/components/Projects/Project.vue
index bef0ce99..0e0bc028 100644
--- a/src/components/Projects/Project.vue
+++ b/src/components/Projects/Project.vue
@@ -16,20 +16,20 @@
                 <div class="row">
                   <div align="center">
                     <router-link v-bind:to="'/projects/'+project.id+'/issues'">
-                      <button type="button" class="btn btn-info btn-md falko-button" v-if="isFromProjectGitHub()">
+                      <button type="button" class="btn btn-info btn-md falko-button" id="backlog" v-if="isFromProjectGitHub()">
                         Backlog
                       </button>
                     </router-link>
                   </div>
                   <div align="center">
                     <router-link v-bind:to="'/projects/'+project.id+'/releases'">
-                      <button type="button" class="btn btn-info btn-md falko-button">
+                      <button type="button" class="btn btn-info btn-md falko-button" id="releases">
                         Releases
                       </button>
                     </router-link>
                   </div>
                   <div align="center">
-                    <addGrade></addGrade>
+                    <addGrade id="addGrade"></addGrade>
                   </div>
                   <div align="center">
                     <edit-project v-on:edited-project="refreshProject()"></edit-project>

From 97663dcc1115ee110f821d5e05d10e55e7bc4905 Mon Sep 17 00:00:00 2001
From: MatheusRich <matheusrichardt@gmail.com>
Date: Mon, 19 Feb 2018 21:41:22 -0300
Subject: [PATCH 5/8] Changing id of Add Sprint button

---
 src/components/Sprints/AddSprint.vue | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/components/Sprints/AddSprint.vue b/src/components/Sprints/AddSprint.vue
index b0517110..44d4ffb2 100644
--- a/src/components/Sprints/AddSprint.vue
+++ b/src/components/Sprints/AddSprint.vue
@@ -1,7 +1,7 @@
 <template>
   <div>
     <div>
-      <button type="button" class="btn btn-info btn-md falko-button" id="addButton" data-toggle="modal" data-target="#addSprintModal">
+      <button type="button" class="btn btn-info btn-md falko-button" id="addSprintButton" data-toggle="modal" data-target="#addSprintModal">
         Add a Sprint
       </button>
     </div>
@@ -107,7 +107,7 @@ export default {
 </script>
 
 <style scoped>
-#addButton {
+#addSprintButton {
   margin-top: 2em;
 }
 

From ddafdd60b6eb98e0cb97606bdef84839cf9852e6 Mon Sep 17 00:00:00 2001
From: MatheusRich <matheusrichardt@gmail.com>
Date: Mon, 19 Feb 2018 21:42:49 -0300
Subject: [PATCH 6/8] Adding dynamic ids

---
 src/components/Releases/ReleaseCard.vue | 2 +-
 src/components/Releases/Releases.vue    | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/components/Releases/ReleaseCard.vue b/src/components/Releases/ReleaseCard.vue
index 3bc5a13c..06cf8ed3 100644
--- a/src/components/Releases/ReleaseCard.vue
+++ b/src/components/Releases/ReleaseCard.vue
@@ -1,6 +1,6 @@
 <template>
   <div>
-    <div class="row release-card align-content-center buttons pointer-cursor" v-bind:id="release[1]" v-on:click="select($event)">
+    <div class="row release-card align-content-center buttons pointer-cursor" v-bind:id="`release${amountOfReleases - release[1]}`" v-on:click="select($event)">
       <div class="col-5" align="center">
         <div class="number-circle align-self-center">
           <p>R{{this.amountOfReleases - this.release[1]}}</p>
diff --git a/src/components/Releases/Releases.vue b/src/components/Releases/Releases.vue
index 8d368622..5421cf47 100644
--- a/src/components/Releases/Releases.vue
+++ b/src/components/Releases/Releases.vue
@@ -17,15 +17,15 @@
         <div class="row justify-content-between" id="releaseHeader">
           <div class="col-5 no-padding" align="center">
             <div class="row">
-              <h1>{{this.releases[this.releaseIndex].name}}</h1>
+              <h1 :id="`release${releaseIndex}Name`">{{this.releases[this.releaseIndex].name}}</h1>
             </div>
-            <div class="row text-muted">
+            <div class="row text-muted" :id="`release${releaseIndex}Description`">
               <p>{{this.releases[this.releaseIndex].description}}</p>
             </div>
           </div>
           <div class="col-5 no-padding">
             <div class="row">
-              <h5>
+              <h5 id="releasePeriod">
                 {{dateConvert(this.releases[this.releaseIndex].initial_date)}}
                 |
                 {{dateConvert(this.releases[this.releaseIndex].final_date)}}

From 391dea727b433e5fda9900227b8105928681841e Mon Sep 17 00:00:00 2001
From: MatheusRich <matheusrichardt@gmail.com>
Date: Mon, 19 Feb 2018 21:46:43 -0300
Subject: [PATCH 7/8] Adding grey class

---
 src/App.vue                             | 4 ++++
 src/components/Releases/AddRelease.vue  | 5 +----
 src/components/Releases/EditRelease.vue | 2 +-
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/src/App.vue b/src/App.vue
index 21fcc1b7..f5870700 100644
--- a/src/App.vue
+++ b/src/App.vue
@@ -208,6 +208,10 @@ label {
   min-height: 7em;
 }
 
+.grey {
+  color: #777;
+}
+
 ::-webkit-scrollbar {
     width: 0.5em;
 }
diff --git a/src/components/Releases/AddRelease.vue b/src/components/Releases/AddRelease.vue
index 905e140a..e8dc3df2 100644
--- a/src/components/Releases/AddRelease.vue
+++ b/src/components/Releases/AddRelease.vue
@@ -25,7 +25,7 @@
           <div class="row modal-body align-content-end">
             <div class="col-6">
               <label>Name</label>
-              <input type="text" v-model="name" id="releaseName" placeholder="Release Name" name="name" v-validate="'required'">
+              <input type="text" v-model="name" class="grey" placeholder="Release Name" name="name" v-validate="'required'">
                 <p class="text-danger" v-if="errors.has('name')">{{ errors.first('name') }}</p>
               </input>
               <br>
@@ -111,9 +111,6 @@ input {
   margin-bottom: 0.5em;
 }
 
-#releaseName {
-  color: #777;
-}
 #addReleaseButton {
   width: 100%;
   border-radius: 0;
diff --git a/src/components/Releases/EditRelease.vue b/src/components/Releases/EditRelease.vue
index 76d3f42c..da25c069 100644
--- a/src/components/Releases/EditRelease.vue
+++ b/src/components/Releases/EditRelease.vue
@@ -16,7 +16,7 @@
           <div class="row modal-body">
             <div class="col-6">
               <label>Name</label>
-              <input type="text" v-model="name" id="releaseName" placeholder="Release Name" name="name" v-validate="'required'">
+              <input type="text" v-model="name" class="grey" placeholder="Release Name" name="name" v-validate="'required'">
                 <p class="text-danger" v-if="errors.has('name')">{{ errors.first('name') }}</p>
               </input>
               <br>

From 6751ed5440e38fe29f4dd7ff4a04e7dd2465d07a Mon Sep 17 00:00:00 2001
From: MatheusRich <matheusrichardt@gmail.com>
Date: Mon, 19 Feb 2018 21:47:03 -0300
Subject: [PATCH 8/8] Adding releses_spec

---
 cypress/fixtures/releases.json       | 20 +++++++++++++
 cypress/integration/releases_spec.js | 43 ++++++++++++++++++++++++++++
 2 files changed, 63 insertions(+)
 create mode 100644 cypress/fixtures/releases.json
 create mode 100644 cypress/integration/releases_spec.js

diff --git a/cypress/fixtures/releases.json b/cypress/fixtures/releases.json
new file mode 100644
index 00000000..daa38b70
--- /dev/null
+++ b/cypress/fixtures/releases.json
@@ -0,0 +1,20 @@
+[
+  {
+    "id": 2,
+    "name": "R2",
+    "description": "Second Release",
+    "amount_of_sprints": 1,
+    "project_id": 1,
+    "initial_date": "2017-01-01",
+    "final_date": "2017-12-01"
+  },
+  {
+    "id": 1,
+    "name": "R1",
+    "description": "First Rrelease",
+    "amount_of_sprints": 2,
+    "project_id": 1,
+    "initial_date": "2016-01-01",
+    "final_date": "2016-12-01"
+  }
+]
\ No newline at end of file
diff --git a/cypress/integration/releases_spec.js b/cypress/integration/releases_spec.js
new file mode 100644
index 00000000..e8723b80
--- /dev/null
+++ b/cypress/integration/releases_spec.js
@@ -0,0 +1,43 @@
+describe('Releases tests', () => {
+  beforeEach(() => {
+    cy.server()
+  
+    cy.route({
+      method: 'GET',
+      url: '/users\/1/projects',
+      status: 200,
+      response: 'fixture:projects.json'
+    }).as('getProjects');
+
+    cy.route({
+      method: 'GET',
+      url: '/projects\/2',
+      status: 200,
+      response: 'fixture:project.json'
+    }).as('getProject')
+
+        cy.route({
+      method: 'GET',
+      url: '/projects\/2/releases',
+      status: 200,
+      response: 'fixture:releases.json'
+    }).as('getReleases');
+
+    cy.login();
+
+  })
+    
+  
+  it('should get releases', () => {
+    cy.get('#projectCard2').click();
+    
+    cy.get('#releases').click();
+    
+    cy.get('div#release2').contains('R2');
+    cy.get('h1#release0Name').should('contain', 'R2');
+    cy.get('#release0Description').should('contain', 'Second Release');
+    cy.get('#releasePeriod');
+    cy.get('#addReleaseButton').should('contain', 'Add Release');
+    cy.get('#addSprintButton').should('contain', 'Add a Sprint');
+  })
+})
\ No newline at end of file