Skip to content

Commit 075ffe9

Browse files
committed
fix and test MockFirebaseDatabase.instance.ref()
1 parent bd766e3 commit 075ffe9

File tree

4 files changed

+19
-7
lines changed

4 files changed

+19
-7
lines changed

example/example.dart

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,13 +8,13 @@ class UserRepository {
88

99
Future<String?> getUserName(String userId) async {
1010
final userNameReference =
11-
firebaseDatabase.reference().child('users').child(userId).child('name');
11+
firebaseDatabase.ref().child('users').child(userId).child('name');
1212
final databaseEvent = await userNameReference.once();
1313
return databaseEvent.snapshot.value as String?;
1414
}
1515

1616
Future<Map<String, dynamic>?> getUser(String userId) async {
17-
final userNode = firebaseDatabase.reference().child('users/$userId');
17+
final userNode = firebaseDatabase.ref().child('users/$userId');
1818
final databaseEvent = await userNode.once();
1919
return databaseEvent.snapshot.value as Map<String, dynamic>?;
2020
}
@@ -40,7 +40,7 @@ void main() {
4040
}
4141
}
4242
};
43-
MockFirebaseDatabase.instance.reference().set(fakeData);
43+
MockFirebaseDatabase.instance.ref().set(fakeData);
4444
setUp(() {
4545
firebaseDatabase = MockFirebaseDatabase.instance;
4646
userRepository = UserRepository(firebaseDatabase);

lib/src/mock_firebase_database.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ class MockFirebaseDatabase extends Mock implements FirebaseDatabase {
1212
@override
1313
DatabaseReference ref([String? path]) {
1414
if (path != null) {
15-
return MockDatabaseReference()..child(path);
15+
return MockDatabaseReference().child(path);
1616
}
1717
return MockDatabaseReference();
1818
}

test/exemples_test.dart

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,13 +8,13 @@ class UserRepository {
88

99
Future<String?> getUserName(String userId) async {
1010
final userNameReference =
11-
firebaseDatabase.reference().child('users').child(userId).child('name');
11+
firebaseDatabase.ref().child('users').child(userId).child('name');
1212
final databaseEvent = await userNameReference.once();
1313
return databaseEvent.snapshot.value as String?;
1414
}
1515

1616
Future<Map<String, dynamic>?> getUser(String userId) async {
17-
final userNode = firebaseDatabase.reference().child('users/$userId');
17+
final userNode = firebaseDatabase.ref().child('users/$userId');
1818
final databaseEvent = await userNode.once();
1919
return databaseEvent.snapshot.value as Map<String, dynamic>?;
2020
}
@@ -40,7 +40,7 @@ void main() {
4040
}
4141
}
4242
};
43-
MockFirebaseDatabase.instance.reference().set(fakeData);
43+
MockFirebaseDatabase.instance.ref().set(fakeData);
4444
setUp(() {
4545
firebaseDatabase = MockFirebaseDatabase.instance;
4646
userRepository = UserRepository(firebaseDatabase);

test/mock_database_reference_test.dart

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,18 @@ void main() {
1111
});
1212

1313
group('Node path handling : ', () {
14+
test("Should get a reference with a initial path", () {
15+
expect(
16+
MockFirebaseDatabase.instance.ref("initialPath").path,
17+
MockDatabaseReference().child("initialPath").path,
18+
);
19+
});
20+
test("Should get reference using the deprecated reference() method", () {
21+
expect(
22+
MockFirebaseDatabase.instance.reference().child("deprecated").path,
23+
MockDatabaseReference().child("deprecated").path,
24+
);
25+
});
1426
test('Should work with slash as prefix', () {
1527
expect(
1628
databaseReference.child('/test').path,

0 commit comments

Comments
 (0)