Added extra test

php-loader.js

@@ -36,14 +36,12 @@ const phpLoader = {
                 content = content.replace(/\?>\s*$/, '_')
 
                 let langObject = {}
-                let parsedContent = null
+
                 try {
-                    parsedContent = phpArrayParser.parse(content);
+                    let parsedContent = phpArrayParser.parse(content);
+                    langObject[filename.replace('.php', '')] = parsedContent;
                 } catch (e) {
-                    console.warn('The file "' + file.path + '" could not be parsed', e.message)
-                }
-                if (parsedContent !== null) {
-                    langObject[filename.replace('.php', '')] = parsedContent
+                    console.warn('The file "' + file.path + '" could not be parsed', e.message);
                 }
 
                 if (typeof options.namespace !== 'undefined') {

test/fixtures/php-with-one-invalid-file/en/invalid.php

@@ -0,0 +1,3 @@
+<?php
+
+return require_once('inexisting-file.php');

test/fixtures/php-with-one-invalid-file/en/translation.php

@@ -0,0 +1,8 @@
+<?php
+
+return [
+    'validation' => [
+        'required' => 'this field is required',
+    ],
+    'another' => 'translation',
+];

test/fixtures/php-with-one-invalid-file/es/translation.php

@@ -0,0 +1,8 @@
+<?php
+
+return [
+    'validation' => [
+        'required' => 'este campo es obrigatorio',
+    ],
+    'another' => 'traducion',
+];

test/php.test.js

@@ -90,4 +90,17 @@ describe('it should load php language files', function () {
             }
         });
     });
+
+    it('should not fail execution with invalid file', function () {
+        let content = phpLoader.execute('./test/fixtures/php-with-one-invalid-file', {});
+
+        assert.deepEqual(content.en, {
+            translation: {
+                validation: {
+                    required: 'this field is required',
+                },
+                another: 'translation',
+            }
+        });
+    });
 });