fix: Integration tests

aerogear · May 13, 2019 · 0270d68 · 0270d68
1 parent 11309d8
commit 0270d68
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 7 deletions.
diff --git a/packages/sync/integration_test/test/offline.test.js b/packages/sync/integration_test/test/offline.test.js
@@ -73,7 +73,7 @@ describe('Offline mutations', function () {
       } catch (ignore) { }
 
 
-      const offlineKeys = await JSON.parse(store.getItem(offlineMetaKey));
+      const offlineKeys = await store.getItem(offlineMetaKey);
       const offlineMutation = await store.getItem("offline:" + offlineKeys[0]);
 
       expect(offlineMutation).to.exist;
@@ -146,7 +146,7 @@ describe('Offline mutations', function () {
         });
       } catch (ignore) { }
 
-      const offlineKeys = await JSON.parse(store.getItem(offlineMetaKey));
+      const offlineKeys = await store.getItem(offlineMetaKey);
       const offlineMutation1 = await store.getItem("offline:" + offlineKeys[0]);
       const offlineMutation2 = await store.getItem("offline:" + offlineKeys[1]);
 
@@ -349,7 +349,7 @@ describe('Offline mutations', function () {
         });
       } catch (ignore) { }
 
-      const offlineKeys = await JSON.parse(store.getItem(offlineMetaKey));
+      const offlineKeys = await store.getItem(offlineMetaKey);
       const offlineMutation1 = await store.getItem("offline:" + offlineKeys[0]);
       const offlineMutation2 = await store.getItem("offline:" + offlineKeys[1]);
 
@@ -409,7 +409,7 @@ describe('Offline mutations', function () {
         });
       } catch (ignore) { }
 
-      const offlineKeys = await JSON.parse(store.getItem(offlineMetaKey));
+      const offlineKeys = await store.getItem(offlineMetaKey);
       const offlineMutation1 = await store.getItem("offline:" + offlineKeys[0]);
       const offlineMutation2 = await store.getItem("offline:" + offlineKeys[1]);
 
@@ -479,7 +479,7 @@ describe('Offline mutations', function () {
         });
       } catch (ignore) { }
 
-      const offlineKeys = await JSON.parse(store.getItem(offlineMetaKey));
+      const offlineKeys = await store.getItem(offlineMetaKey);
       const offlineMutation = await store.getItem("offline:" + offlineKeys[0]);
       expect(offlineMutation).to.exist;
       networkStatus.setOnline(true);

diff --git a/packages/sync/integration_test/utils/testStore.js b/packages/sync/integration_test/utils/testStore.js
@@ -4,14 +4,16 @@ export class TestStore {
   }
 
   getItem(key) {
-    return this.data[key];
+    return Promise.resolve(this.data[key]);
   }
 
   setItem(key, data) {
     this.data[key] = data;
+    return Promise.resolve();
   }
 
   removeItem(key) {
     delete this.data[key];
+    return Promise.resolve();
   }
 }
diff --git a/packages/sync/src/offline/OfflineMutationsHandler.ts b/packages/sync/src/offline/OfflineMutationsHandler.ts
@@ -34,7 +34,7 @@ export class OfflineMutationsHandler {
   public replayOfflineMutations = async () => {
     const offlineData = await this.store.getOfflineData();
     // if there is no offline data  then just exit
-    if (offlineData.length === 0) { return; }
+    if (offlineData && offlineData.length === 0) { return; }
 
     logger("Replying offline mutations after application restart");
     for (const item of offlineData) {