Permalink
Browse files

fix tests, add/on dupe copy/paste bug

  • Loading branch information...
1 parent 5a407b1 commit 649a7848418245e1131377a4529701a81bf920be @rvagg rvagg committed Nov 15, 2012
Showing with 21 additions and 22 deletions.
  1. +0 −1 .jshintrc
  2. +17 −17 tests/add-test.js
  3. +4 −4 tests/delegate-test.js
View
@@ -15,7 +15,6 @@
, "undef": true
, "sub": true
, "white": false
- , "indent": 2
, "asi": true
, "laxbreak": true
, "eqnull": true
View
@@ -1,4 +1,4 @@
-/*global bean:true, buster:true, Syn:true, assert:true, SpyTrigger:true, defer:true, features:true, globalSetUp:true, globalTearDown:true*/
+/*global bean:true, buster:true, Syn:true, assert:true, defer:true, features:true, globalSetUp:true, globalTearDown:true*/
buster.testCase('add', {
'setUp': globalSetUp
@@ -15,7 +15,7 @@ buster.testCase('add', {
}
}
, 'on()': function () { this.runTest('on') }
- , 'add()': function () { this.runTest('on') }
+ , 'add()': function () { this.runTest('add') }
}
, 'should be able to add single events to elements': {
@@ -37,7 +37,7 @@ buster.testCase('add', {
}
}
, 'on()': function (done) { this.runTest(done, 'on') }
- , 'add()': function (done) { this.runTest(done, 'on') }
+ , 'add()': function (done) { this.runTest(done, 'add') }
}
, 'should be able to add single events to objects': {
@@ -60,7 +60,7 @@ buster.testCase('add', {
}
}
, 'on()': function (done) { this.runTest(done, 'on') }
- , 'add()': function (done) { this.runTest(done, 'on') }
+ , 'add()': function (done) { this.runTest(done, 'add') }
}
, 'scope should be equal to element': {
@@ -83,7 +83,7 @@ buster.testCase('add', {
}
}
, 'on()': function (done) { this.runTest(done, 'on') }
- , 'add()': function (done) { this.runTest(done, 'on') }
+ , 'add()': function (done) { this.runTest(done, 'add') }
}
, 'should recieve an event method': {
@@ -107,7 +107,7 @@ buster.testCase('add', {
}
}
, 'on()': function (done) { this.runTest(done, 'on') }
- , 'add()': function (done) { this.runTest(done, 'on') }
+ , 'add()': function (done) { this.runTest(done, 'add') }
}
, 'should be able to pass x amount of additional arguments': {
@@ -133,7 +133,7 @@ buster.testCase('add', {
}
}
, 'on()': function (done) { this.runTest(done, 'on') }
- , 'add()': function (done) { this.runTest(done, 'on') }
+ , 'add()': function (done) { this.runTest(done, 'add') }
}
, 'should be able to add multiple events by space seperating them': {
@@ -155,7 +155,7 @@ buster.testCase('add', {
}
}
, 'on()': function (done) { this.runTest(done, 'on') }
- , 'add()': function (done) { this.runTest(done, 'on') }
+ , 'add()': function (done) { this.runTest(done, 'add') }
}
, 'should be able to add multiple events of the same type': {
@@ -183,7 +183,7 @@ buster.testCase('add', {
}
}
, 'on()': function (done) { this.runTest(done, 'on') }
- , 'add()': function (done) { this.runTest(done, 'on') }
+ , 'add()': function (done) { this.runTest(done, 'add') }
}
, 'should be able to add multiple events simultaneously with an object literal': {
@@ -207,7 +207,7 @@ buster.testCase('add', {
}
}
, 'on()': function (done) { this.runTest(done, 'on') }
- , 'add()': function (done) { this.runTest(done, 'on') }
+ , 'add()': function (done) { this.runTest(done, 'add') }
}
, 'should bubble up dom': {
@@ -230,7 +230,7 @@ buster.testCase('add', {
}
}
, 'on()': function (done) { this.runTest(done, 'on') }
- , 'add()': function (done) { this.runTest(done, 'on') }
+ , 'add()': function (done) { this.runTest(done, 'add') }
}
, 'shouldn\'t trigger event when adding additional custom event listeners': {
@@ -250,7 +250,7 @@ buster.testCase('add', {
}
}
, 'on()': function (done) { this.runTest(done, 'on') }
- , 'add()': function (done) { this.runTest(done, 'on') }
+ , 'add()': function (done) { this.runTest(done, 'add') }
}
, 'should bind onmessage to window': {
@@ -287,13 +287,13 @@ buster.testCase('add', {
}
}
, 'on()': function (done) { this.runTest(done, 'on') }
- , 'add()': function (done) { this.runTest(done, 'on') }
+ , 'add()': function (done) { this.runTest(done, 'add') }
}
, 'one: should only trigger handler once': {
'setUp': function () {
var self = this
- this.runTest = function (done, method) {
+ this.runTest = function (done) {
var el = self.byId('input')
, trigger = self.trigger()
, spy = self.spy()
@@ -310,13 +310,13 @@ buster.testCase('add', {
}
}
, 'on()': function (done) { this.runTest(done, 'on') }
- , 'add()': function (done) { this.runTest(done, 'on') }
+ , 'add()': function (done) { this.runTest(done, 'add') }
}
, 'one: should be removable': {
'setUp': function () {
var self = this
- this.runTest = function (done, method) {
+ this.runTest = function (done) {
var el = self.byId('input')
, spy = self.spy()
@@ -332,6 +332,6 @@ buster.testCase('add', {
}
}
, 'on()': function (done) { this.runTest(done, 'on') }
- , 'add()': function (done) { this.runTest(done, 'on') }
+ , 'add()': function (done) { this.runTest(done, 'add') }
}
})
View
@@ -151,7 +151,7 @@ buster.testCase('delegate', {
}
, 'on()': function (done) {
this.runTest(done, function (el1, wrappedSpy) {
- bean.add(el1, '.bar', 'click', wrappedSpy)
+ bean.on(el1, 'click', '.bar', wrappedSpy)
})
}
, 'add()': function (done) {
@@ -192,7 +192,7 @@ buster.testCase('delegate', {
}
, 'on()': function (done) {
this.runTest(done, function (el1, wrappedSpy) {
- bean.add(el1, '.bar', 'click', wrappedSpy)
+ bean.on(el1, 'click', '.bar', wrappedSpy)
})
}
, 'add()': function (done) {
@@ -233,7 +233,7 @@ buster.testCase('delegate', {
}
, 'on()': function (done) {
this.runTest(done, function (el1, wrappedSpy) {
- bean.add(el1, '.bar', 'click', wrappedSpy)
+ bean.on(el1, 'click', '.bar', wrappedSpy)
})
}
, 'add()': function (done) {
@@ -280,7 +280,7 @@ buster.testCase('delegate', {
}
, 'on()': function (done) {
this.runTest(done, function (el1, selector, wrappedSpy) {
- bean.add(el1, selector, 'click', wrappedSpy)
+ bean.on(el1, 'click', selector, wrappedSpy)
})
}
, 'add()': function (done) {

0 comments on commit 649a784

Please sign in to comment.