Added safery check and more tests

Author: luisdalmolin

php-loader.js

@@ -26,11 +26,17 @@ const phpLoader = {
             }).filter((file) => {
                 return path.extname(file.path) === '.php';
             }).filter((file) => {
-                if (typeof options.includeOnly === 'undefined') return true;
+                if (! options || typeof options.includeOnly === 'undefined') {
+                    return true;
+                }
+
                 var filename = path.basename(file.path, '.php');
                 return options.includeOnly.includes(filename);
             }).filter((file) => {
-                if (typeof options.exclude === 'undefined') return true;
+                if (! options || typeof options.exclude === 'undefined') {
+                    return true;
+                }
+
                 var filename = path.basename(file.path, '.php');
                 return !options.exclude.includes(filename);
             }).forEach((file) => {

test/fixtures/php/en/auth.php

@@ -0,0 +1,5 @@
+<?php
+
+return [
+    'login' => 'Login',
+];

test/php.test.js

@@ -8,7 +8,10 @@ describe('it should load php language files', function () {
     it('should load regular php translation', function () {
         let content = phpLoader.execute('./test/fixtures/php', {}, loaderMock);
 
-        assert.deepEqual(content.en, {
+        assert.deepStrictEqual(content.en, {
+            auth: {
+                login: 'Login',
+            },
             translation: {
                 validation: {
                     required: 'this field is required',
@@ -17,11 +20,38 @@ describe('it should load php language files', function () {
             }
         });
     });
+    
+    it('should be able to includeOnly options', function () {
+        let content = phpLoader.execute('./test/fixtures/php', {
+            includeOnly: ['auth']
+        }, loaderMock);
+
+        assert.deepStrictEqual(content.en, {
+            auth: {
+                login: 'Login',
+            }
+        });
+    });
+    
+    it('should be able to exclude options', function () {
+        let content = phpLoader.execute('./test/fixtures/php', {
+            exclude: ['translation']
+        }, loaderMock);
+
+        assert.deepStrictEqual(content.en, {
+            auth: {
+                login: 'Login',
+            }
+        });
+    });
 
     it('should load both languages', function () {
         let content = phpLoader.execute('./test/fixtures/php', {}, loaderMock);
 
-        assert.deepEqual(content.en, {
+        assert.deepStrictEqual(content.en, {
+            auth: {
+                login: 'Login',
+            },
             translation: {
                 validation: {
                     required: 'this field is required',