adding tests

mapbox · Sep 27, 2021 · 51715be · 51715be
1 parent 9403805
commit 51715be
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 3 deletions.
diff --git a/Sources/MapboxMaps/Offline/TileRegionLoadOptions+MapboxMaps.swift b/Sources/MapboxMaps/Offline/TileRegionLoadOptions+MapboxMaps.swift
@@ -32,11 +32,12 @@ extension TileRegionLoadOptions {
             }
         }
 
-        guard let geometry = geometry else {
-            return nil
+        var commonGeometry: MapboxCommon.Geometry?
+        if let geometry = geometry {
+            commonGeometry = MapboxCommon.Geometry(geometry: geometry)
         }
 
-        self.init(__geometry: MapboxCommon.Geometry(geometry: geometry),
+        self.init(__geometry: commonGeometry,
                   descriptors: descriptors.isEmpty ? nil : descriptors,
                   metadata: metadata,
                   acceptExpired: acceptExpired,

diff --git a/Tests/MapboxMapsTests/Foundation/Integration Tests/OfflineManagerIntegrationTests.swift b/Tests/MapboxMapsTests/Foundation/Integration Tests/OfflineManagerIntegrationTests.swift
@@ -522,4 +522,31 @@ internal class OfflineManagerIntegrationTestCase: IntegrationTestCase {
         closure = nil
         XCTAssertNil(weakTileStore)
     }
+
+    func testGeometryFromNonWrappedTileRegionLoadOptions() {
+        let nonWrappedTileRegionLoadOptions = TileRegionLoadOptions.init(__geometry: MapboxCommon.Geometry(geometry: .point(Point(self.tokyoCoord))),
+                                                                         descriptors: nil,
+                                                                         metadata: nil,
+                                                                         acceptExpired: false,
+                                                                         networkRestriction: .none,
+                                                                         start: nil,
+                                                                         averageBytesPerSecond: nil,
+                                                                         extraOptions: nil)
+
+        XCTAssertNotNil(nonWrappedTileRegionLoadOptions.geometry)
+        XCTAssertEqual(nonWrappedTileRegionLoadOptions.geometry!.type, .Point)
+
+        let geo = nonWrappedTileRegionLoadOptions.geometry!.value as! Point
+        XCTAssertEqual(geo, Point(self.tokyoCoord))
+    }
+
+    func testGeometryFromWrappedTileRegionLoadOptions() {
+        let wrappedTileRegionLoadOptions = TileRegionLoadOptions(geometry: .point(Point(self.tokyoCoord)),
+                                                                 descriptors: [])
+        XCTAssertNotNil(wrappedTileRegionLoadOptions?.geometry)
+        XCTAssertEqual(wrappedTileRegionLoadOptions?.geometry!.type, .Point)
+
+        let geo = wrappedTileRegionLoadOptions?.geometry!.value as! Point
+        XCTAssertEqual(geo, Point(self.tokyoCoord))
+    }
 }