From 22e275430d529aab9554b39f7ccee686999a0900 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Michal=20Kvasni=C4=8D=C3=A1k?= Date: Fri, 17 Jun 2016 12:25:01 +0200 Subject: [PATCH 1/3] try to reproduce the problem from #13 --- error_case/.babelrc | 7 +++++++ error_case/package.json | 11 +++++++++++ error_case/src/index.js | 1 + error_case/src/style.css | 0 error_case/src/utils/processCss.js | 3 +++ 5 files changed, 22 insertions(+) create mode 100644 error_case/.babelrc create mode 100644 error_case/package.json create mode 100644 error_case/src/index.js create mode 100644 error_case/src/style.css create mode 100644 error_case/src/utils/processCss.js diff --git a/error_case/.babelrc b/error_case/.babelrc new file mode 100644 index 0000000..1c9af58 --- /dev/null +++ b/error_case/.babelrc @@ -0,0 +1,7 @@ +{ + "plugins": [ + ["css-modules-transform", { + "processCss": "./src/utils/processCss.js" + }] + ] +} diff --git a/error_case/package.json b/error_case/package.json new file mode 100644 index 0000000..a2a9ba2 --- /dev/null +++ b/error_case/package.json @@ -0,0 +1,11 @@ +{ + "name": "application-name", + "version": "0.0.1", + "dependencies": { + "babel-cli": "6.10.1", + "babel-plugin-css-modules-transform": "0.1.1" + }, + "scripts": { + "test": "babel src" + } +} diff --git a/error_case/src/index.js b/error_case/src/index.js new file mode 100644 index 0000000..66eb862 --- /dev/null +++ b/error_case/src/index.js @@ -0,0 +1 @@ +const styles = require('./style.css'); diff --git a/error_case/src/style.css b/error_case/src/style.css new file mode 100644 index 0000000..e69de29 diff --git a/error_case/src/utils/processCss.js b/error_case/src/utils/processCss.js new file mode 100644 index 0000000..ae0a052 --- /dev/null +++ b/error_case/src/utils/processCss.js @@ -0,0 +1,3 @@ +module.exports = function processCss() { + console.log('test'); +}; From a88d53068bda6ef4f4f942aa4f7f0d41a881dccb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Michal=20Kvasni=C4=8D=C3=A1k?= Date: Tue, 21 Jun 2016 11:46:06 +0200 Subject: [PATCH 2/3] add docker environment for error case --- error_case/.dockerignore | 40 ++++++++++++++++++++++++++++++++++++++++ error_case/Dockerfile | 11 +++++++++++ error_case/README.md | 7 +++++++ 3 files changed, 58 insertions(+) create mode 100644 error_case/.dockerignore create mode 100644 error_case/Dockerfile create mode 100644 error_case/README.md diff --git a/error_case/.dockerignore b/error_case/.dockerignore new file mode 100644 index 0000000..293a66d --- /dev/null +++ b/error_case/.dockerignore @@ -0,0 +1,40 @@ +# Created by .ignore support plugin (hsz.mobi) +### Node template +# Logs +logs +*.log +npm-debug.log* + +# Runtime data +pids +*.pid +*.seed + +# Directory for instrumented libs generated by jscoverage/JSCover +lib-cov + +# Coverage directory used by tools like istanbul +coverage + +# nyc test coverage +.nyc_output + +# Grunt intermediate storage (http://gruntjs.com/creating-plugins#storing-task-files) +.grunt + +# node-waf configuration +.lock-wscript + +# Compiled binary addons (http://nodejs.org/api/addons.html) +build/Release + +# Dependency directories +node_modules +jspm_packages + +# Optional npm cache directory +.npm + +# Optional REPL history +.node_repl_history + diff --git a/error_case/Dockerfile b/error_case/Dockerfile new file mode 100644 index 0000000..f63d7da --- /dev/null +++ b/error_case/Dockerfile @@ -0,0 +1,11 @@ +FROM node:6.2.2 + +ENV NODE_ENV=development + +WORKDIR /var/www/html + +ADD . /var/www/html + +RUN npm install + +ENTRYPOINT npm test diff --git a/error_case/README.md b/error_case/README.md new file mode 100644 index 0000000..cc89309 --- /dev/null +++ b/error_case/README.md @@ -0,0 +1,7 @@ +# Installation + +`docker build -t error_case .` + +# Test + +`docker run error_case` From 65c23ed501aee22dda3f27b4ad536e5f0d3999d6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Michal=20Kvasni=C4=8D=C3=A1k?= Date: Fri, 2 Sep 2016 13:39:45 +0200 Subject: [PATCH 3/3] simulate babel-register too --- error_case/Dockerfile | 6 ++++-- error_case/package.json | 6 ++++-- error_case/run.sh | 4 ++++ error_case/src/import.js | 5 +++++ src/options_resolvers/processCss.js | 1 + src/utils/requireLocalFileOrNodeModule.js | 3 +++ 6 files changed, 21 insertions(+), 4 deletions(-) create mode 100755 error_case/run.sh create mode 100644 error_case/src/import.js diff --git a/error_case/Dockerfile b/error_case/Dockerfile index f63d7da..cd5028b 100644 --- a/error_case/Dockerfile +++ b/error_case/Dockerfile @@ -4,8 +4,10 @@ ENV NODE_ENV=development WORKDIR /var/www/html -ADD . /var/www/html +ADD ./package.json /var/www/html/ RUN npm install -ENTRYPOINT npm test +ADD . /var/www/html + +ENTRYPOINT ./run.sh diff --git a/error_case/package.json b/error_case/package.json index a2a9ba2..1f071df 100644 --- a/error_case/package.json +++ b/error_case/package.json @@ -3,9 +3,11 @@ "version": "0.0.1", "dependencies": { "babel-cli": "6.10.1", - "babel-plugin-css-modules-transform": "0.1.1" + "babel-register": "6.14.0", + "babel-plugin-css-modules-transform": "1.0.0" }, "scripts": { - "test": "babel src" + "test": "babel src", + "test-register": "node src/import.js" } } diff --git a/error_case/run.sh b/error_case/run.sh new file mode 100755 index 0000000..055fc5e --- /dev/null +++ b/error_case/run.sh @@ -0,0 +1,4 @@ +#!/usr/bin/env bash + +npm test +npm run test-register diff --git a/error_case/src/import.js b/error_case/src/import.js new file mode 100644 index 0000000..3b6c1e2 --- /dev/null +++ b/error_case/src/import.js @@ -0,0 +1,5 @@ +require('babel-register')({ + ignore: /(src\/utils\/processCss\.js|node_modules|build)/ +}); + +require('./index'); diff --git a/src/options_resolvers/processCss.js b/src/options_resolvers/processCss.js index e5783ad..94526d5 100644 --- a/src/options_resolvers/processCss.js +++ b/src/options_resolvers/processCss.js @@ -11,6 +11,7 @@ export default function processCss(value/* ,currentConfig */) { if (isModulePath(value)) { const requiredModule = requireLocalFileOrNodeModule(value); + console.dir('required module', requiredModule); if (isFunction(requiredModule)) { return requiredModule; } diff --git a/src/utils/requireLocalFileOrNodeModule.js b/src/utils/requireLocalFileOrNodeModule.js index c421272..183e45e 100644 --- a/src/utils/requireLocalFileOrNodeModule.js +++ b/src/utils/requireLocalFileOrNodeModule.js @@ -11,9 +11,12 @@ export default function requireLocalFileOrNodeModule(path) { try { // first try to require local file + console.log('local file', localFile); return require(localFile); } catch (e) { + console.log('local file error', e); // try to require node_module + console.log('npm module', path); return require(path); } }