diff options
| author | Keyamoon | 2012-12-20 03:18:47 +0330 | 
|---|---|---|
| committer | Igor Minar | 2013-01-08 14:54:15 -0800 | 
| commit | 76a6047af690781b8238ba7924279470ba76d081 (patch) | |
| tree | 1fc16cac57f8fac4ce9f6b84a7dacd0f76d30544 /test/jqLiteSpec.js | |
| parent | b6b7c5a1d66073937709158da8c2d688cb45c9f6 (diff) | |
| download | angular.js-76a6047af690781b8238ba7924279470ba76d081.tar.bz2 | |
fix(jqLite): make next() ignore non-element nodes
next() is supposed to return the next sibling *element* so it
should ignore text nodes. To achieve this, nextElementSibling()
should be used instead of nextSibling().
Diffstat (limited to 'test/jqLiteSpec.js')
| -rw-r--r-- | test/jqLiteSpec.js | 9 | 
1 files changed, 8 insertions, 1 deletions
| diff --git a/test/jqLiteSpec.js b/test/jqLiteSpec.js index 01f9b9ae..b23e23eb 100644 --- a/test/jqLiteSpec.js +++ b/test/jqLiteSpec.js @@ -1,4 +1,3 @@ -  describe('jqLite', function() {    var scope, a, b, c; @@ -1073,6 +1072,14 @@ describe('jqLite', function() {        var i = element.find('i');        expect(b.next()).toJqEqual([i]);      }); + + +    it('should ignore non-element siblings', function() { +      var element = jqLite('<div><b>b</b>TextNode<!-- comment node --><i>i</i></div>'); +      var b = element.find('b'); +      var i = element.find('i'); +      expect(b.next()).toJqEqual([i]); +    });    }); | 
