Merge pull request #320 from hannu/add-wrapper-config

Fix addWrapper issue when styleguide config is not yet loaded

Author: Juuso Backman

lib/app/js/app.js

@@ -93,9 +93,8 @@ angular.module('sgApp', [
   })
   .filter('addWrapper', ['Styleguide', function(Styleguide) {
     return function(html) {
-      var cls = Styleguide.config.data.commonClass;
-      if (cls) {
-        return '<sg-custom-wrapper class="' + cls + '">' + html + '</sg-custom-wrapper>';
+      if (Styleguide.config && Styleguide.config.data && Styleguide.config.data.commonClass) {
+        return '<sg-custom-wrapper class="' + Styleguide.config.data.commonClass + '">' + html + '</sg-custom-wrapper>';
       }
       return html;
     };

test/angular/app.test.js

@@ -106,6 +106,12 @@ describe('sgApp module', function() {
       expect(addWrapper(input)).to.eql(input);
     });
 
+    it('returns input as-is if Styleguide config is not yet loaded', function() {
+      Styleguide = {};
+      var input = 'unchanged';
+      expect(addWrapper(input)).to.eql(input);
+    });
+
     it('returns input wrapped inside a <sg-custom-wrapper> tag with common class if Styleguide config has commonClass', function() {
       Styleguide.config.data.commonClass = 'my-common-class';
       var input = 'wrapped',