Skip to content

Commit

Permalink
spelling: dependent
Browse files Browse the repository at this point in the history
  • Loading branch information
jsoref committed Sep 24, 2017
1 parent 7ee8377 commit 8bed828
Showing 1 changed file with 13 additions and 13 deletions.
26 changes: 13 additions & 13 deletions spec/dependentObservableBehaviors.js
Original file line number Diff line number Diff line change
Expand Up @@ -145,11 +145,11 @@ describe('Dependent Observable', function() {

it('Should automatically update value when a dependency changes', function () {
var observable = new ko.observable(1);
var depedentObservable = ko.computed(function () { return observable() + 1; });
expect(depedentObservable()).toEqual(2);
var dependentObservable = ko.computed(function () { return observable() + 1; });
expect(dependentObservable()).toEqual(2);

observable(50);
expect(depedentObservable()).toEqual(51);
expect(dependentObservable()).toEqual(51);
});

it('Should be able to use \'peek\' on an observable to avoid a dependency', function() {
Expand Down Expand Up @@ -177,12 +177,12 @@ describe('Dependent Observable', function() {
var observableB = new ko.observable("B");
var observableToUse = "A";
var timesEvaluated = 0;
var depedentObservable = ko.computed(function () {
var dependentObservable = ko.computed(function () {
timesEvaluated++;
return observableToUse == "A" ? observableA() : observableB();
});

expect(depedentObservable()).toEqual("A");
expect(dependentObservable()).toEqual("A");
expect(timesEvaluated).toEqual(1);

// Changing an unrelated observable doesn't trigger evaluation
Expand All @@ -192,7 +192,7 @@ describe('Dependent Observable', function() {
// Switch to other observable
observableToUse = "B";
observableA("A2");
expect(depedentObservable()).toEqual("B2");
expect(dependentObservable()).toEqual("B2");
expect(timesEvaluated).toEqual(2);

// Now changing the first observable doesn't trigger evaluation
Expand All @@ -203,8 +203,8 @@ describe('Dependent Observable', function() {
it('Should notify subscribers of changes', function () {
var notifiedValue;
var observable = new ko.observable(1);
var depedentObservable = ko.computed(function () { return observable() + 1; });
depedentObservable.subscribe(function (value) { notifiedValue = value; });
var dependentObservable = ko.computed(function () { return observable() + 1; });
dependentObservable.subscribe(function (value) { notifiedValue = value; });

expect(notifiedValue).toEqual(undefined);
observable(2);
Expand All @@ -227,20 +227,20 @@ describe('Dependent Observable', function() {
it('Should notify "beforeChange" subscribers before changes', function () {
var notifiedValue;
var observable = new ko.observable(1);
var depedentObservable = ko.computed(function () { return observable() + 1; });
depedentObservable.subscribe(function (value) { notifiedValue = value; }, null, "beforeChange");
var dependentObservable = ko.computed(function () { return observable() + 1; });
dependentObservable.subscribe(function (value) { notifiedValue = value; }, null, "beforeChange");

expect(notifiedValue).toEqual(undefined);
observable(2);
expect(notifiedValue).toEqual(2);
expect(depedentObservable()).toEqual(3);
expect(dependentObservable()).toEqual(3);
});

it('Should only update once when each dependency changes, even if evaluation calls the dependency multiple times', function () {
var notifiedValues = [];
var observable = new ko.observable();
var depedentObservable = ko.computed(function () { return observable() * observable(); });
depedentObservable.subscribe(function (value) { notifiedValues.push(value); });
var dependentObservable = ko.computed(function () { return observable() * observable(); });
dependentObservable.subscribe(function (value) { notifiedValues.push(value); });
observable(2);
expect(notifiedValues.length).toEqual(1);
expect(notifiedValues[0]).toEqual(4);
Expand Down

0 comments on commit 8bed828

Please sign in to comment.