fix(specs): jQuery now returns attr name instead of true/false for special attrs

for special attrs like 'checked' or 'multiple', jquery now returns the name or undedefined.

e.g. foo.attr('checked') => 'checked' || undefined

The solution is a combination of updating our tests as well as switching
over to prop() instead which properly returns true/false

1 files changed, 7 insertions, 7 deletions

diff --git a/test/jqLiteSpec.js b/test/jqLiteSpec.js
index a7fded5c..b31b1664 100644
--- a/test/jqLiteSpec.js
+++ b/test/jqLiteSpec.js
@@ -150,18 +150,18 @@ describe('jqLite', function(){
       expect(jqLite(b).attr('prop')).toBeFalsy();
     });
 
-    it('should read special attributes as boolean', function(){
+    it('should read special attributes as strings', function(){
       var select = jqLite('<select>');
-      expect(select.attr('multiple')).toEqual(false);
-      expect(jqLite('<select multiple>').attr('multiple')).toEqual(true);
-      expect(jqLite('<select multiple="">').attr('multiple')).toEqual(true);
-      expect(jqLite('<select multiple="x">').attr('multiple')).toEqual(true);
+      expect(select.attr('multiple')).toBeUndefined();
+      expect(jqLite('<select multiple>').attr('multiple')).toBe('multiple');
+      expect(jqLite('<select multiple="">').attr('multiple')).toBe('multiple');
+      expect(jqLite('<select multiple="x">').attr('multiple')).toBe('multiple');
 
       select.attr('multiple', false);
-      expect(select.attr('multiple')).toEqual(false);
+      expect(select.attr('multiple')).toBeUndefined();
 
       select.attr('multiple', true);
-      expect(select.attr('multiple')).toEqual(true);
+      expect(select.attr('multiple')).toBe('multiple');
     });
 
     it('should return undefined for non-existing attributes', function() {