Permalink
Browse files

Match stub function definitions in /app with args from /tests/app

Based on the discussion of #19, it makes the tests more "user friendly" to
have the arguments in the stub functions match those in the tests (except
in cases such as curryIt or useArguments, where the test depends on
proper use of unspecific arguments)
  • Loading branch information...
1 parent 6262901 commit be632e42c286ea86f122e5a5111d85ac37533d88 @kadamwhite kadamwhite committed Jun 17, 2012
Showing with 7 additions and 7 deletions.
  1. +1 −1 app/async.js
  2. +5 −5 app/functions.js
  3. +1 −1 app/modules.js
View
2 app/async.js
@@ -4,7 +4,7 @@ define(function() {
},
- manipulateRemoteData : function() {
+ manipulateRemoteData : function(url) {
}
};
View
10 app/functions.js
@@ -10,27 +10,27 @@ define(function() {
},
- functionFunction : function() {
+ functionFunction : function(str) {
},
- partial : function() {
+ partial : function(fn, str1, str2) {
},
useArguments : function() {
},
- callIt : function() {
+ callIt : function(fn) {
},
- curryIt : function() {
+ curryIt : function(fn) {
},
- makeClosures : function() {
+ makeClosures : function(arr, fn) {
}
};
View
2 app/modules.js
@@ -2,7 +2,7 @@ if (typeof define !== 'function') { var define = require('amdefine')(module); }
define(function() {
return {
- createModule : function() {
+ createModule : function(str1, str2) {
}
};

0 comments on commit be632e4

Please sign in to comment.