refactor: rename RCTMLN to MLRN (#519)

* refactor: rename package from com.maplibre to org.maplibre
* refactor: rename RCTMLN to MLRN
* refactor: align module prefix to MLRN
* fix: remove unused ios files
* refactor: cleanup java component names
* refactor: use org.maplibre.reactnative package name
* refactor: remove getIOSModuleName
* test: update expo sync fingerprint
* refactor: rename MLRNImageUtils

.gitignore

@@ -66,7 +66,7 @@ package.tgz
 ios/temp.zip
 ios/.framework_version
 ios/Pods/
-ios/RCTMLN.xcodeproj/xcshareddata/
+ios/MLRN.xcodeproj/xcshareddata/
 android/build/
 android/.gradle/
 android/.idea/

.sonarcloud.properties

@@ -1,5 +1,5 @@
 # Path to sources
-sonar.sources=javascript,android/src,ios/RCTMLN
+sonar.sources=javascript,android/src,ios/MLRN
 #sonar.exclusions=
 #sonar.inclusions=
 

__tests__/components/BackgroundLayer.test.js

@@ -9,7 +9,7 @@ describe("BackgroundLayer", () => {
       <BackgroundLayer id="requiredBackgroundLayerID" />,
     );
 
-    const backgroundLayer = queryByTestId("rctmlnBackgroundLayer");
+    const backgroundLayer = queryByTestId("mlrnBackgroundLayer");
     const { props } = backgroundLayer;
 
     expect(props.sourceID).toStrictEqual("DefaultSourceID");
@@ -30,7 +30,7 @@ describe("BackgroundLayer", () => {
     };
 
     const { queryByTestId } = render(<BackgroundLayer {...testProps} />);
-    const backgroundLayer = queryByTestId("rctmlnBackgroundLayer");
+    const backgroundLayer = queryByTestId("mlrnBackgroundLayer");
     const { props } = backgroundLayer;
 
     expect(props.id).toStrictEqual(testProps.id);

__tests__/components/CircleLayer.test.js

@@ -8,7 +8,7 @@ describe("CircleLayer", () => {
     const { queryByTestId } = render(
       <CircleLayer id="requiredCircleLayerID" />,
     );
-    const circleLayer = queryByTestId("rctmlnCircleLayer");
+    const circleLayer = queryByTestId("mlrnCircleLayer");
     const { props } = circleLayer;
 
     expect(props.sourceID).toStrictEqual("DefaultSourceID");
@@ -29,7 +29,7 @@ describe("CircleLayer", () => {
     };
 
     const { queryByTestId } = render(<CircleLayer {...customProps} />);
-    const circleLayer = queryByTestId("rctmlnCircleLayer");
+    const circleLayer = queryByTestId("mlrnCircleLayer");
     const { props } = circleLayer;
 
     expect(props.id).toStrictEqual(customProps.id);

__tests__/components/HeatmapLayer.test.js

@@ -8,7 +8,7 @@ describe("HeatmapLayer", () => {
     const { UNSAFE_getByType } = render(
       <HeatmapLayer id="requiredHeatmapLayerID" />,
     );
-    const heatmapLayer = UNSAFE_getByType("RCTMLNHeatmapLayer");
+    const heatmapLayer = UNSAFE_getByType("MLRNHeatmapLayer");
     const { props } = heatmapLayer;
     expect(props.sourceID).toStrictEqual("DefaultSourceID");
   });
@@ -27,7 +27,7 @@ describe("HeatmapLayer", () => {
       style: { visibility: "none" },
     };
     const { UNSAFE_getByType } = render(<HeatmapLayer {...testProps} />);
-    const { props } = UNSAFE_getByType("RCTMLNHeatmapLayer");
+    const { props } = UNSAFE_getByType("MLRNHeatmapLayer");
 
     expect(props.id).toStrictEqual(testProps.id);
     expect(props.sourceID).toStrictEqual(testProps.sourceID);

__tests__/components/Light.test.js

@@ -6,7 +6,7 @@ import Light from "../../src/components/Light";
 describe("Light", () => {
   test("renders correctly", () => {
     const { queryByTestId } = render(<Light />);
-    const light = queryByTestId("rctmlnLight");
+    const light = queryByTestId("mlrnLight");
     expect(light).toBeDefined();
   });
 
@@ -21,7 +21,7 @@ describe("Light", () => {
 
     const { queryByTestId } = render(<Light style={testStyles} />);
 
-    const customStyles = queryByTestId("rctmlnLight").props.reactStyle;
+    const customStyles = queryByTestId("mlrnLight").props.reactStyle;
     const { anchor } = customStyles;
     const { color } = customStyles;
     const { position } = customStyles;

__tests__/modules/location/locationManager.test.js

@@ -4,8 +4,8 @@ import LocationManager, {
   LocationModuleEventEmitter,
 } from "../../../src/modules/location/locationManager";
 
-const MapLibreGL = NativeModules.MLNModule;
-const MapLibreGLLocationManager = NativeModules.MLNLocationModule;
+const MapLibreRN = NativeModules.MLRNModule;
+const MLRNLocationModule = NativeModules.MLRNLocationModule;
 
 const location = {
   coords: {
@@ -37,16 +37,16 @@ describe("LocationManager", () => {
     describe("#getLastKnownLocation", () => {
       test("gets last known location from native locationManager if non available", async () => {
         jest
-          .spyOn(MapLibreGLLocationManager, "getLastKnownLocation")
+          .spyOn(MLRNLocationModule, "getLastKnownLocation")
           .mockImplementation(() => location);
 
         const lastKnownLocation = await locationManager.getLastKnownLocation();
 
         expect(lastKnownLocation).toStrictEqual(location);
         expect(locationManager._lastKnownLocation).toStrictEqual(location);
-        expect(
-          MapLibreGLLocationManager.getLastKnownLocation,
-        ).toHaveBeenCalledTimes(1);
+        expect(MLRNLocationModule.getLastKnownLocation).toHaveBeenCalledTimes(
+          1,
+        );
 
         locationManager._lastKnownLocation = null;
       });
@@ -58,9 +58,7 @@ describe("LocationManager", () => {
 
         expect(locationManager._lastKnownLocation).toStrictEqual(location);
 
-        expect(
-          MapLibreGLLocationManager.getLastKnownLocation,
-        ).not.toHaveBeenCalled();
+        expect(MLRNLocationModule.getLastKnownLocation).not.toHaveBeenCalled();
 
         // reset
         locationManager._lastKnownLocation = null;
@@ -69,7 +67,7 @@ describe("LocationManager", () => {
 
     describe("#addListener", () => {
       const myListener = jest.fn();
-      MapLibreGL.LocationCallbackName = { Update: "MapboxUserLocationUpdate" };
+      MapLibreRN.LocationCallbackName = { Update: "MapboxUserLocationUpdate" };
 
       afterEach(() => {
         locationManager._listeners = [];
@@ -100,7 +98,7 @@ describe("LocationManager", () => {
     });
 
     describe("#removeListener", () => {
-      MapLibreGLLocationManager.stop = jest.fn();
+      MLRNLocationModule.stop = jest.fn();
 
       test("removes selected listener", () => {
         // just two different functions
@@ -109,22 +107,22 @@ describe("LocationManager", () => {
 
         locationManager.addListener(listenerA);
         expect(locationManager._listeners).toStrictEqual([listenerA]);
-        expect(MapLibreGLLocationManager.stop).not.toHaveBeenCalled();
+        expect(MLRNLocationModule.stop).not.toHaveBeenCalled();
 
         locationManager.addListener(listenerB);
         expect(locationManager._listeners).toStrictEqual([
           listenerA,
           listenerB,
         ]);
-        expect(MapLibreGLLocationManager.stop).not.toHaveBeenCalled();
+        expect(MLRNLocationModule.stop).not.toHaveBeenCalled();
 
         locationManager.removeListener(listenerB);
         expect(locationManager._listeners).toStrictEqual([listenerA]);
-        expect(MapLibreGLLocationManager.stop).not.toHaveBeenCalled();
+        expect(MLRNLocationModule.stop).not.toHaveBeenCalled();
 
         locationManager.removeListener(listenerA);
         expect(locationManager._listeners).toStrictEqual([]);
-        expect(MapLibreGLLocationManager.stop).toHaveBeenCalledTimes(1);
+        expect(MLRNLocationModule.stop).toHaveBeenCalledTimes(1);
       });
     });
 
@@ -148,25 +146,25 @@ describe("LocationManager", () => {
     });
 
     describe("#start", () => {
-      jest.spyOn(MapLibreGLLocationManager, "start");
+      jest.spyOn(MLRNLocationModule, "start");
       jest.spyOn(LocationModuleEventEmitter, "addListener");
 
       afterEach(() => {
         locationManager._isListening = false;
       });
 
       test("starts native location manager and adds event emitter listener", () => {
-        MapLibreGL.LocationCallbackName = {
+        MapLibreRN.LocationCallbackName = {
           Update: "MapboxUserLocationUpdate",
         };
 
         expect(locationManager._isListening).toStrictEqual(false);
 
         locationManager.start();
 
-        expect(MapLibreGLLocationManager.start).toHaveBeenCalledTimes(1);
+        expect(MLRNLocationModule.start).toHaveBeenCalledTimes(1);
         expect(LocationModuleEventEmitter.addListener).toHaveBeenCalledWith(
-          MapLibreGL.LocationCallbackName.Update,
+          MapLibreRN.LocationCallbackName.Update,
           locationManager.onUpdate,
         );
 
@@ -176,8 +174,8 @@ describe("LocationManager", () => {
       test('passes "displacement"', () => {
         locationManager.start(5); // displacement 5meters
 
-        expect(MapLibreGLLocationManager.start).toHaveBeenCalledTimes(1);
-        expect(MapLibreGLLocationManager.start).toHaveBeenCalledWith(5);
+        expect(MLRNLocationModule.start).toHaveBeenCalledTimes(1);
+        expect(MLRNLocationModule.start).toHaveBeenCalledWith(5);
       });
 
       test("does not start when already listening", () => {
@@ -188,7 +186,7 @@ describe("LocationManager", () => {
 
         locationManager.start();
 
-        expect(MapLibreGLLocationManager.start).not.toHaveBeenCalled();
+        expect(MLRNLocationModule.start).not.toHaveBeenCalled();
         expect(LocationModuleEventEmitter.addListener).not.toHaveBeenCalled();
       });
     });
@@ -199,16 +197,16 @@ describe("LocationManager", () => {
         locationManager._isListening = true;
 
         // native location manager has no #stop exposed in tests?
-        MapLibreGLLocationManager.stop = jest.fn();
-        MapLibreGL.LocationCallbackName = {
+        MLRNLocationModule.stop = jest.fn();
+        MapLibreRN.LocationCallbackName = {
           Update: "MapboxUserLocationUpdate",
         };
 
         expect(locationManager._isListening).toStrictEqual(true);
 
         locationManager.stop();
 
-        expect(MapLibreGLLocationManager.stop).toHaveBeenCalledTimes(1);
+        expect(MLRNLocationModule.stop).toHaveBeenCalledTimes(1);
         expect(locationManager.subscription.remove).toHaveBeenCalled();
 
         expect(locationManager._isListening).toStrictEqual(false);
@@ -219,27 +217,25 @@ describe("LocationManager", () => {
         locationManager._isListening = false;
 
         // native location manager has no #stop exposed in tests?
-        MapLibreGLLocationManager.stop = jest.fn();
-        MapLibreGL.LocationCallbackName = {
+        MLRNLocationModule.stop = jest.fn();
+        MapLibreRN.LocationCallbackName = {
           Update: "MapboxUserLocationUpdate",
         };
 
         expect(locationManager._isListening).toStrictEqual(false);
 
         locationManager.stop();
 
-        expect(MapLibreGLLocationManager.stop).toHaveBeenCalledTimes(1);
+        expect(MLRNLocationModule.stop).toHaveBeenCalledTimes(1);
         expect(locationManager.subscription.remove).not.toHaveBeenCalled();
       });
     });
 
     describe("#setMinDisplacement", () => {
       test('calls native "setMinDisplacement"', () => {
-        MapLibreGLLocationManager.setMinDisplacement = jest.fn();
+        MLRNLocationModule.setMinDisplacement = jest.fn();
         locationManager.setMinDisplacement(5);
-        expect(
-          MapLibreGLLocationManager.setMinDisplacement,
-        ).toHaveBeenCalledWith(5);
+        expect(MLRNLocationModule.setMinDisplacement).toHaveBeenCalledWith(5);
       });
     });
 

__tests__/modules/offline/OfflinePack.test.js

@@ -20,7 +20,7 @@ describe("OfflinePack", () => {
 
   it("should resume pack download", () => {
     const spy = jest.spyOn(
-      NativeModules.MLNOfflineModule,
+      NativeModules.MLRNOfflineModule,
       "resumePackDownload",
     );
     const offlinePack = new OfflinePack(fakeNativePack);
@@ -30,15 +30,18 @@ describe("OfflinePack", () => {
   });
 
   it("should pause pack download", () => {
-    const spy = jest.spyOn(NativeModules.MLNOfflineModule, "pausePackDownload");
+    const spy = jest.spyOn(
+      NativeModules.MLRNOfflineModule,
+      "pausePackDownload",
+    );
     const offlinePack = new OfflinePack(fakeNativePack);
     offlinePack.pause();
     expect(spy).toHaveBeenCalled();
     spy.mockRestore();
   });
 
   it("should get pack status", () => {
-    const spy = jest.spyOn(NativeModules.MLNOfflineModule, "getPackStatus");
+    const spy = jest.spyOn(NativeModules.MLRNOfflineModule, "getPackStatus");
     const offlinePack = new OfflinePack(fakeNativePack);
     offlinePack.status();
     expect(spy).toHaveBeenCalled();

__tests__/modules/offline/offlineManager.test.js

@@ -71,7 +71,10 @@ describe("offlineManager", () => {
 
   it("should set max tile count limit", () => {
     const expectedLimit = 2000;
-    const spy = jest.spyOn(NativeModules.MLNOfflineModule, "setTileCountLimit");
+    const spy = jest.spyOn(
+      NativeModules.MLRNOfflineModule,
+      "setTileCountLimit",
+    );
     MapLibreGL.offlineManager.setTileCountLimit(expectedLimit);
     expect(spy).toHaveBeenCalledWith(expectedLimit);
     spy.mockRestore();
@@ -80,7 +83,7 @@ describe("offlineManager", () => {
   it("should set progress event throttle value", () => {
     const expectedThrottleValue = 500;
     const spy = jest.spyOn(
-      NativeModules.MLNOfflineModule,
+      NativeModules.MLRNOfflineModule,
       "setProgressEventThrottle",
     );
     MapLibreGL.offlineManager.setProgressEventThrottle(expectedThrottleValue);
@@ -194,7 +197,10 @@ describe("offlineManager", () => {
     beforeEach(() => (Platform.OS = "android"));
 
     it("should set pack observer manually", async () => {
-      const spy = jest.spyOn(NativeModules.MLNOfflineModule, "setPackObserver");
+      const spy = jest.spyOn(
+        NativeModules.MLRNOfflineModule,
+        "setPackObserver",
+      );
 
       const name = `test-${Date.now()}`;
       const noop = () => {};
@@ -207,7 +213,10 @@ describe("offlineManager", () => {
     });
 
     it("should not set pack observer manually during create flow", async () => {
-      const spy = jest.spyOn(NativeModules.MLNOfflineModule, "setPackObserver");
+      const spy = jest.spyOn(
+        NativeModules.MLRNOfflineModule,
+        "setPackObserver",
+      );
 
       const name = `test-${Date.now()}`;
       const noop = () => {};
@@ -223,7 +232,10 @@ describe("offlineManager", () => {
     beforeEach(() => (Platform.OS = "ios"));
 
     it("should not set pack observer manually", async () => {
-      const spy = jest.spyOn(NativeModules.MLNOfflineModule, "setPackObserver");
+      const spy = jest.spyOn(
+        NativeModules.MLRNOfflineModule,
+        "setPackObserver",
+      );
 
       const name = `test-${Date.now()}`;
       const noop = () => {};

__tests__/modules/snapshot/SnapshotOptions.test.js

@@ -14,7 +14,7 @@ describe("SnapshotOptions", () => {
     const options = new SnapshotOptions({ centerCoordinate });
 
     expect(options.toJSON()).toEqual({
-      styleURL: NativeModules.MLNModule.StyleURL.Default,
+      styleURL: NativeModules.MLRNModule.StyleURL.Default,
       heading: 0.0,
       pitch: 0.0,
       zoomLevel: 16.0,
@@ -36,7 +36,7 @@ describe("SnapshotOptions", () => {
       height: 600,
       writeToDisk: true,
       withLogo: true,
-      styleURL: NativeModules.MLNModule.StyleURL.Default,
+      styleURL: NativeModules.MLRNModule.StyleURL.Default,
     };
 
     const options = new SnapshotOptions(expectedOptions);
@@ -54,7 +54,7 @@ describe("SnapshotOptions", () => {
       ],
       width: 400,
       height: 600,
-      styleURL: NativeModules.MLNModule.StyleURL.Default,
+      styleURL: NativeModules.MLRNModule.StyleURL.Default,
       writeToDisk: false,
       withLogo: true,
     };