fix: update skeletons to use es6, closes #482

CHANGELOG.md

@@ -6,6 +6,9 @@ All notable changes to this project will be documented in this file.
 ### Added
 - Warning with Sequelize v4
 
+### Changed
+- Skeletons are now ES6 based [#482](https://github.com/sequelize/cli/issues/482)
+
 ## v3.0.0-0 - 8th, Sep 2017
 
 ### Removed

src/assets/migrations/create-table.js

@@ -1,7 +1,7 @@
 'use strict';
 
 module.exports = {
-  up: function (queryInterface, Sequelize) {
+  up: (queryInterface, Sequelize) => {
     return queryInterface
       .createTable('<%= tableName %>', {
         id: {
@@ -29,7 +29,7 @@ module.exports = {
       });
   },
 
-  down: function (queryInterface, Sequelize) {
+  down: (queryInterface, Sequelize) => {
     return queryInterface.dropTable('<%= tableName %>');
   }
 };

src/assets/migrations/skeleton.js

@@ -1,7 +1,7 @@
 'use strict';
 
 module.exports = {
-  up: function (queryInterface, Sequelize) {
+  up: (queryInterface, Sequelize) => {
     /*
       Add altering commands here.
       Return a promise to correctly handle asynchronicity.
@@ -11,7 +11,7 @@ module.exports = {
     */
   },
 
-  down: function (queryInterface, Sequelize) {
+  down: (queryInterface, Sequelize) => {
     /*
       Add reverting commands here.
       Return a promise to correctly handle asynchronicity.

src/assets/models/index.js

@@ -16,15 +16,15 @@ if (config.use_env_variable) {
 
 fs
   .readdirSync(__dirname)
-  .filter(function(file) {
+  .filter(file => {
     return (file.indexOf('.') !== 0) && (file !== basename) && (file.slice(-3) === '.js');
   })
-  .forEach(function(file) {
+  .forEach(file => {
     var model = sequelize['import'](path.join(__dirname, file));
     db[model.name] = model;
   });
 
-Object.keys(db).forEach(function(modelName) {
+Object.keys(db).forEach(modelName => {
   if (db[modelName].associate) {
     db[modelName].associate(db);
   }

src/assets/models/model.js

@@ -1,6 +1,6 @@
 'use strict';
 
-module.exports = function(sequelize, DataTypes) {
+module.exports = (sequelize, DataTypes) => {
   var <%= name %> = sequelize.define('<%= name %>', {
     <% Object.keys(attributes).forEach(function(fieldName, index) { %>
       <%= fieldName %>: DataTypes.<%= attributes[fieldName].toUpperCase() %>

src/assets/seeders/skeleton.js

@@ -1,7 +1,7 @@
 'use strict';
 
 module.exports = {
-  up: function (queryInterface, Sequelize) {
+  up: (queryInterface, Sequelize) => {
     /*
       Add altering commands here.
       Return a promise to correctly handle asynchronicity.
@@ -14,7 +14,7 @@ module.exports = {
     */
   },
 
-  down: function (queryInterface, Sequelize) {
+  down: (queryInterface, Sequelize) => {
     /*
       Add reverting commands here.
       Return a promise to correctly handle asynchronicity.

test/migration/create.test.js

@@ -46,8 +46,8 @@ const gulp      = require('gulp');
           .src(Support.resolveSupportPath('tmp', 'migrations'))
           .pipe(helpers.readFile('*-' + migrationFile))
           .pipe(helpers.expect(stdout => {
-            expect(stdout).to.contain('up: function (queryInterface, Sequelize) {');
-            expect(stdout).to.contain('down: function (queryInterface, Sequelize) {');
+            expect(stdout).to.contain('up: (queryInterface, Sequelize) => {');
+            expect(stdout).to.contain('down: (queryInterface, Sequelize) => {');
           }))
           .pipe(helpers.teardown(done));
       });

test/seed/create.test.js

@@ -48,8 +48,8 @@ const gulp      = require('gulp');
           .src(Support.resolveSupportPath('tmp', 'seeders'))
           .pipe(helpers.readFile('*-' + seedFile))
           .pipe(helpers.expect(stdout => {
-            expect(stdout).to.contain('up: function (queryInterface, Sequelize) {');
-            expect(stdout).to.contain('down: function (queryInterface, Sequelize) {');
+            expect(stdout).to.contain('up: (queryInterface, Sequelize) => {');
+            expect(stdout).to.contain('down: (queryInterface, Sequelize) => {');
           }))
           .pipe(helpers.teardown(done));
       });