From 8cf689c8db58452d295247a518744aee027059ad Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ali=20R=C4=B1za=20Kat?= <ali.kat@count.ly>
Date: Tue, 20 Aug 2024 18:52:39 +0300
Subject: [PATCH] Recording Different Values in Storage

---
 test/tests_storage.js | 75 ++++++++++++++++++++++---------------------
 1 file changed, 38 insertions(+), 37 deletions(-)

diff --git a/test/tests_storage.js b/test/tests_storage.js
index d883470..715a097 100644
--- a/test/tests_storage.js
+++ b/test/tests_storage.js
@@ -64,6 +64,38 @@ function validateDeviceId(deviceId, deviceIdType, expectedDeviceId, expectedDevi
     assert.equal(deviceIdType, expectedDeviceIdType);
     checkRequestsForT(rq, expectedDeviceIdType);
 }
+function recordValuesToStorageAndValidate() {
+    // Set values
+    var deviceIdType = cc.deviceIdTypeEnums.DEVELOPER_SUPPLIED;
+    storage.storeSet("cly_id", "SpecialDeviceId");
+    storage.storeSet("cly_id_type", deviceIdType);
+
+    // Set values with different data types
+    storage.storeSet("cly_count", 42);
+    storage.storeSet("cly_object", { key: "value" });
+    storage.storeSet("cly_null", null);
+
+    // Retrieve and assert values
+    assert.equal(storage.storeGet("cly_id"), "SpecialDeviceId");
+    assert.equal(storage.storeGet("cly_id_type"), deviceIdType);
+    assert.equal(storage.storeGet("cly_count"), 42);
+    assert.deepEqual(storage.storeGet("cly_object"), { key: "value" });
+    assert.equal(storage.storeGet("cly_null"), null);
+
+    // Remove specific items by overriding with null or empty array
+    storage.storeSet("cly_id", null);
+    storage.storeSet("cly_object", []);
+    assert.equal(storage.storeGet("cly_id"), null);
+    assert.deepEqual(storage.storeGet("cly_object"), []);
+
+    // Reset storage and check if it's empty again
+    storage.resetStorage();
+    assert.equal(storage.storeGet("cly_id"), undefined);
+    assert.equal(storage.storeGet("cly_id_type"), undefined);
+    assert.equal(storage.storeGet("cly_count"), undefined);
+    assert.equal(storage.storeGet("cly_object"), undefined);
+    assert.equal(storage.storeGet("cly_null"), undefined);
+}
 
 describe("Storage Tests", () => {
     it("1- Store Generated Device ID", (done) => {
@@ -202,18 +234,11 @@ describe("Storage Tests", () => {
 
     it("10- Recording to Storage with Default Storage Path /no-init", (done) => {
         storage.resetStorage();
-        // will set to default storage path
+
+        // Set to default storage path
         storage.setStoragePath();
         assert.equal(storage.getStoragePath(), "../data/");
-
-        // setting values
-        var deviceIdType = cc.deviceIdTypeEnums.DEVELOPER_SUPPLIED;
-        storage.storeSet("cly_id", "SpecialDeviceId");
-        storage.storeSet("cly_id_type", deviceIdType);
-
-        // retrieve values
-        assert.equal(storage.storeGet("cly_id"), "SpecialDeviceId");
-        assert.equal(storage.storeGet("cly_id_type"), deviceIdType);
+        recordValuesToStorageAndValidate();
         done();
     });
 
@@ -222,15 +247,7 @@ describe("Storage Tests", () => {
         // will set to default storage path
         storage.setStoragePath("../test/customStorageDirectory/");
         assert.equal(storage.getStoragePath(), "../test/customStorageDirectory/");
-
-        // setting values
-        var deviceIdType = cc.deviceIdTypeEnums.DEVELOPER_SUPPLIED;
-        storage.storeSet("cly_id", "SpecialDeviceId");
-        storage.storeSet("cly_id_type", deviceIdType);
-
-        // retrieve values
-        assert.equal(storage.storeGet("cly_id"), "SpecialDeviceId");
-        assert.equal(storage.storeGet("cly_id_type"), deviceIdType);
+        recordValuesToStorageAndValidate();
         done();
     });
 
@@ -239,15 +256,7 @@ describe("Storage Tests", () => {
         // will set to default storage path
         storage.setBulkDataPath();
         assert.equal(storage.getStoragePath(), "../bulk_data/");
-
-        // setting values
-        var deviceIdType = cc.deviceIdTypeEnums.DEVELOPER_SUPPLIED;
-        storage.storeSet("cly_id", "SpecialDeviceId");
-        storage.storeSet("cly_id_type", deviceIdType);
-
-        // retrieve values
-        assert.equal(storage.storeGet("cly_id"), "SpecialDeviceId");
-        assert.equal(storage.storeGet("cly_id_type"), deviceIdType);
+        recordValuesToStorageAndValidate();
         done();
     });
 
@@ -256,15 +265,7 @@ describe("Storage Tests", () => {
         // will set to default storage path
         storage.setBulkDataPath("../test/customStorageDirectory/");
         assert.equal(storage.getStoragePath(), "../test/customStorageDirectory/");
-
-        // setting values
-        var deviceIdType = cc.deviceIdTypeEnums.DEVELOPER_SUPPLIED;
-        storage.storeSet("cly_id", "SpecialDeviceId");
-        storage.storeSet("cly_id_type", deviceIdType);
-
-        // retrieve values
-        assert.equal(storage.storeGet("cly_id"), "SpecialDeviceId");
-        assert.equal(storage.storeGet("cly_id_type"), deviceIdType);
+        recordValuesToStorageAndValidate();
         done();
     });
 });