few fixes to make tests pass with jquery

mhevery · mhevery · commit ee327a1f4f75 · 2010-04-06T14:04:08.000-07:00
diff --git a/example/widgets.html b/example/widgets.html
diff --git a/scenario/widgets.html b/scenario/widgets.html
@@ -2,7 +2,7 @@
 <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
   <head>
     <link rel="stylesheet" type="text/css" href="style.css"></link>
-    <!--<script type="text/javascript" src="../lib/jquery/jquery-1.4.2.js"></script>-->
+    <script type="text/javascript" src="../lib/jquery/jquery-1.4.2.js"></script>
     <script type="text/javascript" src="../src/angular-bootstrap.js#autobind"></script>
   </head>
   <body ng-init="$window.$scope = this">
@@ -82,6 +82,16 @@
         </td>
         <td>button={{button}}</td>
       </tr>
+      <tr><th colspan="3">Repeaters</th></tr>
+      <tr>
+        <td>ng-repeat</td>
+        <td>
+          <ul>
+            <li ng-repeat="name in ['misko', 'adam']">{{name}}</li>
+          </ul>
+        </td>
+        <td></td>
+      </tr>
     </table>
   </body>
  </html>
diff --git a/src/jqLite.js b/src/jqLite.js
@@ -92,6 +92,7 @@ JQLite.prototype = {
     });
   },
 
+  //TODO: remove
   trigger: function(type) {
     var evnt = document.createEvent('MouseEvent');
     evnt.initMouseEvent(type, true, true, window, 0, 0, 0, 0, 0, false, false, false, false, 0, null);
diff --git a/test/BinderTest.js b/test/BinderTest.js
@@ -706,13 +706,13 @@ BinderTest.prototype.testItShouldSelectTheCorrectRadioBox = function() {
   var female = jqLite(c.node[0].childNodes[0]);
   var male = jqLite(c.node[0].childNodes[1]);
 
-  female.click();
+  trigger(female, 'click');
   assertEquals("female", c.scope.sex);
   assertEquals(true, female[0].checked);
   assertEquals(false, male[0].checked);
   assertEquals("female", female.val());
 
-  male.click();
+  trigger(male, 'click');
   assertEquals("male", c.scope.sex);
   assertEquals(false, female[0].checked);
   assertEquals(true, male[0].checked);
diff --git a/test/CompilerSpec.js b/test/CompilerSpec.js
@@ -72,7 +72,7 @@ describe('compiler', function(){
     var scope = compile('<span hello="misko" stop="true"><span hello="adam"/></span>');
     expect(log).toEqual("hello misko");
   });
-
+  
   it('should allow creation of templates', function(){
     directives.duplicate = function(expr, element){
       element.replaceWith(document.createComment("marker"));
@@ -97,7 +97,7 @@ describe('compiler', function(){
       if (text == 'middle') {
         expect(textNode.text()).toEqual(text);
         parentNode.attr('hello', text);
-        textNode.text('replaced');
+        textNode[0].textContent = 'replaced';
       }
     });
     var scope = compile('before<span>middle</span>after');
diff --git a/test/directivesSpec.js b/test/directivesSpec.js
@@ -142,19 +142,19 @@ describe("directives", function(){
   it('should ng-show', function(){
     var scope = compile('<div ng-hide="hide"></div>');
     scope.$eval();
-    expect(element.css('display')).toEqual('');
+    expect(isVisible(element)).toEqual(true);
     scope.$set('hide', true);
     scope.$eval();
-    expect(element.css('display')).toEqual('none');
+    expect(isVisible(element)).toEqual(false);
   });
 
   it('should ng-hide', function(){
     var scope = compile('<div ng-show="show"></div>');
     scope.$eval();
-    expect(element.css('display')).toEqual('none');
+    expect(isVisible(element)).toEqual(false);
     scope.$set('show', true);
     scope.$eval();
-    expect(element.css('display')).toEqual('');
+    expect(isVisible(element)).toEqual(true);
   });
 
   it('should ng-controller', function(){
diff --git a/test/testabilityPatch.js b/test/testabilityPatch.js
@@ -27,6 +27,12 @@ extend(angular, {
 });
 
 
+function trigger(element, type) {
+  var evnt = document.createEvent('MouseEvent');
+  evnt.initMouseEvent(type, true, true, window, 0, 0, 0, 0, 0, false, false, false, false, 0, null);
+  (element[0] || element).dispatchEvent(evnt);
+}
+
 function sortedHtml(element) {
   var html = "";
   (function toString(node) {
diff --git a/test/widgetsSpec.js b/test/widgetsSpec.js
@@ -115,10 +115,10 @@ describe("input widget", function(){
   it('should type="checkbox"', function(){
     compile('<input type="checkbox" name="checkbox" checked ng-change="action = true"/>');
     expect(scope.checkbox).toEqual(true);
-    element.click();
+    trigger(element, 'click');
     expect(scope.checkbox).toEqual(false);
     expect(scope.action).toEqual(true);
-    element.click();
+    trigger(element, 'click');
     expect(scope.checkbox).toEqual(true);
   });
 
@@ -142,7 +142,7 @@ describe("input widget", function(){
     expect(b.checked).toEqual(true);
     expect(scope.clicked).not.toBeDefined();
 
-    jqLite(a).click();
+    trigger(a, 'click');
     expect(scope.chose).toEqual('A');
     expect(scope.clicked).toEqual(1);
   });
