integration tests

opencomponents · Jan 12, 2017 · cde708d · cde708d
1 parent 57911f3
commit cde708d
Show file tree

Hide file tree

Showing 6 changed files with 209 additions and 405 deletions.
diff --git a/src/cli/domain/package-server-script/bundle/config/externalDependenciesHandlers.js b/src/cli/domain/package-server-script/bundle/config/externalDependenciesHandlers.js
@@ -12,27 +12,27 @@ var strings = require('../../../../../resources');
 
 
 module.exports = function externalDependenciesHandlers(dependencies){
-  var deps = dependencies || {}
+  var deps = dependencies || {};
 
   var missingExternalDependecy = function(dep, dependencies) {
     return !_.contains(_.keys(dependencies), dep);
-  }
+  };
 
   return [
     function(context, req, callback) {
       if (/^[a-z@][a-z\-\/0-9]+$/.test(req)) {
         var dependencyName = req;
         if (/\//g.test(dependencyName)) {
-          dependencyName = dependencyName.substring(0, dependencyName.indexOf("/"));
+          dependencyName = dependencyName.substring(0, dependencyName.indexOf('/'));
         }
         if (missingExternalDependecy(dependencyName, deps)) {
           return callback(new Error(format(strings.errors.cli.SERVERJS_DEPENDENCY_NOT_DECLARED, JSON.stringify(dependencyName))));
         }
       }
-      callback()
+      callback();
     },
     /^[a-z@][a-z\-\/0-9]+$/
-  ]
+  ];
 };
 
 
diff --git a/src/cli/domain/package-server-script/bundle/config/index.js b/src/cli/domain/package-server-script/bundle/config/index.js
@@ -28,7 +28,7 @@ module.exports = function webpackConfigGenerator(params){
           loaders: [
             'falafel-loader',
             'babel-loader?' + JSON.stringify({
-              cacheDirectory: false,
+              cacheDirectory: true,
               'presets': [
                 ['env', {
                   'targets': {
@@ -43,13 +43,13 @@ module.exports = function webpackConfigGenerator(params){
     },
     plugins: [
       new webpack.optimize.OccurenceOrderPlugin(),
-      // new webpack.optimize.UglifyJsPlugin({
-      //   compressor: {
-      //     warnings: false,
-      //     screw_ie8: true
-      //   },
-      //   sourceMap: false
-      // }),
+      new webpack.optimize.UglifyJsPlugin({
+        compressor: {
+          warnings: false,
+          screw_ie8: true
+        },
+        sourceMap: false
+      }),
       new webpack.DefinePlugin({
         'process.env.NODE_ENV': JSON.stringify(process.env.NODE_ENV)
       })

diff --git a/src/cli/domain/package-server-script/bundle/config/wrapLoops.js b/src/cli/domain/package-server-script/bundle/config/wrapLoops.js
@@ -18,7 +18,7 @@ module.exports = function wrapLoops(node){
   }
 
   if(loopKeywords.indexOf(node.parent.type) > -1 && node.type === 'BlockStatement'){
-    node.update('{ if(__ITER <=0){ throw new Error("loop exceeded maximum '
+    node.update('{ if(__ITER <=0){ throw new Error("Loop exceeded maximum '
       + 'allowed iterations"); } '
       + node.source().substr(1).slice(0, -1)
       + ' __ITER--; }'

diff --git a/src/cli/domain/package-server-script/bundle/index.js b/src/cli/domain/package-server-script/bundle/index.js
@@ -10,7 +10,7 @@ var webpack = require('webpack');
 var memoryFs = new MemoryFS();
 
 module.exports = function bundle(params, callBack) {
-  var config = webpackConfig(params)
+  var config = webpackConfig(params);
   var compiler = webpack(config);
   compiler.outputFileSystem = memoryFs;
 
@@ -39,4 +39,4 @@ module.exports = function bundle(params, callBack) {
     var serverContentBundled = memoryFs.readFileSync('/build/server.js', 'UTF8');
     callBack(warning, serverContentBundled);
   });
-}
+};
diff --git a/src/cli/domain/package-server-script/index.js b/src/cli/domain/package-server-script/index.js
@@ -15,7 +15,7 @@ var webpackDefaults =  {
 };
 
 module.exports = function packageServerScript(params, callback){
-  var fileName = 'server.js'
+  var fileName = 'server.js';
   var publishPath = params.publishPath;
 
   var bundleParams = {