fix static analysis issues

Author: vaind

lib/src/store.dart

@@ -130,7 +130,7 @@ class Store {
 
   /// Return an existing SyncClient associated with the store or null if not available.
   /// See [Sync.client()] to create one first.
-  SyncClient syncClient() => SyncClientsStorage[this];
+  SyncClient syncClient() => syncClientsStorage[this];
 
   /// The low-level pointer to this store.
   Pointer<Void> get ptr => _cStore;

lib/src/sync.dart

@@ -99,7 +99,7 @@ class SyncClient {
   void close() {
     final err = bindings.obx_sync_close(_cSync);
     _cSync = nullptr;
-    SyncClientsStorage.remove(_store);
+    syncClientsStorage.remove(_store);
     StoreCloseObserver.removeListener(_store, this);
     syncOrObserversExclusive.unmark(_store);
     checkObx(err);
@@ -233,12 +233,12 @@ class Sync {
   ///       Make sure the SyncClient is not destroyed and thus synchronization can keep running in the background.
   static SyncClient client(
       Store store, String serverUri, SyncCredentials creds) {
-    if (SyncClientsStorage.containsKey(store)) {
+    if (syncClientsStorage.containsKey(store)) {
       throw Exception('Only one sync client can be active for a store');
     }
     syncOrObserversExclusive.mark(store);
     final client = SyncClient(store, serverUri, creds);
-    SyncClientsStorage[store] = client;
+    syncClientsStorage[store] = client;
     StoreCloseObserver.addListener(store, client, client.close);
     return client;
   }

lib/src/util.dart

@@ -41,7 +41,7 @@ class StoreCloseObserver {
 }
 
 /// Global internal storage of sync clients - one client per store.
-final Map<Store, SyncClient> SyncClientsStorage = {};
+final Map<Store, SyncClient> syncClientsStorage = {};
 
 // Currently, either SyncClient or Observers can be used at the same time.
 // TODO: lift this condition after #142 is fixed.

test/sync_test.dart

@@ -41,7 +41,7 @@ void main() {
   test('Model Entity has sync enabled', () {
     final model = getObjectBoxModel().model;
     final entity =
-        model.entities.firstWhere((ModelEntity e) => e.name == "TestEntity");
+        model.entities.firstWhere((ModelEntity e) => e.name == 'TestEntity');
     expect(entity.hasFlag(OBXEntityFlag.SYNC_ENABLED), isTrue);
   });
 
@@ -70,11 +70,11 @@ void main() {
         final error = throwsA(predicate((Exception e) => e.toString().contains(
             'Using observers/query streams in combination with SyncClient is currently not supported')));
 
-        SyncClient c = createClient(store);
+        createClient(store);
         expect(() => env.box.query().build().findStream(), error);
       });
     });
-    
+
     test('SyncClient lifecycle', () {
       expect(store.syncClient(), isNull);