fix CI and improve test coverage

Author: sitatec

.github/workflows/main.yml

@@ -39,9 +39,7 @@ jobs:
         run: flutter test --coverage
 
       - name: Exclude set_up_mocks.dart form coverage
-        run: | 
-          dart pub global activate remove_from_coverage
-          remove_from_coverage -f coverage/lcov.info -r 'set_up_mocks\.dart$'
+        run: dart pub global activate remove_from_coverage && remove_from_coverage -f coverage/lcov.info -r 'set_up_mocks\.dart$'
 
       - name: Upload coverage to Codecov 
         uses: codecov/codecov-action@v1 

lib/src/mock_database_reference.dart

@@ -43,7 +43,7 @@ class MockDatabaseReference extends Mock implements DatabaseReference {
     path = (_nodePath + path).replaceAll(RegExp(r'^/+'), '');
     return MockDatabaseReference._(
       path,
-      MockFirebaseDatabase.persistData ? _volatileData : <String, dynamic>{},
+      _data,
     );
   }
 

test/mock_database_reference_test.dart

@@ -178,7 +178,12 @@ void main() {
   // group('Set data at any node reference :', null);
 
   group('Data persistence : ', () {
+    tearDown(() {
+      MockFirebaseDatabase.setDataPersistanceEnabled(ennabled: true);
+    });
+
     test('Should persist data while test running', () async {
+      MockFirebaseDatabase.setDataPersistanceEnabled(ennabled: true);
       var databaseReference = MockDatabaseReference();
       await databaseReference.child('test1').set('value1');
       await databaseReference.child('test2/test2').set('value2');
@@ -201,30 +206,29 @@ void main() {
         equals('value3'),
       );
     });
-  });
-
-  test('Should return a stream of data', () async {
-    await databaseReference.child('streamTest').set('StreamVal');
-    final stream = databaseReference.child('streamTest').onValue;
-    expect((await stream.first).snapshot.value, equals('StreamVal'));
-  });
-
-  group('Work with data persistance disabled : ', () {
-    // MockFirebaseDatabase.setDataPersistanceEnabled(ennabled: false);
-    final _databaseReference = MockDatabaseReference();
-    test('Should set String', () async {
-      await _databaseReference.child('test_').set('value');
+    test('Should not persist data', () async {
+      MockFirebaseDatabase.setDataPersistanceEnabled(ennabled: false);
+      await databaseReference.child('test_').set('value');
       expect(
-        (await _databaseReference.child('test_').once()).value,
+        (await databaseReference.child('test_').once()).value,
         equals('value'),
       );
-      await _databaseReference.child('otherTest_/test').set('otherValue');
+      await databaseReference.child('otherTest_/test').set('otherValue');
       expect(
-        (await _databaseReference.child('otherTest_/test').once()).value,
+        (await databaseReference.child('otherTest_/test').once()).value,
         equals('otherValue'),
       );
+      expect(
+          (await MockDatabaseReference().child('test_').once()).value, isNull);
     });
   });
+
+  test('Should return a stream of data', () async {
+    await databaseReference.child('streamTest').set('StreamVal');
+    final stream = databaseReference.child('streamTest').onValue;
+    expect((await stream.first).snapshot.value, equals('StreamVal'));
+  });
+
   // Todo implement all dataSnapshot, dbReference and fbDatabase getters and setters if possible.
 
   // test(