add tests for utils

removes unused dateToStringWithOffset
removes convertEnumCaseToValue

Author: fadi-george

lib/src/utils.dart

@@ -1,79 +1,5 @@
-import 'package:onesignal_flutter/src/defines.dart';
 import 'dart:convert';
 
-// produces a string like this: 2018-07-23T17:56:30.951030 UTC-7:00
-String dateToStringWithOffset(DateTime date) {
-  var offsetHours = date.timeZoneOffset.inHours;
-  var offsetMinutes = date.timeZoneOffset.inMinutes % 60;
-  var dateString = "${date.toIso8601String()} ";
-
-  dateString += "UTC" +
-      ((offsetHours > 10 || offsetHours < 0)
-          ? "$offsetHours"
-          : "0$offsetHours");
-  dateString += ":" +
-      ((offsetMinutes.abs() > 10) ? "$offsetMinutes" : "0$offsetMinutes:00");
-
-  return dateString;
-}
-
-// in some places, we want to send an enum value to
-// ObjC. Before we can do this, we must convert it
-// to a string/int/etc.
-// However, in some places such as iOS init settings,
-// there could be multiple different types of enum,
-// so we've combined it into this one function.
-dynamic convertEnumCaseToValue(dynamic key) {
-  switch (key) {
-    case OSiOSSettings.autoPrompt:
-      return "kOSSettingsKeyAutoPrompt";
-    case OSiOSSettings.inAppAlerts:
-      return "kOSSettingsKeyInAppAlerts";
-    case OSiOSSettings.inAppLaunchUrl:
-      return "kOSSettingsKeyInAppLaunchURL";
-    case OSiOSSettings.inFocusDisplayOption:
-      return "kOSSettingsKeyInFocusDisplayOption";
-    case OSiOSSettings.promptBeforeOpeningPushUrl:
-      return "kOSSSettingsKeyPromptBeforeOpeningPushURL";
-  }
-
-  switch (key) {
-    case OSCreateNotificationBadgeType.increase:
-      return "Increase";
-    case OSCreateNotificationBadgeType.setTo:
-      return "SetTo";
-  }
-
-  switch (key) {
-    case OSCreateNotificationDelayOption.lastActive:
-      return "last_active";
-    case OSCreateNotificationDelayOption.timezone:
-      return "timezone";
-  }
-
-  switch (key) {
-    case OSNotificationDisplayType.none:
-      return 0;
-    case OSNotificationDisplayType.alert:
-      return 1;
-    case OSNotificationDisplayType.notification:
-      return 2;
-  }
-
-  switch (key) {
-    case OSSession.DIRECT:
-      return "DIRECT";
-    case OSSession.INDIRECT:
-      return "INDIRECT";
-    case OSSession.UNATTRIBUTED:
-      return "UNATTRIBUTED";
-    case OSSession.DISABLED:
-      return "DISABLED";
-  }
-
-  return key;
-}
-
 /// An abstract class to provide JSON decoding
 abstract class JSONStringRepresentable {
   String jsonRepresentation();

test/utils_test.dart

@@ -0,0 +1,149 @@
+import 'package:flutter_test/flutter_test.dart';
+import 'package:onesignal_flutter/src/utils.dart';
+
+void main() {
+  group('JSONStringRepresentable', () {
+    test('convertToJsonString formats simple map correctly', () {
+      final testObj = TestJSONStringRepresentable();
+      final map = {'key': 'value', 'number': 42};
+
+      final result = testObj.convertToJsonString(map);
+
+      expect(result, contains('"key": "value"'));
+      expect(result, contains('"number": 42'));
+      expect(result, contains('\n'));
+    });
+
+    test('convertToJsonString formats nested map correctly', () {
+      final testObj = TestJSONStringRepresentable();
+      final map = {
+        'outer': {
+          'inner': 'value',
+          'nested': {'deep': 'data'}
+        }
+      };
+
+      final result = testObj.convertToJsonString(map);
+
+      expect(result, contains('"outer"'));
+      expect(result, contains('"inner": "value"'));
+      expect(result, contains('"nested"'));
+      expect(result, contains('"deep": "data"'));
+    });
+
+    test('convertToJsonString formats list correctly', () {
+      final testObj = TestJSONStringRepresentable();
+      final map = {
+        'items': [1, 2, 3],
+        'strings': ['a', 'b', 'c']
+      };
+
+      final result = testObj.convertToJsonString(map);
+
+      expect(result, contains('"items"'));
+      expect(result, contains('"strings"'));
+      expect(result, contains('['));
+      expect(result, contains(']'));
+    });
+
+    test('convertToJsonString handles null map', () {
+      final testObj = TestJSONStringRepresentable();
+
+      final result = testObj.convertToJsonString(null);
+
+      expect(result, equals('null'));
+    });
+
+    test('convertToJsonString handles empty map', () {
+      final testObj = TestJSONStringRepresentable();
+      final map = <String, dynamic>{};
+
+      final result = testObj.convertToJsonString(map);
+
+      expect(result, equals('{}'));
+    });
+
+    test('convertToJsonString removes escaped newlines', () {
+      final testObj = TestJSONStringRepresentable();
+      final map = {'text': 'line1\nline2'};
+
+      final result = testObj.convertToJsonString(map);
+
+      expect(result, contains('"text": "line1\nline2"'));
+    });
+
+    test('convertToJsonString removes escaped backslashes', () {
+      final testObj = TestJSONStringRepresentable();
+      final map = {'path': 'folder/file'};
+
+      final result = testObj.convertToJsonString(map);
+
+      expect(result, contains('"path": "folder/file"'));
+    });
+
+    test('convertToJsonString formats with proper indentation', () {
+      final testObj = TestJSONStringRepresentable();
+      final map = {
+        'level1': {'level2': 'value'}
+      };
+
+      final result = testObj.convertToJsonString(map);
+
+      expect(result, contains('  '));
+      final lines = result.split('\n');
+      expect(lines.length, greaterThan(1));
+    });
+
+    test('convertToJsonString handles boolean values', () {
+      final testObj = TestJSONStringRepresentable();
+      final map = {'isTrue': true, 'isFalse': false};
+
+      final result = testObj.convertToJsonString(map);
+
+      expect(result, contains('"isTrue": true'));
+      expect(result, contains('"isFalse": false'));
+    });
+
+    test('convertToJsonString handles numeric values', () {
+      final testObj = TestJSONStringRepresentable();
+      final map = {'integer': 42, 'double': 3.14, 'negative': -10};
+
+      final result = testObj.convertToJsonString(map);
+
+      expect(result, contains('42'));
+      expect(result, contains('3.14'));
+      expect(result, contains('-10'));
+    });
+
+    test('convertToJsonString handles mixed types', () {
+      final testObj = TestJSONStringRepresentable();
+      final map = {
+        'string': 'text',
+        'number': 123,
+        'bool': true,
+        'null': null,
+        'list': [1, 2],
+        'map': {'nested': 'value'}
+      };
+
+      final result = testObj.convertToJsonString(map);
+
+      expect(result, contains('"string": "text"'));
+      expect(result, contains('"number": 123'));
+      expect(result, contains('"bool": true'));
+      expect(result, contains('"null": null'));
+      expect(result, contains('"list":'));
+      expect(result, contains('"map":'));
+      expect(result, contains('"nested": "value"'));
+      expect(result, contains('{'));
+      expect(result, contains('}'));
+    });
+  });
+}
+
+class TestJSONStringRepresentable extends JSONStringRepresentable {
+  @override
+  String jsonRepresentation() {
+    return convertToJsonString({'test': 'data'});
+  }
+}