Skip to content

Commit 3729559

Browse files
devcrustdevcrust
committed
- added sorting for all tests
1 parent e69249d commit 3729559

File tree

2 files changed

+38
-20
lines changed

2 files changed

+38
-20
lines changed

‎test/dependency/container.js

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -51,10 +51,10 @@ suite('Dependency Container', function ()
5151
{
5252

5353
// Check equal
54-
assert.deepEqual(container.getDependencies(), Object.keys(dependencies), 'Result does not match');
54+
assert.deepEqual(container.getDependencies().sort(), Object.keys(dependencies).sort(), 'Result does not match');
5555

5656
// Check equal
57-
assert.deepEqual(Object.keys(dependencies), container.getDependencies(), 'Result does not match');
57+
assert.deepEqual(Object.keys(dependencies).sort(), container.getDependencies().sort(), 'Result does not match');
5858

5959
});
6060

@@ -74,10 +74,10 @@ suite('Dependency Container', function ()
7474
assert.equal(container.hasDependency('http'), true, 'Dependency does not exist');
7575

7676
// Check equal
77-
assert.deepEqual(container.getDependencies(), Object.keys(dependencies2), 'Result does not match');
77+
assert.deepEqual(container.getDependencies().sort(), Object.keys(dependencies2).sort(), 'Result does not match');
7878

7979
// Check equal
80-
assert.deepEqual(Object.keys(dependencies2), container.getDependencies(), 'Result does not match');
80+
assert.deepEqual(Object.keys(dependencies2).sort(), container.getDependencies().sort(), 'Result does not match');
8181

8282
/*
8383
* +++++++++++++++
@@ -89,10 +89,10 @@ suite('Dependency Container', function ()
8989
assert.deepEqual(container.setDependencies(dependencies), container, 'Result does not match');
9090

9191
// Check equal
92-
assert.deepEqual(container.getDependencies(), Object.keys(dependencies), 'Result does not match');
92+
assert.deepEqual(container.getDependencies().sort(), Object.keys(dependencies).sort(), 'Result does not match');
9393

9494
// Check equal
95-
assert.deepEqual(Object.keys(dependencies), container.getDependencies(), 'Result does not match');
95+
assert.deepEqual(Object.keys(dependencies).sort(), container.getDependencies().sort(), 'Result does not match');
9696

9797
});
9898

@@ -115,10 +115,10 @@ suite('Dependency Container', function ()
115115
assert.equal(container.hasDependency('http'), true, 'Dependency does not exist');
116116

117117
// Check equal
118-
assert.deepEqual(container.getDependencies(), Object.keys(dependencies2), 'Result does not match');
118+
assert.deepEqual(container.getDependencies().sort(), Object.keys(dependencies2).sort(), 'Result does not match');
119119

120120
// Check equal
121-
assert.deepEqual(Object.keys(dependencies2), container.getDependencies(), 'Result does not match');
121+
assert.deepEqual(Object.keys(dependencies2).sort(), container.getDependencies().sort(), 'Result does not match');
122122

123123
/*
124124
* +++++++++++++++
@@ -133,10 +133,10 @@ suite('Dependency Container', function ()
133133
assert.deepEqual(container.addDependencies(dependencies), container, 'Result does not match');
134134

135135
// Check equal
136-
assert.deepEqual(container.getDependencies(), Object.keys(dependencies), 'Result does not match');
136+
assert.deepEqual(container.getDependencies().sort(), Object.keys(dependencies).sort(), 'Result does not match');
137137

138138
// Check equal
139-
assert.deepEqual(Object.keys(dependencies), container.getDependencies(), 'Result does not match');
139+
assert.deepEqual(Object.keys(dependencies).sort(), container.getDependencies().sort(), 'Result does not match');
140140

141141
});
142142

‎test/dips.js

Lines changed: 28 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -465,14 +465,17 @@ suite('Dips', function ()
465465
*/
466466

467467
// Check equal
468-
assert.deepEqual(dips.getDependencies(), (function ()
468+
assert.deepEqual(dips.getDependencies().sort(), (function ()
469469
{
470470

471471
var keys = Object.keys(dependencies);
472472

473473
// Add dips
474474
keys.push('dips');
475475

476+
// Sort keys
477+
keys.sort()
478+
476479
return keys;
477480

478481

@@ -485,14 +488,17 @@ suite('Dips', function ()
485488
*/
486489

487490
// Check equal
488-
assert.deepEqual(dips2.getDependencies(), (function ()
491+
assert.deepEqual(dips2.getDependencies().sort(), (function ()
489492
{
490493

491494
var keys = Object.keys(dependencies);
492495

493496
// Add dips
494497
keys.unshift('dips');
495498

499+
// Sort keys
500+
keys.sort()
501+
496502
return keys;
497503

498504

@@ -516,13 +522,13 @@ suite('Dips', function ()
516522
assert.equal(dips.hasDependency('http'), true, 'Dependency does not exist');
517523

518524
// Check equal
519-
assert.deepEqual(dips.getDependencies(), Object.keys(dependencies2), 'Result does not match');
525+
assert.deepEqual(dips.getDependencies().sort(), Object.keys(dependencies2).sort(), 'Result does not match');
520526

521527
// Set dependencies
522528
assert.deepEqual(dips.setDependencies(dependencies), dips, 'Result does not match');
523529

524530
// Check equal
525-
assert.deepEqual(dips.getDependencies(), Object.keys(dependencies), 'Result does not match');
531+
assert.deepEqual(dips.getDependencies().sort(), Object.keys(dependencies).sort(), 'Result does not match');
526532

527533
/*
528534
* ---------------
@@ -537,13 +543,13 @@ suite('Dips', function ()
537543
assert.equal(dips2.hasDependency('http'), true, 'Dependency does not exist');
538544

539545
// Check equal
540-
assert.deepEqual(dips2.getDependencies(), Object.keys(dependencies2), 'Result does not match');
546+
assert.deepEqual(dips2.getDependencies().sort(), Object.keys(dependencies2).sort(), 'Result does not match');
541547

542548
// Set dependencies
543549
assert.deepEqual(dips2.setDependencies(dependencies), dips2, 'Result does not match');
544550

545551
// Check equal
546-
assert.deepEqual(dips2.getDependencies(), Object.keys(dependencies), 'Result does not match');
552+
assert.deepEqual(dips2.getDependencies().sort(), Object.keys(dependencies).sort(), 'Result does not match');
547553

548554
});
549555

@@ -789,14 +795,17 @@ suite('Dips', function ()
789795
});
790796

791797
// Check equal
792-
assert.deepEqual(dips.getDependencies(), (function ()
798+
assert.deepEqual(dips.getDependencies().sort(), (function ()
793799
{
794800

795801
var keys = Object.keys(require('../lib/dependency/npm.js').getDependencies('npm'));
796802

797803
// Add dips
798804
keys.unshift('dips');
799805

806+
// Sort keys
807+
keys.sort()
808+
800809
return keys;
801810

802811

@@ -824,14 +833,17 @@ suite('Dips', function ()
824833
});
825834

826835
// Check equal
827-
assert.deepEqual(dips.getDependencies(), (function ()
836+
assert.deepEqual(dips.getDependencies().sort(), (function ()
828837
{
829838

830839
var keys = Object.keys(require('../lib/dependency/npm.js').getDependencies(undefined, ['mocha']));
831840

832841
// Add dips
833842
keys.unshift('dips');
834843

844+
// Sort keys
845+
keys.sort()
846+
835847
return keys;
836848

837849

@@ -877,7 +889,7 @@ suite('Dips', function ()
877889
});
878890

879891
// Check equal
880-
assert.deepEqual(dips.getDependencies(), (function ()
892+
assert.deepEqual(dips.getDependencies().sort(), (function ()
881893
{
882894

883895
var keys = Object.keys(extend(require('../lib/dependency/npm.js').getDependencies(),
@@ -886,6 +898,9 @@ suite('Dips', function ()
886898
// Add dips
887899
keys.unshift('dips');
888900

901+
// Sort keys
902+
keys.sort();
903+
889904
return keys;
890905

891906

@@ -919,7 +934,7 @@ suite('Dips', function ()
919934
});
920935

921936
// Check equal
922-
assert.deepEqual(dips.getDependencies(), (function ()
937+
assert.deepEqual(dips.getDependencies().sort(), (function ()
923938
{
924939

925940
var keys = Object.keys(extend(require('../lib/dependency/npm.js').getDependencies('npm'),
@@ -928,6 +943,9 @@ suite('Dips', function ()
928943
// Add dips
929944
keys.unshift('dips');
930945

946+
// Sort keys
947+
keys.sort();
948+
931949
return keys;
932950

933951

0 commit comments

Comments
 (0)