Skip to content
This repository was archived by the owner on Mar 17, 2025. It is now read-only.

Commit 7422ecf

Browse files
author
Jacob Wenger
committed
Merge pull request #521 from bendrucker/firebase-mock
Simplify MockFirebase.override call for mock setup
2 parents eb7750e + fa29b1f commit 7422ecf

File tree

6 files changed

+3
-12
lines changed

6 files changed

+3
-12
lines changed

tests/mocks/mocks.firebase.js

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1 @@
1-
2-
angular.module('mock.firebase', [])
3-
.run(function($window) {
4-
$window.MockFirebase.override();
5-
});
1+
MockFirebase.override();

tests/unit/FirebaseArray.spec.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@ describe('$FirebaseArray', function () {
3030

3131
var $firebase, $fb, $fbOldTodo, arr, $FirebaseArray, $utils, $rootScope, $timeout, destroySpy, testutils;
3232
beforeEach(function() {
33-
module('mock.firebase');
3433
module('firebase');
3534
module('testutils');
3635
inject(function ($firebase, _$FirebaseArray_, $firebaseUtils, _$rootScope_, _$timeout_, _testutils_) {

tests/unit/FirebaseAuth.spec.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ describe('FirebaseAuth',function(){
99
warn:[]
1010
};
1111

12-
module('mock.firebase');
1312
module('firebase',function($provide){
1413
$provide.value('$log',{
1514
warn:function(){
@@ -506,4 +505,4 @@ describe('FirebaseAuth',function(){
506505
expect(status).toEqual('resolved');
507506
});
508507
});
509-
});
508+
});

tests/unit/FirebaseObject.spec.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ describe('$FirebaseObject', function() {
1111
};
1212

1313
beforeEach(function () {
14-
module('mock.firebase');
1514
module('firebase');
1615
module('testutils');
1716
inject(function (_$firebase_, _$interval_, _$FirebaseObject_, _$timeout_, $firebaseUtils, _$rootScope_, _testutils_) {
@@ -615,4 +614,4 @@ describe('$FirebaseObject', function() {
615614
return offSpy;
616615
});
617616
}
618-
});
617+
});

tests/unit/firebase.spec.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ describe('$firebase', function () {
77

88
beforeEach(function() {
99
module('firebase');
10-
module('mock.firebase');
1110
module('mock.utils');
1211
// have to create these before the first call to inject
1312
// or they will not be registered with the angular mock injector

tests/unit/utils.spec.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
describe('$firebaseUtils', function () {
33
var $utils, $timeout, testutils;
44
beforeEach(function () {
5-
module('mock.firebase');
65
module('firebase');
76
module('testutils');
87
inject(function (_$firebaseUtils_, _$timeout_, _testutils_) {

0 commit comments

Comments
 (0)