jmug/do_more
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Added tests for the new methods in firestore provider

Browse source
This commit is contained in:
Mariano Uvalle 2019-03-02 23:41:11 -06:00
parent 79a5196aff
commit 0b06d86d0a
2 changed files with 35 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

4
lib/src/resources/firestore_provider.dart
View file

@ -61,6 +61,9 @@ class FirestoreProvider {
return querySnapshot.documents.length > 0; return querySnapshot.documents.length > 0;
} }
/// Updates a user's data in Firestore.
///
/// Updates are only pushed if at least one property is provided.
Future<void> updateUser( Future<void> updateUser(
String id, { String id, {
List<String> tasks, List<String> tasks,
@ -70,7 +73,6 @@ class FirestoreProvider {
int pendingLow, int pendingLow,
}) async { }) async {
final newData = <String, dynamic>{ final newData = <String, dynamic>{
'id': id,
'tasks': tasks, 'tasks': tasks,
'summary': summary, 'summary': summary,
'pendingHigh': pendingHigh, 'pendingHigh': pendingHigh,

38
test/src/resources/firestore_provider_test.dart
View file

@ -44,6 +44,32 @@ main() {
); );
group('FirestoreProvider', () { group('FirestoreProvider', () {
test('should create a user', () {
final firestore = MockFirestore();
final collection = MockCollectionReference();
final provider = FirestoreProvider(firestore);
when(firestore.collection('users')).thenReturn(collection);
provider.createUser(user);
verify(collection.add(user.toFirestoreMap()));
});
test('should update the information of a user', () {
final firestore = MockFirestore();
final collection = MockCollectionReference();
final document = MockDocumentReference();
final provider = FirestoreProvider(firestore);
when(firestore.collection('users')).thenReturn(collection);
when(collection.document(user.id)).thenReturn(document);
provider.updateUser(user.id, pendingHigh: 1);
verify(document.setData({'pendingHigh': 1}, merge: true));
});
test('should listen to updates of a single user object', () { test('should listen to updates of a single user object', () {
final firestore = MockFirestore(); final firestore = MockFirestore();
final collection = MockCollectionReference(); final collection = MockCollectionReference();
@ -63,7 +89,7 @@ main() {
expect(provider.getUser(user.username), emits(user)); expect(provider.getUser(user.username), emits(user));
}); });
test('should create task in firestore', () { test('should create task', () {
final firestore = MockFirestore(); final firestore = MockFirestore();
final collection = MockCollectionReference(); final collection = MockCollectionReference();
final provider = FirestoreProvider(firestore); final provider = FirestoreProvider(firestore);
@ -90,7 +116,7 @@ main() {
expectLater(provider.getTask('1'), emits(task)); expectLater(provider.getTask('1'), emits(task));
}); });
test('should delete a task from firestore', () { test('should delete a task', () {
final firestore = MockFirestore(); final firestore = MockFirestore();
final collection = MockCollectionReference(); final collection = MockCollectionReference();
final document = MockDocumentReference(); final document = MockDocumentReference();
@ -105,7 +131,7 @@ main() {
verify(document.delete()); verify(document.delete());
}); });
test('should update a task in firestore', () { test('should update a task', () {
final firestore = MockFirestore(); final firestore = MockFirestore();
final collection = MockCollectionReference(); final collection = MockCollectionReference();
final document = MockDocumentReference(); final document = MockDocumentReference();
@ -163,7 +189,7 @@ main() {
}); });
}); });
test('should create event in firestore', () { test('should create event', () {
final firestore = MockFirestore(); final firestore = MockFirestore();
final collection = MockCollectionReference(); final collection = MockCollectionReference();
final provider = FirestoreProvider(firestore); final provider = FirestoreProvider(firestore);
@ -191,7 +217,7 @@ main() {
expectLater(provider.getEvent('123', event.id), emits(event)); expectLater(provider.getEvent('123', event.id), emits(event));
}); });
test('should delete an event from firestore', () { test('should delete an event', () {
final firestore = MockFirestore(); final firestore = MockFirestore();
final document = MockDocumentReference(); final document = MockDocumentReference();
final provider = FirestoreProvider(firestore); final provider = FirestoreProvider(firestore);
@ -205,7 +231,7 @@ main() {
verify(document.delete()); verify(document.delete());
}); });
test('should update an event in firestore', () { test('should update an event', () {
final firestore = MockFirestore(); final firestore = MockFirestore();
final document = MockDocumentReference(); final document = MockDocumentReference();
final provider = FirestoreProvider(firestore); final provider = FirestoreProvider(firestore);