Merge pull request #333 from heap/develop

Dev -> Master

Author: semmerich-heap

CHANGELOG.md

@@ -15,6 +15,13 @@ __BEGIN_UNRELEASED__
 ### Security
 __END_UNRELEASED__
 
+## [0.22.0] - 2022-09-23
+### Added
+ - Added tests for sending bad data to the native bridge
+### Changed
+ - Changed track bridge to warn instead of throw an exception when bad data is sent.
+ - Upgraded iOS sdk from 8.2 to 9.0
+
 ## [0.21.0] - 2022-06-29
 
 ### Added

android/src/main/java/com/heapanalytics/reactnative/RNHeapLibraryModule.java

@@ -12,6 +12,8 @@
 import java.util.HashMap;
 import java.util.Map;
 
+import android.util.Log;
+
 public class RNHeapLibraryModule extends ReactContextBaseJavaModule {
 
   private final ReactApplicationContext reactContext;
@@ -81,7 +83,7 @@ private static Map<String, String> convertToStringMap(ReadableMap readableMap) {
         // The JS bridge will flatten maps and arrays in a uniform manner across both
         // platforms.
         // If we get them at this point, we shouldn't continue.
-        throw new RNHeapException("Property objects must be flattened before being sent across the JS bridge.");
+        Log.w("Property objects must be flattened before being sent across the JS bridge. If you get this warning please inspect for non-flattenable objects being sent to Heap");
       }
     }
     return stringMap;

integration-tests/src/screens/PropertiesScreen.tsx

@@ -40,6 +40,12 @@ export const PropertiesScreen = ({navigation}) => {
           Heap.track('getSessionId', {value: await Heap.getSessionId()})
         }
       />
+      <Button
+        title="Send Custom Event"
+        onPress={async () =>
+          Heap.track('custom-event', { a: {b: null, c: [1,2,3], d: new (class Test {}) } })
+        }
+      />
       <Button title="Back" onPress={() => navigation.goBack()} />
     </ScrollView>
   );

integration-tests/tests/heap_properties.e2e.js

@@ -63,4 +63,17 @@ describe('Heap Static Properties', () => {
       );
     });
   });
+
+  describe('track', () => {
+    it('Should serialize properties without crashing', async () => {
+      await tapButton('Send Custom Event');
+
+      const message = await tools.server.expectSourceCustomEventWithProperties(
+        'custom-event',{
+        'a.c.0': '1',
+        'a.c.1': '2',
+        'a.c.1': '3'}
+      )
+    })
+  })
 });

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@heap/react-native-heap",
-  "version": "0.21.0",
+  "version": "0.22.0",
   "description": "React Native event tracking with Heap.",
   "license": "MIT",
   "author": "Heap <http://www.heapanalytics.com>",

package.json

@@ -15,7 +15,7 @@ Pod::Spec.new do |s|
   s.platform = :ios, "10.0"
 
   s.dependency "React"
-  s.dependency "Heap", "~> 8.2"
+  s.dependency "Heap", "~> 9.0"
 
   s.frameworks = "SystemConfiguration"