Skip to content

Commit

Permalink
Simplify subscriber test. As discussed in SteveSanderson#69
Browse files Browse the repository at this point in the history
  • Loading branch information
jamesfoster committed Apr 19, 2012
1 parent 14166c0 commit 0dffab2
Showing 1 changed file with 3 additions and 16 deletions.
19 changes: 3 additions & 16 deletions spec/proxyDependentObservableBehaviors.js
Expand Up @@ -297,13 +297,11 @@ var generateProxyTests = function(useComputed) {
asyncTest('dependentObservable dependencies trigger subscribers', function() {
var obj = {
inner: {
inner2: {
dependency: 1
}
dependency: 1
}
};

var inner2 = function(data) {
var inner = function(data) {
var _this = this;
ko.mapping.fromJS(data, {}, _this);

Expand All @@ -317,17 +315,6 @@ var generateProxyTests = function(useComputed) {
});
};

var inner = function(data) {
var _this = this;
ko.mapping.fromJS(data, {
inner2: {
create: function(options) {
return new inner2(options.data);
}
}
}, _this);
};

var mapping = {
inner: {
create: function(options) {
Expand All @@ -337,7 +324,7 @@ var generateProxyTests = function(useComputed) {
};

var mapped = ko.mapping.fromJS(obj, mapping);
var i = mapped.inner.inner2;
var i = mapped.inner;
equal(i.evaluationCount, 0);
window.setTimeout(function() {
start();
Expand Down

0 comments on commit 0dffab2

Please sign in to comment.