make getModuleBrands return an array with all brands defined if the o…

…rigami.json files has no brands property

lib/helpers/files.js

@@ -84,17 +84,18 @@ function getComponentName(cwd) {
 function getModuleBrands(cwd) {
 	return getOrigamiJson(cwd)
 		.then(origamiJson => {
-			if (origamiJson && origamiJson.brands && Array.isArray(origamiJson.brands)) {
+			const hasBrandsDefined = origamiJson && origamiJson.brands && Array.isArray(origamiJson.brands) && origamiJson.brands.length > 0;
+			if (hasBrandsDefined) {
 				return origamiJson.brands;
 			}
-			return [];
+			return ['master', 'internal', 'whitelabel'];
 		});
 }
 
 /**
  * Get individual demo configuration for the component.
  * @param {string} cwd - The component's directory (the current working directory).
- * @return {array<object>} - An array of objects representing a component demo.
+ * @return {Promise<Array<object>>} - An array of objects representing a component demo.
  */
 async function getComponentDemos(cwd) {
 	const origamiJson = await getOrigamiJson(cwd);

test/unit/helpers/files.test.js

@@ -277,4 +277,30 @@ describe('Files helper', function () {
 			proclaim.lengthEquals(Object.keys(demosDefaults), 0, 'Expected an empty object given missing default demo configuration.');
 		});
 	});
+
+	describe('getModuleBrands', function() {
+		context('if no brands are defined', function() {
+			it('should return an array containing only `master`, `internal`, `whitelabel`', async function(){
+				const brands = await files.getModuleBrands();
+				proclaim.deepStrictEqual(brands, ['master', 'internal', 'whitelabel']);
+			});
+		});
+
+		context('if brands is defined as an array with no items', function(){
+			it('should return an array containing only `master`, `internal`, `whitelabel`', async function(){
+				const brands = await files.getModuleBrands();
+				proclaim.deepStrictEqual(brands, ['master', 'internal', 'whitelabel']);
+			});
+		});
+
+		context('if brands is defined as an array with items', function(){
+			it('should return an arary with the same items as those defined', async function() {
+				const origamiManifest = JSON.parse(fs.readFileSync('origami.json', 'utf8'));
+				origamiManifest.brands = ['master'];
+				fs.writeFileSync('origami.json', JSON.stringify(origamiManifest), 'utf8');
+				const brands = await files.getModuleBrands();
+				proclaim.deepStrictEqual(brands, ['master']);
+			});
+		});
+	});
 });