NYT API Example Error Handling

This change will handle more error codes returned from NYT API than just 403 HTTP errors.
Remove NYT API from the unit tests since it would depend on NYT's endpoint which may not be predictable.

controllers/api.js

@@ -150,8 +150,8 @@ exports.getNewYorkTimes = (req, res, next) => {
   };
   request.get({ url: 'http://api.nytimes.com/svc/books/v2/lists', qs: query }, (err, request, body) => {
     if (err) { return next(err); }
-    if (request.statusCode === 403) {
-      return next(new Error('Invalid New York Times API Key'));
+    if (request.statusCode >= 400) {
+      return next(new Error(`New York Times API - ${body}`));
     }
     const books = JSON.parse(body).results;
     res.render('api/nyt', {

test/app.js

@@ -49,14 +49,6 @@ describe('GET /api/lastfm', () => {
   });
 });
 
-describe('GET /api/nyt', () => {
-  it('should return 200 OK', (done) => {
-    request(app)
-      .get('/api/nyt')
-      .expect(200, done);
-  });
-});
-
 describe('GET /api/twilio', () => {
   it('should return 200 OK', (done) => {
     request(app)