Merge pull request #42 from kikihakiem/master

Fix text node with space character issue
searls · Apr 24, 2015 · 5694b50 · 5694b50
2 parents 3c93cdd + c6b5abb
commit 5694b50
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/app/js/jasmine-fixture.coffee b/app/js/jasmine-fixture.coffee
@@ -15,7 +15,7 @@
 
     create = (selectorOptions, attach) ->
       $top=null
-      _(selectorOptions.split(/[ ](?=[^\]]*?(?:\[|$))/)).inject(($parent, elementSelector) ->
+      _(selectorOptions.split(/[ ](?![^\{]*\})(?=[^\]]*?(?:\[|$))/)).inject(($parent, elementSelector) ->
         return $parent if elementSelector == ">"
         $el = createHTMLBlock($,elementSelector)
         $el.appendTo($parent) if attach || $top
@@ -289,7 +289,7 @@ createHTMLBlock = ( ->
   regId = /(?:^|\b)#([\w-!]+)/i
   regTagNotContent = /((([#\.]?[\w-]+)?(\[([\w!]+(="([^"]|\\")+")? {0,})+\])?)+)/i
   ###
-   See lookahead syntax (?!) at https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp 
+   See lookahead syntax (?!) at https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp
   ###
   regClasses = /(\.[\w-]+)(?!["\w])/g
   regClass = /\.([\w-]+)/i

diff --git a/spec/jasmine-fixture-spec.coffee b/spec/jasmine-fixture-spec.coffee
@@ -40,6 +40,10 @@ describe "jasmine.fixture", ->
       Given -> affix('a[data-target-pane="#pane-id"]')
       Then -> expect($('body')).not.toHas('#pane-id')
 
+    context "text node with space char", ->
+      Given -> affix('a.link{Text With Space}')
+      Then -> expect($('.link').text()).toEqual('Text With Space')
+
   describe ".create", ->
     Given -> @subject = jasmine.fixture
 
@@ -72,3 +76,7 @@ describe "jasmine.fixture", ->
       When -> @$result = @$container.affix('#content')
       Then -> expect(@$container).toHas('#content')
       And -> expect(@$result).toIs('#content')
+
+    context "text node with space char", ->
+      Given -> @result = @subject.create('a.link{Text With Space}')
+      Then -> @result.find('With').length == 0