From 0e5a24c584d88b17297010676b8e0005b3545efd Mon Sep 17 00:00:00 2001
From: Igor Minar
Date: Thu, 15 Sep 2011 00:47:15 +0200
Subject: 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
---
test/scenario/dslSpec.js | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
(limited to 'test/scenario/dslSpec.js')
diff --git a/test/scenario/dslSpec.js b/test/scenario/dslSpec.js
index 8e22e469..0334dd78 100644
--- a/test/scenario/dslSpec.js
+++ b/test/scenario/dslSpec.js
@@ -504,15 +504,15 @@ describe("angular.scenario.dsl", function() {
it('should toggle checkbox state', function() {
doc.append('');
expect(_jQuery('input[name="test.input"]').
- attr('checked')).toBeTruthy();
+ prop('checked')).toBe(true);
var chain = $root.dsl.input('test.input');
chain.check();
expect(_jQuery('input[name="test.input"]').
- attr('checked')).toBeFalsy();
+ prop('checked')).toBe(false);
$window.angular.reset();
chain.check();
expect(_jQuery('input[name="test.input"]').
- attr('checked')).toBeTruthy();
+ prop('checked')).toBe(true);
});
it('should return error if checkbox did not match', function() {
@@ -527,17 +527,17 @@ describe("angular.scenario.dsl", function() {
''
);
// HACK! We don't know why this is sometimes false on chrome
- _jQuery('input[name="0@test.input"][value="bar"]').attr('checked', true);
+ _jQuery('input[name="0@test.input"][value="bar"]').prop('checked', true);
expect(_jQuery('input[name="0@test.input"][value="bar"]').
- attr('checked')).toBeTruthy();
+ prop('checked')).toBe(true);
expect(_jQuery('input[name="0@test.input"][value="foo"]').
- attr('checked')).toBeFalsy();
+ prop('checked')).toBe(false);
var chain = $root.dsl.input('test.input');
chain.select('foo');
expect(_jQuery('input[name="0@test.input"][value="bar"]').
- attr('checked')).toBeFalsy();
+ prop('checked')).toBe(false);
expect(_jQuery('input[name="0@test.input"][value="foo"]').
- attr('checked')).toBeTruthy();
+ prop('checked')).toBe(true);
});
it('should return error if radio button did not match', function() {
--
cgit v1.2.3