Unit test to cover correct application of multiple partials with differing style modifiers

Author: bmuenzenmeyer

builder/pattern_assembler.js

@@ -207,7 +207,7 @@
 
         //do something with the regular old partials
         for(i = 0; i < foundPatternPartials.length; i++){
-          var partialKey = foundPatternPartials[i].replace(/{{>([ ])?([\w\-\.\/~]+)(:[A-z-_|]+)?(?:\:[A-Za-z0-9-]+)?(?:(| )\(.*)?([ ])?}}/g, '$2');
+          var partialKey = foundPatternPartials[i].replace(/{{>([ ])?([\w\-\.\/~]+)(:[A-z-_|]+)?(?:\:[A-Za-z0-9-_]+)?(?:(| )\(.*)?([ ])?}}/g, '$2');
 
           var partialPath;
 

test/files/_patterns/00-test/03-styled-atom.mustache

@@ -0,0 +1,3 @@
+<span class="test_base {{styleModifier}}">
+    {{message}}
+</span>

test/files/_patterns/00-test/04-group.mustache

@@ -0,0 +1,6 @@
+<div class="test_group">
+    {{> test-styled-atom:test_1 }}
+    {{> test-styled-atom:test_2 }}
+    {{> test-styled-atom:test_3 }}
+    {{> test-styled-atom:test_4 }}
+</div>

test/files/_patterns/00-test/05-group2.mustache

@@ -0,0 +1,6 @@
+<div class="test_group">
+    {{> test-styled-atom:test_1(message: "1" ) }}
+    {{> test-styled-atom:test_2(message: "2" ) }}
+    {{> test-styled-atom:test_3(message: "3" ) }}
+    {{> test-styled-atom:test_4(message: "4" ) }}
+</div>

test/pattern_assembler_tests.js

@@ -2,6 +2,7 @@
 	"use strict";
 
 	var pa = require('../builder/pattern_assembler');
+	var object_factory = require('../builder/object_factory');
 
 	exports['pattern_assembler'] = {
 		'find_pattern_partials finds partials' : function(test){
@@ -283,6 +284,38 @@
 			//test that 00-foo.mustache included partial 01-bar.mustache
 			test.equals(fooExtended, 'bar');
 
+			test.done();
+		},
+		'processPatternRecursive - correctly replaces all stylemodifiers when multiple duplicate patterns with different stylemodifiers found' : function(test){
+			//arrange
+			var fs = require('fs-extra');
+			var pattern_assembler = new pa();
+
+			var pl = {};
+			pl.config = {};
+			pl.data = {};
+			pl.data.link = {};
+			pl.config.debug = false;
+			pl.patterns = [];
+			var patterns_dir = './test/files/_patterns';
+
+			var atomPattern = new object_factory.oPattern('test/files/_patterns/00-test/03-styled-atom.mustache', '00-test', '03-styled-atom.mustache');
+			atomPattern.template = fs.readFileSync(patterns_dir + '/00-test/03-styled-atom.mustache', 'utf8');
+			atomPattern.stylePartials = pattern_assembler.find_pattern_partials_with_style_modifiers(atomPattern);
+
+
+			var groupPattern = new object_factory.oPattern('test/files/_patterns/00-test/04-group.mustache', '00-test', '04-group.mustache');
+			groupPattern.template = fs.readFileSync(patterns_dir + '/00-test/04-group.mustache', 'utf8');
+			groupPattern.stylePartials = pattern_assembler.find_pattern_partials_with_style_modifiers(groupPattern);
+
+			pl.patterns.push(atomPattern);
+			pl.patterns.push(groupPattern);
+
+			//act
+			pattern_assembler.process_pattern_recursive('test/files/_patterns/00-test/04-group.mustache', pl, {});
+			//assert
+			var expectedValue = '<div class="test_group"> <span class="test_base test_1"> {{message}} </span> <span class="test_base test_2"> {{message}} </span> <span class="test_base test_3"> {{message}} </span> <span class="test_base test_4"> {{message}} </span> </div>';
+			test.equals(groupPattern.extendedTemplate.replace(/\s\s+/g, ' ').trim(), expectedValue.trim());
 			test.done();
 		}
 	};