Merge pull request

Adicionando testes de unidade

package.json

@@ -77,6 +77,7 @@
     "opn": "^5.1.0",
     "optimize-css-assets-webpack-plugin": "^2.0.0",
     "ora": "^1.2.0",
+    "phantomjs-polyfill-object-assign": "0.0.2",
     "phantomjs-prebuilt": "^2.1.15",
     "rimraf": "^2.6.0",
     "selenium-server": "^3.0.1",
@@ -88,6 +89,7 @@
     "vue-loader": "^13.0.4",
     "vue-style-loader": "^3.0.1",
     "vue-template-compiler": "^2.4.2",
+    "vue-test-utils": "^1.0.0-beta.6",
     "webpack": "^2.6.1",
     "webpack-bundle-analyzer": "^2.2.1",
     "webpack-dev-middleware": "^1.10.0",

src/components/Authentication/Login.vue

@@ -45,40 +45,17 @@ export default {
     };
   },
 
-  computed: {
-    ...mapState({
-      token: state => state.auth.token,
-      userId: state => state.auth.userId,
-    }),
-  },
-
   methods: {
     login() {
       const thisOne = this;
-      this.$store.dispatch('login', { email: this.email, password: this.password })
-        .then(() => {
-          this.$router.push({ name: 'Projects' });
-        })
-        .catch((err) => {
-          thisOne.errors.add('wrong-credentials', 'Wrong Credentials');
-          console.log(err); // It goes here!
-        });
-    },
-
-    isGitHubAuthenticated() {
-      const headers = { Authorization: this.token };
 
-      HTTP.get(`users/${this.userId}`, { headers })
-        .then((response) => {
-          if (response.data.access_token != null) {
-            localStorage.setItem('is_github_authenticated', 'true');
-          } else {
-            localStorage.setItem('is_github_authenticated', 'false');
-          }
-        })
-        .catch((err) => {
-          console.log(err); // It goes here!
-        });
+      this.$store.dispatch('login', { email: this.email, password: this.password })
+      .then((res) => {
+        this.$router.push({ name: 'Projects' });
+      })
+      .catch((err) => {
+        thisOne.errors.add('wrong-credentials', 'Wrong Credentials');
+      });
     },
   },
 };

src/components/GitHub/GitHubCallBack.vue

@@ -20,14 +20,10 @@ export default {
   mounted() {
     const code = window.location.search.split('=')[1];
     const headers = { Authorization: this.token };
-    HTTP.post('request_github_token', {
-      code,
-      id: this.userId,
-    }, { headers })
-      .then((response) => {
-        this.github_token = response.data.access_token;
+    this.$store.dispatch('linkGithub', { code, headers, userId: this.userId })
+      .then(() => {
         this.$router.push({ name: 'UserProfile' });
-      })
+      }) 
       .catch((e) => {
         this.errors.push(e);
       });

src/components/Projects/DeleteProject.vue

@@ -36,7 +36,6 @@ export default {
   computed: {
     ...mapState({
       token: state => state.auth.token,
-      userId: state => state.auth.userId,
     }),
   },
   methods: {

src/components/Projects/Projects.vue

@@ -93,7 +93,6 @@ export default {
 
     refreshProjects() {
       this.getProjects();
-      this.gitHubAuthenticated();
     },
 
     isProjectsEmpty() {

src/components/Releases/AddRelease.vue

@@ -77,7 +77,7 @@ export default {
           this.description = '';
           this.initialDate = '';
           this.finalDate = '';
-          EventBus.$emit('added-release', 1);
+          this.$emit('added');
         })
         .catch((e) => {
           this.errors.push(e);

src/components/Releases/Release.vue

@@ -8,9 +8,9 @@
           </li>
           <li class="list-inline-item vertical-center small-float-right">
             <h5 class="float-left">
-              {{dateConvert(release.initial_date)}}
+              {{converted_initial_date}}
               |
-              {{dateConvert(release.final_date)}}
+              {{converted_final_date}}
             </h5>
           </li>
           <p class="text-justify text-muted">
@@ -55,6 +55,8 @@ export default {
   data() {
     return {
       release: {},
+      converted_initial_date: "",
+      converted_final_date: ""
     };
   },
 
@@ -73,7 +75,8 @@ export default {
       HTTP.get(`releases/${this.$route.params.id}`, { headers })
         .then((response) => {
           this.release = response.data;
-
+          this.converted_initial_date = this.dateConvert(this.release.initial_date);
+          this.converted_final_date = this.dateConvert(this.release.final_date)
           this.setReleaseDates();
         })
         .catch((e) => {
@@ -91,11 +94,7 @@ export default {
     },
   },
 
-  ready() {
-    this.dateConvert();
-  },
-
-  mounted() {
+  created() {
     this.getRelease();
   },
 };

src/components/Releases/Releases.vue

@@ -5,7 +5,7 @@
     </div>
     <div class="row">
       <div class="col">
-        <add-release></add-release>
+        <add-release v-on:added="getReleases()"></add-release>
       </div>
     </div>
 
@@ -87,7 +87,7 @@ export default {
   methods: {
     getReleases() {
       const headers = { Authorization: this.token };
-      console.log(headers);
+
       HTTP.get(`projects/${this.$route.params.id}/releases`, { headers })
         .then((response) => {
           this.releases = response.data;
@@ -102,17 +102,9 @@ export default {
 
   },
 
-  ready() {
-    this.dateConvert();
-  },
-
   mounted() {
     this.getReleases();
   },
-
-  updated() {
-    this.getReleases();
-  },
 };
 </script>
 

src/components/Retrospective/DeleteRetrospective.vue

@@ -49,12 +49,12 @@ export default {
   methods: {
     async deleteRetrospective() {
       const headers = { Authorization: this.token };
-
       try {
         const response = await HTTP.get(`/retrospectives/${this.$route.params.id}`, { headers });
         const id = response.data.sprint_id;
 
         await HTTP.delete(`/retrospectives/${this.$route.params.id}`, { headers });
+
         this.$router.push({ path: `/Sprints/${id}` });
       } catch (err) {
         console.log(err);

src/components/Sprints/DeleteSprint.vue

@@ -1,9 +1,9 @@
 <template>
   <div class="delsprint">
-    <button type="button" class="btn btn-info btn-md falko-button-danger" id="deletebutton" data-toggle="modal" data-target="#delSprintModal">
+    <button type="button" class="btn btn-info btn-md falko-button-danger" id="deletebutton" data-toggle="modal" data-target="#deleteSprintModal">
       Delete
     </button>
-    <div class="modal fade" id ="delSprintModal" role="dialog">
+    <div class="modal fade" id ="deleteSprintModal" role="dialog">
       <div class="modal-dialog">
         <div class="modal-content">
           <div class="modal-header">
@@ -18,7 +18,7 @@
             <p><label> Are you sure?</label></p>
           </div>
           <div class="modal-footer">
-            <button v-on:click="delSprint" type="button" class="btn btn-info btn-md falko-button" data-dismiss="modal" >Yes</button>
+            <button v-on:click="deleteSprint" type="button" class="btn btn-info btn-md falko-button" data-dismiss="modal" >Yes</button>
             <button type="button" class="btn btn-info btn-md falko-button-grey" data-dismiss="modal">No</button>
           </div>
         </div>
@@ -39,7 +39,7 @@ export default {
     }),
   },
   methods: {
-    async delSprint() {
+    async deleteSprint() {
       const headers = { Authorization: this.token };
 
       try {

src/components/Sprints/Sprint.vue

@@ -151,10 +151,8 @@ export default{
     },
 
     getRevision() {
-      const token = localStorage.getItem('token');
-      const tokenSimple = token.replace(/"/, '');
-      const tokenSimple2 = tokenSimple.replace(/"/, '');
-      const headers = { Authorization: tokenSimple2 };
+      const headers = { Authorization: this.token };
+
       HTTP.get(`sprints/${this.$route.params.id}/revisions`, { headers })
       .then((response) => {
         this.sprintRevision = response.data;

src/services/auth/index.js

@@ -2,13 +2,15 @@ import { HTTP } from '../../http-common';
 
 const LOGIN = 'LOGIN';
 const LOGOUT = 'LOGOUT';
-
+const LINK_GITHUB = 'LINK_GITHUB';
+const UNLINK_GITHUB = 'UNLINK_GITHUB';
 const auth = {
   state() {
     return {
       authenticated: false,
       token: null,
       userId: null,
+      githubAuthenticated: false,
     };
   },
   mutations: {
@@ -24,6 +26,14 @@ const auth = {
       localState.userId = null;
       localState.authenticated = false;
     },
+    [LINK_GITHUB](state) {
+      const localState = state;
+      localState.githubAuthenticated = true;
+    },
+    [UNLINK_GITHUB](state) {
+      const localState = state;
+      localState.githubAuthenticated = false;
+    },
   },
   actions: {
     login({ commit }, credentials) {
@@ -40,6 +50,16 @@ const auth = {
     logout({ commit }) {
       commit(LOGOUT);
     },
+
+    linkGithub({ commit }, credentials) {
+      return HTTP.post('request_github_token', {
+        code: credentials.code,
+        id: credentials.userId,
+      }, { headers: credentials.headers })
+        .then(() => {
+          commit(LINK_GITHUB);
+        });
+    },
   },
 };
 

test/unit/karma.conf.js

@@ -10,7 +10,9 @@ module.exports = function (config) {
     // available frameworks: https://npmjs.org/browse/keyword/karma-adapter
     frameworks: ['mocha', 'sinon-chai'],
     reporters: ['spec', 'coverage'],
-    files: ['../../node_modules/es6-promise/dist/es6-promise.auto.js', './index.js'],
+    files: ['../../node_modules/es6-promise/dist/es6-promise.auto.js',
+            '../../node_modules/phantomjs-polyfill-object-assign/object-assign-polyfill.js',
+            './index.js'],
     // we will pass the entry file to webpack for bundling.
     preprocessors: {
       './index.js': ['webpack', 'sourcemap'],