rust-lang · carols10cents · Jun 9, 2017 · May 9, 2017 · May 9, 2017 · May 25, 2017
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -0,0 +1,13 @@
+module.exports = {
+  root: true,
+  parserOptions: {
+    ecmaVersion: 2017,
+    sourceType: 'module'
+  },
+  extends: 'eslint:recommended',
+  env: {
+    browser: true
+  },
+  rules: {
+  }
+};
diff --git a/app/adapters/application.js b/app/adapters/application.js
@@ -1,5 +1,5 @@
 import DS from 'ember-data';
 
 export default DS.RESTAdapter.extend({
-    namespace: 'api/v1',
+    namespace: 'api/v1'
 });
diff --git a/app/app.js b/app/app.js
@@ -8,8 +8,6 @@ let App;
 
 let CargoResolver = Resolver.extend(RouteAliasResolverMixin);
 
-Ember.MODEL_FACTORY_INJECTIONS = true;
-
 App = Ember.Application.extend({
     modulePrefix: config.modulePrefix,
     podModulePrefix: config.podModulePrefix,

diff --git a/app/index.html b/app/index.html
@@ -5,7 +5,7 @@
     <meta http-equiv="X-UA-Compatible" content="IE=edge">
     <meta name="viewport" content="width=device-width, initial-scale=1">
 
-    {{content-for 'head'}}
+    {{content-for "head"}}
 
     <script>
       (function(i,s,o,g,r,a,m){i['GoogleAnalyticsObject']=r;i[r]=i[r]||function(){
@@ -20,13 +20,13 @@
     <link rel="stylesheet" href="{{rootURL}}assets/vendor.css">
     <link rel="stylesheet" href="{{rootURL}}assets/cargo.css">
 
-    {{content-for 'head-footer'}}
+    {{content-for "head-footer"}}
     <link rel="shortcut icon" href="/favicon.ico" type="image/x-icon">
     <link rel="icon" href="/favicon.ico" type="image/x-icon">
     <link rel="search" href="/opensearch.xml" type="application/opensearchdescription+xml" title="Cargo">
   </head>
   <body>
-    {{content-for 'body'}}
+    {{content-for "body"}}
     <noscript>
         <div id="main">
             <div class='noscript'>
@@ -38,6 +38,6 @@
     <script src="{{rootURL}}assets/vendor.js"></script>
     <script src="{{rootURL}}assets/cargo.js"></script>
 
-    {{content-for 'body-footer'}}
+    {{content-for "body-footer"}}
   </body>
 </html>
diff --git a/app/models/crate.js b/app/models/crate.js
@@ -20,13 +20,9 @@ export default DS.Model.extend({
 
     versions: DS.hasMany('versions', { async: true }),
     badges: DS.attr(),
-    enhanced_badges: Ember.computed.map('badges', badge => ({
-        // jshint ignore:start
-        // needed until https://github.com/jshint/jshint/issues/2991 is fixed
-        ...badge,
-        // jshint ignore:end
+    enhanced_badges: Ember.computed.map('badges', badge => Ember.assign({
         component_name: `badge-${badge.badge_type}`
-    })),
+    }, badge)),
     badge_sort: ['badge_type'],
     annotated_badges: Ember.computed.sort('enhanced_badges', 'badge_sort'),
     owners: DS.hasMany('users', { async: true }),

diff --git a/app/routes/login.js b/app/routes/login.js
@@ -11,7 +11,9 @@ export default Ember.Route.extend({
     beforeModel(transition) {
         try {
             localStorage.removeItem('github_response');
-        } catch (e) {}
+        } catch (e) {
+            // TODO: handle error
+        }
 
         delete window.github_response;
         var win = window.open('/github_login', 'Authorization',

diff --git a/app/serializers/crate.js b/app/serializers/crate.js
@@ -1,13 +1,8 @@
 import DS from 'ember-data';
 
 export default DS.RESTSerializer.extend({
-    isNewSerializerAPI: true,
-
-    extractRelationships(modelClass, resourceHash) {
-        if (resourceHash.versions == null) {
-            delete resourceHash.versions;
-        }
-
-        return this._super(modelClass, resourceHash);
+    normalize(modelClass, resourceHash) {
+        modelClass.eachRelationship(key => resourceHash[key] !== null || delete resourceHash[key]);
+        return this._super(...arguments);
     }
 });
diff --git a/app/services/session.js b/app/services/session.js
@@ -22,7 +22,9 @@ export default Ember.Service.extend({
         this.set('currentUser', user);
         try {
             localStorage.setItem('isLoggedIn', '1');
-        } catch (e) {}
+        } catch (e) {
+            // TODO: handle exception
+        }
     },
 
     logoutUser() {
@@ -32,6 +34,8 @@ export default Ember.Service.extend({
 
         try {
             localStorage.removeItem('isLoggedIn');
-        } catch (e) {}
+        } catch (e) {
+            // TODO: handle exception
+        }
     }
 });
diff --git a/bower.json b/bower.json
@@ -1,13 +1,11 @@
 {
   "name": "cargo",
   "dependencies": {
-    "Faker": "3.1.0",
-    "jquery": "3.2.0",
+    "Faker": "4.1.0",
     "lodash": "4.17.4",
-    "ember": "2.12.0",
-    "moment": "2.17.1",
-    "moment-timezone": "0.5.11",
-    "normalize-css": "5.0.0",
+    "moment": "2.18.1",
+    "moment-timezone": "0.5.13",
+    "normalize-css": "7.0.0",
     "pretender": "1.4.2",
     "clipboard": "1.6.1"
   }

diff --git a/config/environment.js b/config/environment.js
@@ -1,4 +1,4 @@
-/* jshint node: true */
+/* eslint-env node */
 
 module.exports = function(environment) {
   var ENV = {

diff --git a/config/targets.js b/config/targets.js
@@ -0,0 +1,10 @@
+/* eslint-env node */
+
+module.exports = {
+  browsers: [
+    'ie 9',
+    'last 1 Chrome versions',
+    'last 1 Firefox versions',
+    'last 1 Safari versions'
+  ]
+};
diff --git a/ember-cli-build.js b/ember-cli-build.js
@@ -1,5 +1,4 @@
-/*jshint node:true*/
-/* global require, module */
+/* eslint-env node */
 const EmberApp = require('ember-cli/lib/broccoli/ember-app');
 
 module.exports = function(defaults) {

diff --git a/package.json b/package.json
@@ -1,11 +1,13 @@
 {
   "name": "cargo",
   "version": "0.0.0",
-  "private": true,
+  "license": "MIT",
+  "author": "",
   "directories": {
     "doc": "doc",
     "test": "tests"
   },
+  "repository": "",
   "scripts": {
     "build": "yarn run ember build",
     "start": "yarn run ember server",
@@ -15,46 +17,45 @@
     "test": "yarn run ember test",
     "ember": "ember"
   },
-  "repository": "",
-  "engines": {
-    "node": "= 4.6.0"
-  },
-  "author": "",
-  "license": "MIT",
   "devDependencies": {
     "bower": "1.8.0",
-    "broccoli-asset-rev": "2.5.0",
-    "ember-ajax": "2.5.6",
-    "ember-cli": "2.12.0",
-    "ember-cli-app-version": "2.0.2",
-    "ember-cli-babel": "5.2.4",
-    "ember-cli-dependency-checker": "1.3.0",
-    "ember-cli-htmlbars": "1.2.0",
-    "ember-cli-htmlbars-inline-precompile": "0.3.6",
+    "broccoli-asset-rev": "^2.4.5",
+    "ember-ajax": "^3.0.0",
+    "ember-cli": "2.13.2",
+    "ember-cli-app-version": "^3.0.0",
+    "ember-cli-babel": "^6.0.0",
+    "ember-cli-dependency-checker": "^1.3.0",
+    "ember-cli-eslint": "^3.0.0",
+    "ember-cli-htmlbars": "^1.1.1",
+    "ember-cli-htmlbars-inline-precompile": "^0.4.0",
     "ember-cli-ic-ajax": "1.0.0",
-    "ember-cli-inject-live-reload": "1.6.1",
-    "ember-cli-jshint": "2.0.1",
-    "ember-cli-meta-tags": "3.0.4",
-    "ember-cli-mirage": "0.2.9",
-    "ember-cli-moment-shim": "3.0.1",
-    "ember-cli-qunit": "3.1.2",
+    "ember-cli-inject-live-reload": "^1.4.1",
+    "ember-cli-meta-tags": "3.1.0",
+    "ember-cli-mirage": "0.3.2",
+    "ember-cli-moment-shim": "3.3.0",
+    "ember-cli-qunit": "^4.0.0",
     "ember-cli-release": "0.2.9",
-    "ember-cli-sass": "6.1.2",
-    "ember-cli-shims": "1.0.2",
-    "ember-cli-sri": "2.1.1",
-    "ember-cli-test-loader": "1.1.1",
-    "ember-cli-uglify": "1.2.0",
-    "ember-data": "2.12.1",
-    "ember-export-application-global": "1.1.1",
-    "ember-load-initializers": "0.6.3",
-    "ember-lodash": "4.17.1",
-    "ember-moment": "7.3.0",
-    "ember-page-title": "3.1.5",
-    "ember-resolver": "3.0.0",
+    "ember-cli-sass": "6.1.3",
+    "ember-cli-shims": "^1.1.0",
+    "ember-cli-sri": "^2.1.0",
+    "ember-cli-test-loader": "2.1.0",
+    "ember-cli-uglify": "^1.2.0",
+    "ember-data": "^2.13.0",
+    "ember-export-application-global": "^2.0.0",
+    "ember-load-initializers": "^1.0.0",
+    "ember-lodash": "4.17.4",
+    "ember-moment": "7.3.1",
+    "ember-page-title": "3.1.6",
+    "ember-resolver": "^4.0.0",
     "ember-route-alias": "0.1.5",
-    "ember-router-scroll": "0.1.1",
+    "ember-router-scroll": "0.2.0",
+    "ember-source": "~2.13.0",
     "ember-suave": "4.0.1",
-    "emberx-select": "2.2.3",
-    "loader.js": "4.2.3"
-  }
+    "emberx-select": "^3.0.1",
+    "loader.js": "^4.2.3"
+  },
+  "engines": {
+    "node": ">= 4"
+  },
+  "private": true
 }
diff --git a/testem.js b/testem.js
@@ -1,4 +1,4 @@
-/*jshint node:true*/
+/* eslint-env node */
 module.exports = {
   "test_page": "tests/index.html?hidepassed",
   "disable_watching": true,

diff --git a/testem.json b/testem.json
diff --git a/tests/.eslintrc.js b/tests/.eslintrc.js
@@ -0,0 +1,5 @@
+module.exports = {
+  env: {
+    embertest: true
+  }
+};
diff --git a/tests/acceptance/categories-test.js b/tests/acceptance/categories-test.js
@@ -1,3 +1,4 @@
+/*global hasText*/
 import { test } from 'qunit';
 import moduleForAcceptance from 'cargo/tests/helpers/module-for-acceptance';
 

diff --git a/tests/acceptance/crate-test.js b/tests/acceptance/crate-test.js
@@ -1,3 +1,4 @@
+/*global hasText, matchesText*/
 import { test } from 'qunit';
 import moduleForAcceptance from 'cargo/tests/helpers/module-for-acceptance';
 

diff --git a/tests/acceptance/front-page-test.js b/tests/acceptance/front-page-test.js
@@ -1,3 +1,4 @@
+/*global hasText*/
 import { test } from 'qunit';
 import moduleForAcceptance from 'cargo/tests/helpers/module-for-acceptance';
 

diff --git a/tests/acceptance/search-test.js b/tests/acceptance/search-test.js
@@ -1,3 +1,4 @@
+/*global hasText, wait*/
 import { test } from 'qunit';
 import moduleForAcceptance from 'cargo/tests/helpers/module-for-acceptance';
 

diff --git a/tests/helpers/destroy-app.js b/tests/helpers/destroy-app.js
@@ -1,3 +1,4 @@
+/*global server*/
 import Ember from 'ember';
 
 export default function destroyApp(application) {

diff --git a/tests/helpers/start-app.js b/tests/helpers/start-app.js
@@ -5,16 +5,13 @@ import './has-text';
 import './matches-text';
 
 export default function startApp(attrs) {
-    let application;
+    let attributes = Ember.merge({}, config.APP);
+    attributes = Ember.merge(attributes, attrs); // use defaults, but you can override;
 
-    // use defaults, but you can override
-    let attributes = Ember.assign({}, config.APP, attrs);
-
-    Ember.run(() => {
-        application = Application.create(attributes);
+    return Ember.run(() => {
+        let application = Application.create(attributes);
         application.setupForTesting();
         application.injectTestHelpers();
+        return application;
     });
-
-    return application;
 }
diff --git a/tests/index.html b/tests/index.html
@@ -3,6 +3,7 @@
   <head>
     <meta charset="utf-8">
     <meta http-equiv="X-UA-Compatible" content="IE=edge">
+    <title>Cargo Tests</title>
     <meta name="description" content="">
     <meta name="viewport" content="width=device-width, initial-scale=1">
 

diff --git a/tests/test-helper.js b/tests/test-helper.js
@@ -1,8 +1,8 @@
 import resolver from './helpers/resolver';
-import registerSelectHelper from './helpers/register-select-helper';
-registerSelectHelper();
 import {
   setResolver
 } from 'ember-qunit';
+import { start } from 'ember-cli-qunit';
 
 setResolver(resolver);
+start();