Chore deploy to heroku #36

Billmike · 2018-02-10T18:09:36Z

What does this PR do?

Setup app for final deployment on heroku

Description of tasks to be completed?

Stabilize application for deployment on heroku

How should this be manually tested?

Visit the application on heroku here

Setup API for user signup

Add API for user sign in

Ft add recipe api-152046467

Setup API for the modification of a recipe by an authenticated user - #152046516

…plication

Add API that allows logged-in users to retreive all recipes in the application - #152352790

Add API for the posting of review by an authenticated user - #152046736

Add API enpoint for the deleting of recipes by an authenticated user - #152046612

Add API that enables users get their favorite recipes - #152046694

Ft api sort votes 152046874

Add API for sending notifications to users

Add dot.env file to application so as to protect important tokens

- redesign user-interface of the application - implement toastr to alert user of actions - fix bug in delete recipe functionality

- implement review for recipe - add emoji to application - install react-emoji-render - setup 404 page - handle individual errors

- write tests for actions and action creators - write tests for reducers - fix bug in add recipe feature - refactor dashboard component - setup jest - install expect package - install moxios - install redux-mock-store

- add more tests for reducers - refactor reducers

- modify files to prepare for final deployment

- fix bug in signup and signin component - modify code to use jsdoc

- remove console.log statement from component

- refactor travis file

houndci-bot · 2018-02-10T18:10:09Z

server/__test__/seed/recipeSeed.js

+  instructions: '[Array, of, instructions]',
+}];
+
+export default recipes;


'export' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-10T18:10:09Z

server/__test__/seed/recipeSeed.js

@@ -0,0 +1,19 @@
+const recipes = [{


'const' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).

houndci-bot · 2018-02-10T18:10:09Z

server/__test__/seed/reviewSeed.js

+  content: 'Amazing recipe you got there man',
+};
+
+export default reviews;


'export' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-10T18:10:09Z

server/__test__/seed/reviewSeed.js

@@ -0,0 +1,6 @@
+const reviews = {


'const' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).

houndci-bot · 2018-02-10T18:10:09Z

server/__test__/recipes.spec.js

+            expect(res.status).to.equal(400);
+            expect(res.body.message).to.equal('Name is required.');
+            done();
+          });


Unrecoverable syntax error. (18% scanned).

houndci-bot · 2018-02-10T18:10:10Z

server/__test__/recipes.spec.js

+    );
+    it('Should successfully create a recipe as a logged in user', (done) => {
+      const testUser = { ...users[0] };
+      request.post(`${recipesApi}?token=${testUser.tokens[0].token}`)


'template literal syntax' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-10T18:10:10Z

server/__test__/recipes.spec.js

+      }
+    );
+    it('Should successfully create a recipe as a logged in user', (done) => {
+      const testUser = { ...users[0] };


'const' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).
Expected '}' to match '{' from line 30 and instead saw '...'.
Missing semicolon.
Expected an assignment or function call and instead saw an expression.
Expected ')' and instead saw ';'.

houndci-bot · 2018-02-10T18:10:10Z

server/__test__/recipes.spec.js

+          });
+      }
+    );
+    it('Should successfully create a recipe as a logged in user', (done) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-10T18:10:10Z

server/__test__/recipes.spec.js

+          .set('Content-Type', 'application/json')
+          .type('form')
+          .send(recipes[0])
+          .end((err, res) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-10T18:10:10Z

server/__test__/recipes.spec.js

+  describe('#Test Creating a new Recipe', () => {
+    it(
+      'Should return a 403 when a user tries to create a recipe without a token',
+      (done) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

BolajiOlajide

Hey! I feel you should gitignore your public folder.

Why?

It's not really a good idea to push your bundled files to version control, it should be a. user generatede thing.

Also, files like .DS_Store should be ignored.

BolajiOlajide · 2018-02-10T18:42:47Z

.babelrc

+    "plugins": [
+        "transform-react-jsx",
+        "transform-class-properties",
+        "transform-object-rest-spread"
    ]
 }


Add a new line here

Thanks for the feedback. Implemented it.

BolajiOlajide · 2018-02-10T18:44:49Z

.codeclimate.yml

+ratings:
+  paths:
+  - "server/**/*.js"
+  - "client/**/*.js"


I believe you can target both .js and .jsx files with this line client/**/*.{js,jsx}

Thanks for the feedback. Implemented it.

BolajiOlajide · 2018-02-10T18:45:07Z

.codeclimate.yml

+  - "client/src/__tests__"
+  - "server/migrations"
+  - "server/models"
+  - "server/seeders"


New line needed here

Thanks for the feedback. Implemented it.

BolajiOlajide · 2018-02-10T19:09:40Z

.travis.yml

+env:
+  global:
+    - NODE_ENV=test
+    - CC_TEST_REPORTER_ID=1944fd38c05bd77f1f23b769f5e0753618ad880b9317d37b6b8c985d37587594


Not sure if this is safe to push to version control. If it isn't you should look into encrypting it.

Thanks for the feedback. Implemented it.

BolajiOlajide · 2018-02-10T19:09:56Z

.travis.yml

+
+after_success:
+  - ./cc-test-reporter after-build --exit-code $TRAVIS_TEST_RESULT
+


Add a new line here.

Thanks for the feedback. Implemented it.

BolajiOlajide · 2018-02-11T02:30:01Z

client/src/components/SignupForm.jsx

+    }
+  }
+
+  onUsernameChange(event) {


You can make use of one onChange method for all inputs

BolajiOlajide · 2018-02-11T02:31:19Z

client/src/index.jsx

@@ -0,0 +1,23 @@
+import React from 'react';
+import ReactDOM from 'react-dom';


Instead of importing the whole ReactDOM package, why not import what you need.
import { render } from 'react-dom';

BolajiOlajide · 2018-02-11T02:31:51Z

client/src/index.jsx

+
+const store = configureStore();
+
+if (localStorage.authToken) {


Read the article about localStorage

BolajiOlajide · 2018-02-11T02:32:13Z

client/src/index.jsx

+.render(
+  <Provider store={store}>
+    <App />
+  </Provider>, document.getElementById('root'));


Nitpick

New line needed here.

BolajiOlajide · 2018-02-11T02:34:03Z

database.yml.travis

+    adapter: postgres
+    database: more_recipes_test
+    username: bill
+    password: rocketmail


Nitpick

New line needed here.

houndci-bot · 2018-02-11T08:59:13Z

server/build/service-worker.js

@@ -0,0 +1 @@
+"use strict";function setOfCachedUrls(e){return e.keys().then(function(e){return e.map(function(e){return e.url})}).then(function(e){return new Set(e)})}var precacheConfig=[["/index.html","eb657bf261c4ab789cd793da99d3f7fa"],["/static/css/main.7cf33c00.css","46f792d383b5e2f870454414ed0bc7d2"],["/static/js/main.7065254e.js","2b4f759bc791fda57a1e267422dd9a01"],["/static/media/cookies.1a59c333.jpeg","1a59c33306fbf49f9009b4c1b093a0ce"],["/static/media/dessert.0760c662.jpeg","0760c662e9529adcee1fef6551de5a7c"],["/static/media/fontawesome-webfont.674f50d2.eot","674f50d287a8c48dc19ba404d20fe713"],["/static/media/fontawesome-webfont.912ec66d.svg","912ec66d7572ff821749319396470bde"],["/static/media/fontawesome-webfont.af7ae505.woff2","af7ae505a9eed503f8b8e6982036873e"],["/static/media/fontawesome-webfont.b06871f2.ttf","b06871f281fee6b241d60582ae9369b9"],["/static/media/fontawesome-webfont.fee66e71.woff","fee66e712a8a08eef5805a46892932ad"],["/static/media/pancakes.30892fc8.jpeg","30892fc8000a03c65d8552729c2d28ad"],["/static/media/pizzza.cba2d80e.jpg","cba2d80ed4c200dc6f80c88dbb6b5476"]],cacheName="sw-precache-v3-sw-precache-webpack-plugin-"+(self.registration?self.registration.scope:""),ignoreUrlParametersMatching=[/^utm_/],addDirectoryIndex=function(e,t){var n=new URL(e);return"/"===n.pathname.slice(-1)&&(n.pathname+=t),n.toString()},cleanResponse=function(e){if(!e.redirected)return Promise.resolve(e);return("body"in e?Promise.resolve(e.body):e.blob()).then(function(t){return new Response(t,{headers:e.headers,status:e.status,statusText:e.statusText})})},createCacheKey=function(e,t,n,a){var r=new URL(e);return a&&r.pathname.match(a)||(r.search+=(r.search?"&":"")+encodeURIComponent(t)+"="+encodeURIComponent(n)),r.toString()},isPathWhitelisted=function(e,t){if(0===e.length)return!0;var n=new URL(t).pathname;return e.some(function(e){return n.match(e)})},stripIgnoredUrlParameters=function(e,t){var n=new URL(e);return n.hash="",n.search=n.search.slice(1).split("&").map(function(e){return e.split("=")}).filter(function(e){return t.every(function(t){return!t.test(e[0])})}).map(function(e){return e.join("=")}).join("&"),n.toString()},hashParamName="_sw-precache",urlsToCacheKeys=new Map(precacheConfig.map(function(e){var t=e[0],n=e[1],a=new URL(t,self.location),r=createCacheKey(a,hashParamName,n,/\.\w{8}\./);return[a.toString(),r]}));self.addEventListener("install",function(e){e.waitUntil(caches.open(cacheName).then(function(e){return setOfCachedUrls(e).then(function(t){return Promise.all(Array.from(urlsToCacheKeys.values()).map(function(n){if(!t.has(n)){var a=new Request(n,{credentials:"same-origin"});return fetch(a).then(function(t){if(!t.ok)throw new Error("Request for "+n+" returned a response with status "+t.status);return cleanResponse(t).then(function(t){return e.put(n,t)})})}}))})}).then(function(){return self.skipWaiting()}))}),self.addEventListener("activate",function(e){var t=new Set(urlsToCacheKeys.values());e.waitUntil(caches.open(cacheName).then(function(e){return e.keys().then(function(n){return Promise.all(n.map(function(n){if(!t.has(n.url))return e.delete(n)}))})}).then(function(){return self.clients.claim()}))}),self.addEventListener("fetch",function(e){if("GET"===e.request.method){var t,n=stripIgnoredUrlParameters(e.request.url,ignoreUrlParametersMatching),a="index.html";(t=urlsToCacheKeys.has(n))||(n=addDirectoryIndex(n,a),t=urlsToCacheKeys.has(n));var r="/index.html";!t&&"navigate"===e.request.mode&&isPathWhitelisted(["^(?!\\/__).*"],e.request.url)&&(n=new URL(r,self.location).toString(),t=urlsToCacheKeys.has(n)),t&&e.respondWith(caches.open(cacheName).then(function(e){return e.match(urlsToCacheKeys.get(n)).then(function(e){if(e)return e;throw Error("The cached response that was expected is missing.")})}).catch(function(t){return console.warn('Couldn\'t serve response for "%s" from cache: %O',e.request.url,t),fetch(e.request)}))}});


Use the function form of "use strict".
Missing semicolon.
Expected an assignment or function call and instead saw an expression.
'Set' is not defined.
'self' is not defined.
'URL' is not defined.
Too many errors. (100% scanned).

houndci-bot · 2018-02-11T08:59:13Z

server/bin/www.js

@@ -1,7 +1,7 @@
 import http from 'http';
 import app from '../app';

-const port = parseInt(process.env.PORT, 10) || 8080;
+const port = parseInt(process.env.PORT, 10) || 8000;


'const' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).

houndci-bot · 2018-02-11T08:59:13Z

server/app.js

+  res.sendFile(path.join(appPath, '/index.html'));
+});
+
+app.get('*', (req, res) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-11T08:59:14Z

server/app.js

-app.get('*', (req, res) => res.status(200).send({
-  message: 'Welcome to the beginning of aawesomeness',
-}));
+app.get('*', (req, res) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-11T08:59:14Z

server/app.js

+app.use(bodyParser.json());
+app.use(express.static(appPath));
+app.use(express.static('public'));
+app.use((req, res, next) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-11T08:59:14Z

server/__test__/vote.spec.js

+    );
+    it(
+      'Should successfully downvote a recipe by an authenticated user',
+      (done) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-11T08:59:14Z

server/__test__/vote.spec.js

+          .set('Cconnection', 'keep alive')
+          .set('Content-Type', 'application/json')
+          .type('form')
+          .end((err, res) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-11T08:59:15Z

server/__test__/vote.spec.js

+    it(
+      'Should successfully upvote a recipe by an authenticated user',
+      (done) => {
+        request.post(`${recipesApi}/${recipe[0]


'template literal syntax' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-11T08:59:15Z

server/__test__/vote.spec.js

+    );
+    it(
+      'Should successfully upvote a recipe by an authenticated user',
+      (done) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-11T08:59:15Z

server/__test__/vote.spec.js

+          .set('Connection', 'keep alive')
+          .set('Content-Type', 'application/json')
+          .type('form')
+          .end((err, res) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-11T08:59:15Z

server/build/service-worker.js

@@ -0,0 +1 @@
+"use strict";function setOfCachedUrls(e){return e.keys().then(function(e){return e.map(function(e){return e.url})}).then(function(e){return new Set(e)})}var precacheConfig=[["/index.html","eb657bf261c4ab789cd793da99d3f7fa"],["/static/css/main.7cf33c00.css","46f792d383b5e2f870454414ed0bc7d2"],["/static/js/main.7065254e.js","2b4f759bc791fda57a1e267422dd9a01"],["/static/media/cookies.1a59c333.jpeg","1a59c33306fbf49f9009b4c1b093a0ce"],["/static/media/dessert.0760c662.jpeg","0760c662e9529adcee1fef6551de5a7c"],["/static/media/fontawesome-webfont.674f50d2.eot","674f50d287a8c48dc19ba404d20fe713"],["/static/media/fontawesome-webfont.912ec66d.svg","912ec66d7572ff821749319396470bde"],["/static/media/fontawesome-webfont.af7ae505.woff2","af7ae505a9eed503f8b8e6982036873e"],["/static/media/fontawesome-webfont.b06871f2.ttf","b06871f281fee6b241d60582ae9369b9"],["/static/media/fontawesome-webfont.fee66e71.woff","fee66e712a8a08eef5805a46892932ad"],["/static/media/pancakes.30892fc8.jpeg","30892fc8000a03c65d8552729c2d28ad"],["/static/media/pizzza.cba2d80e.jpg","cba2d80ed4c200dc6f80c88dbb6b5476"]],cacheName="sw-precache-v3-sw-precache-webpack-plugin-"+(self.registration?self.registration.scope:""),ignoreUrlParametersMatching=[/^utm_/],addDirectoryIndex=function(e,t){var n=new URL(e);return"/"===n.pathname.slice(-1)&&(n.pathname+=t),n.toString()},cleanResponse=function(e){if(!e.redirected)return Promise.resolve(e);return("body"in e?Promise.resolve(e.body):e.blob()).then(function(t){return new Response(t,{headers:e.headers,status:e.status,statusText:e.statusText})})},createCacheKey=function(e,t,n,a){var r=new URL(e);return a&&r.pathname.match(a)||(r.search+=(r.search?"&":"")+encodeURIComponent(t)+"="+encodeURIComponent(n)),r.toString()},isPathWhitelisted=function(e,t){if(0===e.length)return!0;var n=new URL(t).pathname;return e.some(function(e){return n.match(e)})},stripIgnoredUrlParameters=function(e,t){var n=new URL(e);return n.hash="",n.search=n.search.slice(1).split("&").map(function(e){return e.split("=")}).filter(function(e){return t.every(function(t){return!t.test(e[0])})}).map(function(e){return e.join("=")}).join("&"),n.toString()},hashParamName="_sw-precache",urlsToCacheKeys=new Map(precacheConfig.map(function(e){var t=e[0],n=e[1],a=new URL(t,self.location),r=createCacheKey(a,hashParamName,n,/\.\w{8}\./);return[a.toString(),r]}));self.addEventListener("install",function(e){e.waitUntil(caches.open(cacheName).then(function(e){return setOfCachedUrls(e).then(function(t){return Promise.all(Array.from(urlsToCacheKeys.values()).map(function(n){if(!t.has(n)){var a=new Request(n,{credentials:"same-origin"});return fetch(a).then(function(t){if(!t.ok)throw new Error("Request for "+n+" returned a response with status "+t.status);return cleanResponse(t).then(function(t){return e.put(n,t)})})}}))})}).then(function(){return self.skipWaiting()}))}),self.addEventListener("activate",function(e){var t=new Set(urlsToCacheKeys.values());e.waitUntil(caches.open(cacheName).then(function(e){return e.keys().then(function(n){return Promise.all(n.map(function(n){if(!t.has(n.url))return e.delete(n)}))})}).then(function(){return self.clients.claim()}))}),self.addEventListener("fetch",function(e){if("GET"===e.request.method){var t,n=stripIgnoredUrlParameters(e.request.url,ignoreUrlParametersMatching),a="index.html";(t=urlsToCacheKeys.has(n))||(n=addDirectoryIndex(n,a),t=urlsToCacheKeys.has(n));var r="/index.html";!t&&"navigate"===e.request.mode&&isPathWhitelisted(["^(?!\\/__).*"],e.request.url)&&(n=new URL(r,self.location).toString(),t=urlsToCacheKeys.has(n)),t&&e.respondWith(caches.open(cacheName).then(function(e){return e.match(urlsToCacheKeys.get(n)).then(function(e){if(e)return e;throw Error("The cached response that was expected is missing.")})}).catch(function(t){return console.warn('Couldn\'t serve response for "%s" from cache: %O',e.request.url,t),fetch(e.request)}))}});


Use the function form of "use strict".
Missing semicolon.
Expected an assignment or function call and instead saw an expression.
'Set' is not defined.
'self' is not defined.
'URL' is not defined.
Too many errors. (100% scanned).

houndci-bot · 2018-02-11T08:59:15Z

server/bin/www.js

@@ -1,7 +1,7 @@
 import http from 'http';
 import app from '../app';

-const port = parseInt(process.env.PORT, 10) || 8080;
+const port = parseInt(process.env.PORT, 10) || 8000;


'const' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).

houndci-bot · 2018-02-11T08:59:16Z

server/app.js

+  res.sendFile(path.join(appPath, '/index.html'));
+});
+
+app.get('*', (req, res) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-11T08:59:16Z

server/app.js

-app.get('*', (req, res) => res.status(200).send({
-  message: 'Welcome to the beginning of aawesomeness',
-}));
+app.get('*', (req, res) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-11T08:59:16Z

server/app.js

+app.use(bodyParser.json());
+app.use(express.static(appPath));
+app.use(express.static('public'));
+app.use((req, res, next) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-11T08:59:17Z

server/__test__/vote.spec.js

+    );
+    it(
+      'Should successfully downvote a recipe by an authenticated user',
+      (done) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-11T08:59:17Z

server/__test__/vote.spec.js

+          .set('Cconnection', 'keep alive')
+          .set('Content-Type', 'application/json')
+          .type('form')
+          .end((err, res) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-11T08:59:17Z

server/__test__/vote.spec.js

+    it(
+      'Should successfully upvote a recipe by an authenticated user',
+      (done) => {
+        request.post(`${recipesApi}/${recipe[0]


'template literal syntax' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-11T08:59:17Z

server/__test__/vote.spec.js

+    );
+    it(
+      'Should successfully upvote a recipe by an authenticated user',
+      (done) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-11T08:59:17Z

server/__test__/vote.spec.js

+          .set('Connection', 'keep alive')
+          .set('Content-Type', 'application/json')
+          .type('form')
+          .end((err, res) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-12T04:21:37Z

server/__test__/userfavorites.spec.js

+          expect(res.status).to.equal(201);
+          expect(res.body.message).to.equal('User has no favorites.');
+          done();
+        })


Missing semicolon.
Unrecoverable syntax error. (73% scanned).

houndci-bot · 2018-02-12T04:21:37Z

server/__test__/userfavorites.spec.js

+        .set('Connection', 'keep alive')
+        .set('Content-Type', 'application/json')
+        .type('form')
+        .end((err, res) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-12T04:21:37Z

server/__test__/userfavorites.spec.js

+    });
+    it('Should return an empty array of favorites for a logged in user without a favorite', (done) => {
+      const testUser = { ...users[2] };
+      request.get(`${favoriteApi}/${testUser.id}/favorites?token=${testUser.tokens[0].token}`)


'template literal syntax' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-12T04:21:37Z

server/__test__/userfavorites.spec.js

+        });
+    });
+    it('Should return an empty array of favorites for a logged in user without a favorite', (done) => {
+      const testUser = { ...users[2] };


'const' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).
Expected '}' to match '{' from line 75 and instead saw '...'.
Missing semicolon.
Expected an assignment or function call and instead saw an expression.
Expected ')' and instead saw ';'.

houndci-bot · 2018-02-12T04:21:37Z

server/__test__/userfavorites.spec.js

+          done();
+        });
+    });
+    it('Should return an empty array of favorites for a logged in user without a favorite', (done) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-12T04:21:38Z

server/__test__/userfavorites.spec.js

+          .set('Connection', 'keep alive')
+          .set('Content-Type', 'application/json')
+          .type('form')
+          .end((err, res) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-12T04:21:38Z

server/__test__/userfavorites.spec.js

+      'Should successfuly favorite a recipe by an authenticated user',
+      (done) => {
+        const testUser = { ...users[1] };
+        request.post(`${recipesApi}/${recipe[0].id}/favorites?token=${testUser.tokens[0].token}`)


'template literal syntax' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-12T04:21:38Z

server/__test__/userfavorites.spec.js

+    it(
+      'Should successfuly favorite a recipe by an authenticated user',
+      (done) => {
+        const testUser = { ...users[1] };


'const' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).
Expected '}' to match '{' from line 46 and instead saw '...'.
Missing semicolon.
Expected an assignment or function call and instead saw an expression.
Expected ')' and instead saw ';'.

houndci-bot · 2018-02-12T04:21:38Z

server/__test__/userfavorites.spec.js

+    });
+    it(
+      'Should successfuly favorite a recipe by an authenticated user',
+      (done) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

houndci-bot · 2018-02-12T04:21:38Z

server/__test__/userfavorites.spec.js

+        .set('Connection', 'keep alive')
+        .set('Content-Type', 'application/json')
+        .type('form')
+        .end((err, res) => {


'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').

Billmike added 30 commits October 27, 2017 16:45

Update Html pages

d1057f8

Update Html pages

673f37b

Make final change

ee9ec97

Modify README.MD File

c2f9454

Setup API for user signup

ba14e0e

Merge pull request #14 from Billmike/ft-signup-api-152046264

11e0ac0

Setup API for user signup

Add API for user sign in

a27af0e

Merge pull request #15 from Billmike/ft-signin-api-152046338

cd7c7b1

Add API for user sign in

Setup API route for the addition of recipe

e02613f

Setup API route for the addition of Recipes

ff105ae

Merge pull request #17 from Billmike/ft-add-recipe-api

d31762a

Ft add recipe api-152046467

Setup API for the modification of a recipe by an authenticated user

9b6264a

Merge pull request #18 from Billmike/ft-modify-recipes-152046516

80b38dd

Setup API for the modification of a recipe by an authenticated user - #152046516

Add API that allows logged-in users to retreive all recipes in the ap…

7c75cd3

…plication

Add API that allows users to retreive all recipes in the application

b5f2976

Merge pull request #19 from Billmike/ft-get-recipes-api-152352790

bfb65dc

Add API that allows logged-in users to retreive all recipes in the application - #152352790

Add API for the posting of review by an authenticated user

80dc960

Merge pull request #20 from Billmike/ft-reviews-api-152046736

c4ccd00

Add API for the posting of review by an authenticated user - #152046736

Add API enpoint for the deleting of recipes by an authenticated user

c3f88c2

Merge pull request #21 from Billmike/ft-delete-recipe-api-152046612

341191d

Add API enpoint for the deleting of recipes by an authenticated user - #152046612

Add API that enables users get their favorite recipes

421c9a9

Merge pull request #22 from Billmike/ft-api-retrieve-favs-152046694

ae27b13

Add API that enables users get their favorite recipes - #152046694

Add API for the sorting of recipes according to most votes

df23a1a

Add API for the sorting of recipes according to most upvotes

39ee57f

Add API for the sorting of recipes according to the most upvotes

558aef9

Merge pull request #23 from Billmike/ft-api-sort-votes-152046874

8fc1c91

Ft api sort votes 152046874

Add API for sending notifications to users

242ed49

Merge pull request #24 from Billmike/ft-add-email-notification-152063754

8777ee7

Add API for sending notifications to users

Add dot.env file to application so as to protect important tokens

183bc7c

Merge pull request #25 from Billmike/ch-setup-dot-env

a8e4f57

Add dot.env file to application so as to protect important tokens

Billmike added 16 commits February 5, 2018 02:45

chore/redesign-ui

888a3cf

- redesign user-interface of the application - implement toastr to alert user of actions - fix bug in delete recipe functionality

feature:implement review and refactor UI

93040e8

- implement review for recipe - add emoji to application - install react-emoji-render - setup 404 page - handle individual errors

chore:write client-side tests

67ad4f4

- write tests for actions and action creators - write tests for reducers - fix bug in add recipe feature - refactor dashboard component - setup jest - install expect package - install moxios - install redux-mock-store

chore:add more reducer tests

56a5d0f

- add more tests for reducers - refactor reducers

Prepare for heroku deployment:

5a900f1

- modify files to prepare for final deployment

Prepare for heroku deployment:

45f3505

- modify files to prepare for final deployment

bug-fix: fix bug in signup and signin component

ee1239c

- fix bug in signup and signin component - modify code to use jsdoc

chore:remove console statements from components

c628d5d

- remove console.log statement from component

chore:remove console statements from components

007eeb8

- remove console.log statement from component

chore: refactor travis file for final push

73fb95d

- refactor travis file

chore: refactor travis file for final push

50dcab7

- refactor travis file

chore: refactor travis file for final push

0c9e9d6

- refactor travis file

chore: refactor travis file for final push

35ab784

- refactor travis file

chore: refactor travis file for final push

58e4985

- refactor travis file

chore: refactor travis file for final push

ae5ee17

- refactor travis file

chore: refactor travis file for final push

97b6e77

- refactor travis file

houndci-bot reviewed Feb 10, 2018

View reviewed changes

BolajiOlajide reviewed Feb 11, 2018

View reviewed changes

bug: fix jquery issues

56cb52e

Billmike temporarily deployed to more-recipes-app1 February 11, 2018 08:58 Inactive

houndci-bot reviewed Feb 11, 2018

View reviewed changes

Billmike temporarily deployed to more-recipes-app1 February 11, 2018 08:59 Inactive

chore: refactor codebase

dfbad32

Billmike temporarily deployed to more-recipes-app1 February 12, 2018 04:21 Inactive

houndci-bot reviewed Feb 12, 2018

View reviewed changes

Billmike temporarily deployed to more-recipes-app1 February 12, 2018 04:35 Inactive

chore: refactor codebase

933f898

Billmike temporarily deployed to more-recipes-app1 February 12, 2018 09:26 Inactive

Update README.md

74085a2


		after_success:
		- ./cc-test-reporter after-build --exit-code $TRAVIS_TEST_RESULT

		@@ -0,0 +1,23 @@
		import React from 'react';
		import ReactDOM from 'react-dom';


		const store = configureStore();

		if (localStorage.authToken) {

		@@ -0,0 +1 @@
		"use strict";function setOfCachedUrls(e){return e.keys().then(function(e){return e.map(function(e){return e.url})}).then(function(e){return new Set(e)})}var precacheConfig=[["/index.html","eb657bf261c4ab789cd793da99d3f7fa"],["/static/css/main.7cf33c00.css","46f792d383b5e2f870454414ed0bc7d2"],["/static/js/main.7065254e.js","2b4f759bc791fda57a1e267422dd9a01"],["/static/media/cookies.1a59c333.jpeg","1a59c33306fbf49f9009b4c1b093a0ce"],["/static/media/dessert.0760c662.jpeg","0760c662e9529adcee1fef6551de5a7c"],["/static/media/fontawesome-webfont.674f50d2.eot","674f50d287a8c48dc19ba404d20fe713"],["/static/media/fontawesome-webfont.912ec66d.svg","912ec66d7572ff821749319396470bde"],["/static/media/fontawesome-webfont.af7ae505.woff2","af7ae505a9eed503f8b8e6982036873e"],["/static/media/fontawesome-webfont.b06871f2.ttf","b06871f281fee6b241d60582ae9369b9"],["/static/media/fontawesome-webfont.fee66e71.woff","fee66e712a8a08eef5805a46892932ad"],["/static/media/pancakes.30892fc8.jpeg","30892fc8000a03c65d8552729c2d28ad"],["/static/media/pizzza.cba2d80e.jpg","cba2d80ed4c200dc6f80c88dbb6b5476"]],cacheName="sw-precache-v3-sw-precache-webpack-plugin-"+(self.registration?self.registration.scope:""),ignoreUrlParametersMatching=[/^utm_/],addDirectoryIndex=function(e,t){var n=new URL(e);return"/"===n.pathname.slice(-1)&&(n.pathname+=t),n.toString()},cleanResponse=function(e){if(!e.redirected)return Promise.resolve(e);return("body"in e?Promise.resolve(e.body):e.blob()).then(function(t){return new Response(t,{headers:e.headers,status:e.status,statusText:e.statusText})})},createCacheKey=function(e,t,n,a){var r=new URL(e);return a&&r.pathname.match(a)\|\|(r.search+=(r.search?"&":"")+encodeURIComponent(t)+"="+encodeURIComponent(n)),r.toString()},isPathWhitelisted=function(e,t){if(0===e.length)return!0;var n=new URL(t).pathname;return e.some(function(e){return n.match(e)})},stripIgnoredUrlParameters=function(e,t){var n=new URL(e);return n.hash="",n.search=n.search.slice(1).split("&").map(function(e){return e.split("=")}).filter(function(e){return t.every(function(t){return!t.test(e[0])})}).map(function(e){return e.join("=")}).join("&"),n.toString()},hashParamName="_sw-precache",urlsToCacheKeys=new Map(precacheConfig.map(function(e){var t=e[0],n=e[1],a=new URL(t,self.location),r=createCacheKey(a,hashParamName,n,/\.\w{8}\./);return[a.toString(),r]}));self.addEventListener("install",function(e){e.waitUntil(caches.open(cacheName).then(function(e){return setOfCachedUrls(e).then(function(t){return Promise.all(Array.from(urlsToCacheKeys.values()).map(function(n){if(!t.has(n)){var a=new Request(n,{credentials:"same-origin"});return fetch(a).then(function(t){if(!t.ok)throw new Error("Request for "+n+" returned a response with status "+t.status);return cleanResponse(t).then(function(t){return e.put(n,t)})})}}))})}).then(function(){return self.skipWaiting()}))}),self.addEventListener("activate",function(e){var t=new Set(urlsToCacheKeys.values());e.waitUntil(caches.open(cacheName).then(function(e){return e.keys().then(function(n){return Promise.all(n.map(function(n){if(!t.has(n.url))return e.delete(n)}))})}).then(function(){return self.clients.claim()}))}),self.addEventListener("fetch",function(e){if("GET"===e.request.method){var t,n=stripIgnoredUrlParameters(e.request.url,ignoreUrlParametersMatching),a="index.html";(t=urlsToCacheKeys.has(n))\|\|(n=addDirectoryIndex(n,a),t=urlsToCacheKeys.has(n));var r="/index.html";!t&&"navigate"===e.request.mode&&isPathWhitelisted(["^(?!\\/__).*"],e.request.url)&&(n=new URL(r,self.location).toString(),t=urlsToCacheKeys.has(n)),t&&e.respondWith(caches.open(cacheName).then(function(e){return e.match(urlsToCacheKeys.get(n)).then(function(e){if(e)return e;throw Error("The cached response that was expected is missing.")})}).catch(function(t){return console.warn('Couldn\'t serve response for "%s" from cache: %O',e.request.url,t),fetch(e.request)}))}});

Chore deploy to heroku #36

Are you sure you want to change the base?

Chore deploy to heroku #36

Conversation

Billmike commented Feb 10, 2018

What does this PR do?

Description of tasks to be completed?

How should this be manually tested?

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

BolajiOlajide left a comment

Choose a reason for hiding this comment

Why?

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Nitpick

Choose a reason for hiding this comment

Nitpick

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment