diff options
| -rw-r--r-- | src/Angular.js | 4 | ||||
| -rw-r--r-- | test/AngularSpec.js | 13 |
2 files changed, 15 insertions, 2 deletions
diff --git a/src/Angular.js b/src/Angular.js index 691ddbc6..478ef2a2 100644 --- a/src/Angular.js +++ b/src/Angular.js @@ -565,9 +565,9 @@ var trim = (function() { * @returns {boolean} True if `value` is a DOM element (or wrapped jQuery element). */ function isElement(node) { - return node && + return !!(node && (node.nodeName // we are a direct element - || (node.on && node.find)); // we have an on and find method part of jQuery API + || (node.on && node.find))); // we have an on and find method part of jQuery API } /** diff --git a/test/AngularSpec.js b/test/AngularSpec.js index 1b08a18e..36d4926e 100644 --- a/test/AngularSpec.js +++ b/test/AngularSpec.js @@ -1079,4 +1079,17 @@ describe('angular', function() { } }); + describe('isElement', function() { + it('should return a boolean value', inject(function($compile, $document, $rootScope) { + var element = $compile('<p>Hello, world!</p>')($rootScope), + body = $document.find('body')[0], + expected = [false, false, false, false, false, false, false, true, true], + tests = [null, undefined, "string", 1001, {}, 0, false, body, element]; + angular.forEach(tests, function(value, idx) { + var result = angular.isElement(value); + expect(typeof result).toEqual('boolean'); + expect(result).toEqual(expected[idx]); + }); + })); + }); }); |