Skip to content

Commit 0963df5

Browse files
committed
change proper variable name and clean the4 space in unit test
1 parent fa9b21f commit 0963df5

File tree

1 file changed

+7
-16
lines changed

1 file changed

+7
-16
lines changed

tests/javascript/binary-search-tree.js

Lines changed: 7 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@ var jsonify = function(object) {
2525

2626
var expect;
2727

28-
2928
describe('binary search tree', function() {
3029
it('root should pass a valid binary search tree', function() {
3130
expect = node(50,
@@ -75,7 +74,6 @@ describe('binary search tree', function() {
7574
searchResult = root.search(125);
7675
assert.ok(!searchResult);
7776

78-
7977
searchExpect = node(7, node(6));
8078
searchResult = root.search(7); // if existed
8179
assert.deepEqual(jsonify(searchResult), jsonify(searchExpect));
@@ -91,32 +89,25 @@ describe('binary search tree', function() {
9189
});
9290

9391
it('delete root', function() {
94-
var searchExpect;
92+
var rootExpect;
9593

9694
root.delete(50);
97-
searchExpect = node(10, node(8, node(7, node(6))),
95+
rootExpect = node(10, node(8, node(7, node(6))),
9896
node(100, undefined, node(150, undefined, node(175))));
99-
assert.deepEqual(jsonify(root), jsonify(searchExpect));
97+
assert.deepEqual(jsonify(root), jsonify(rootExpect));
10098

10199
//remove left sub tree
102100
root.delete(10).delete(8).delete(7).delete(6);
103-
searchExpect = node(100, undefined, node(150, undefined, node(175)));
104-
assert.deepEqual(jsonify(root), jsonify(searchExpect));
101+
rootExpect = node(100, undefined, node(150, undefined, node(175)));
102+
assert.deepEqual(jsonify(root), jsonify(rootExpect));
105103

106104
//remove right sub tree
107105
root.delete(175).delete(150).add(6).add(5).add(8).add(7).delete(100);
108-
searchExpect = node(6, node(5), node(8, node(7)));
109-
assert.deepEqual(jsonify(root), jsonify(searchExpect));
106+
rootExpect = node(6, node(5), node(8, node(7)));
107+
assert.deepEqual(jsonify(root), jsonify(rootExpect));
110108

111109
//remove every node.
112110
root.delete(6).delete(5).delete(8).delete(7);
113-
114111
assert.equal(undefined, root.value);
115-
116-
117-
118112
});
119-
120-
121-
122113
});

0 commit comments

Comments
 (0)