Added: tests for both *nix and Windows.

rmariuzzo · rmariuzzo · commit 0ae7f411ec7c · 2018-07-09T00:32:27.000-04:00
diff --git a/README.md b/README.md
@@ -30,7 +30,7 @@ module.exports = {
     rules: [
       {
         // Matches all PHP or JSON files in `resources/lang` directory.
-        test: /resources(\\|\/)lang.+\.(php|json)$/,
+        test: /resources[\\\/]lang.+\.(php|json)$/,
         loader: 'laravel-localization-loader',
       }
     ]
@@ -47,7 +47,7 @@ mix.webpackConfig({
     rules: [
       {
         // Matches all PHP or JSON files in `resources/lang` directory.
-        test: /resources(\\|\/)lang.+\.(php|json)$/,
+        test: /resources[\\\/]lang.+\.(php|json)$/,
         loader: 'laravel-localization-loader',
       }
     ]
diff --git a/tests/fixtures/index.js b/tests/fixtures/index.js
@@ -1,2 +1 @@
 var message = require('./resources/lang/en/messages.php')
-console.log(message);
diff --git a/tests/loader.test.js b/tests/loader.test.js
@@ -29,19 +29,22 @@ describe('laravel-localization-loader', () => {
     return runWebpack(testDir.name, {
       entry: path.join(__dirname, './fixtures/resources/lang/en/messages.php'),
     })
-    .then((result) => {
+    .then((output) => {
+      const result = require(output)
       expect(result).toBeDefined()
       expect(result).toHaveProperty('string', 'Rubens')
       expect(result).toHaveProperty('number', 123)
       expect(result).toHaveProperty('parent.child', 'Mariuzzo')
       expect(result).toHaveProperty('escape', '\'escaped\'')
     })
   })
+
   it('should load JSON Laravel translation file', () => {
     return runWebpack(testDir.name, {
       entry: path.join(__dirname, './fixtures/resources/lang/en/messages.json'),
     })
-    .then((result) => {
+    .then((output) => {
+      const result = require(output)
       expect(result).toBeDefined()
       expect(result).toHaveProperty('string', 'Rubens')
       expect(result).toHaveProperty('number', 123)
@@ -74,7 +77,7 @@ function runWebpack(outputDir, config) {
       module: {
         rules: [
           {
-            test: /resources\/lang.+\.(php|json)$/,
+            test: /resources[\\\/]lang.+\.(php|json)$/,
             loader: 'laravel-localization-loader',
           }
         ]
@@ -98,11 +101,13 @@ function runWebpack(outputDir, config) {
         return reject(error)
       }
 
+      const output = `${outputDir}/translation.js`
+
       // Remove webpack output from Node require's cache.
-      delete require.cache[`${outputDir}/translation.js`]
+      delete require.cache[output]
 
-      // Resolve with the recente webpack's output.
-      return resolve(require(`${outputDir}/translation.js`))
+      // Resolve with the recent webpack's output.
+      return resolve(output)
     })
   })
 }

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ module.exports = {`
`30`	`30`	`rules: [`
`31`	`31`	`{`
`32`	`32`	// Matches all PHP or JSON files in `resources/lang` directory.
`33`		`- test: /resources(\\\|\/)lang.+\.(php\|json)$/,`
	`33`	`+ test: /resources[\\\/]lang.+\.(php\|json)$/,`
`34`	`34`	`loader: 'laravel-localization-loader',`
`35`	`35`	`}`
`36`	`36`	`]`
`@@ -47,7 +47,7 @@ mix.webpackConfig({`
`47`	`47`	`rules: [`
`48`	`48`	`{`
`49`	`49`	// Matches all PHP or JSON files in `resources/lang` directory.
`50`		`- test: /resources(\\\|\/)lang.+\.(php\|json)$/,`
	`50`	`+ test: /resources[\\\/]lang.+\.(php\|json)$/,`
`51`	`51`	`loader: 'laravel-localization-loader',`
`52`	`52`	`}`
`53`	`53`	`]`
Original file line number	Diff line number	Diff line change
`@@ -1,2 +1 @@`
`1`	`1`	`var message = require('./resources/lang/en/messages.php')`
`2`		`-console.log(message);`