Never let getStyleFiles throw an error, complete branch coverage also

clay · May 29, 2018 · 04dc6c0 · 04dc6c0
1 parent 474973b
commit 04dc6c0
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/lib/media.js b/lib/media.js
@@ -182,7 +182,7 @@ function getStyleFiles(state) {
     assetPath = site && site.assetPath || '',
     assetHost = site && site.assetHost || '',
     siteStyleguide = site && site.styleguide;
-  let cssFilePaths;
+  let cssFilePaths = [];
 
   // legacy behavior: load <component>.css and <component>.<site>.css
   if (!site.styleguide) {

diff --git a/lib/media.test.js b/lib/media.test.js
@@ -195,6 +195,22 @@ describe(_.startCase(filename), function () {
       ]);
     });
 
+    it('returns no styles when there is a style guide but no _usedVariations object', function () {
+      const localState = {
+        locals: {
+          edit: true,
+          site: {
+            slug: 'foo',
+            assetDir: '/foo',
+            styleguide: 'bar'
+          }
+        },
+        _components: []
+      };
+
+      expect(fn(localState)).to.deep.equal([]);
+    });
+
     describe('with no site styleguide (legacy)', function () {
       it('handles an empty component list', function () {
         expect(fn({