Merge pull request magesuite#63 from Igloczek/feature/theme-glob-pattern

Add support for bundling only themes defined by glob pattern

Author: krzksz

README.md

@@ -89,6 +89,9 @@ magepack bundle
 
 This command will iterate over each deployed locale (excluding Magento/blank) and prepare bundles for each of them.
 
+There is one optional param you can set:
+`-g, --glob` - [Glob pattern](https://facelessuser.github.io/wcmatch/glob/#syntax) defining where to look for locales to bundle.
+
 ### Enabling
 
 Once you made sure [Magepack Magento module](https://github.com/magesuite/magepack-magento) is installed, what is left is to enable it via admin panel under Stores->Configuration->Advanced->Developer or CLI:

cli.js

@@ -38,6 +38,7 @@ program
         'Configuration file path.',
         'magepack.config.js'
     )
+    .option('-g, --glob <path>', 'Glob pattern of themes to bundle.')
     .option('-d, --debug', 'Enable logging of debugging information.')
     .action(({ config, debug }) => {
         if (debug) {

lib/bundle.js

@@ -11,14 +11,14 @@ const checkMinifyOn = require('./bundle/checkMinifyOn');
 const moduleWrapper = require('./bundle/moduleWrapper');
 const modulePathMapper = require('./bundle/moduleMapResolver');
 
-module.exports = async (bundlingConfigPath) => {
+module.exports = async (bundlingConfigPath, localesGlobPattern) => {
     const bundlingConfigRealPath = path.resolve(bundlingConfigPath);
 
     logger.info(`Using bundling config from "${bundlingConfigRealPath}".`);
 
     const bundlingConfig = require(bundlingConfigRealPath);
 
-    const localesPaths = getLocales();
+    const localesPaths = getLocales(localesGlobPattern);
 
     const isMinifyOn = checkMinifyOn(localesPaths);
 

lib/bundle/getLocales.js

@@ -7,9 +7,9 @@ const path = require('path');
  *
  * @returns {string[]}
  */
-const getLocales = () => {
+const getLocales = (localesGlobPattern = 'pub/static/frontend/*/*/*') => {
     const locales = glob
-        .sync('pub/static/frontend/*/*/*')
+        .sync(localesGlobPattern)
         .filter((locale) => !locale.includes('Magento/blank'))
         .filter(
             (locale) =>

lib/bundle/getLocales.test.js

@@ -27,6 +27,17 @@ describe('getLocales utility', () => {
         expect(getLocales()).toEqual(['pub/static/frontend/Magento/luma']);
     });
 
+    test('returns found locales when using custom glob', () => {
+        glob.sync.mockReturnValue([
+            'pub/static/frontend/Magento/luma'
+        ]);
+        fs.existsSync.mockReturnValue(true);
+
+        expect(getLocales('pub/static/frontend/Magento/luma')).toEqual(
+            ['pub/static/frontend/Magento/luma']
+        );
+    });
+
     test('throws error when locales are found but have no requirejs-config.js file', () => {
         glob.sync.mockReturnValue([]);
         fs.existsSync.mockReturnValue(false);