Fixed linter errors

mapbox · Sep 29, 2021 · 70cc847 · 70cc847
1 parent 2db44f9
commit 70cc847
Show file tree

Hide file tree

Showing 5 changed files with 16 additions and 8 deletions.
diff --git a/Tests/MapboxMapsTests/Foundation/GeoJSON/FeatureCollectionTests.swift b/Tests/MapboxMapsTests/Foundation/GeoJSON/FeatureCollectionTests.swift
@@ -3,7 +3,7 @@ import CoreLocation
 @testable import MapboxMaps
 
 // Disabling rules against force casting for test file.
-// swiftlint:disable explicit_top_level_acl explicit_acl force_try force_cast
+// swiftlint:disable explicit_top_level_acl explicit_acl force_try
 class FeatureCollectionTests: XCTestCase {
 
     func testFeatureCollection() {
@@ -88,13 +88,17 @@ class FeatureCollectionTests: XCTestCase {
     func testFeatureCollectionDecodeWithoutProperties() {
         let data = try! Fixture.geojsonData(from: "featurecollection-no-properties")!
         let geojson = try! JSONDecoder().decode(GeoJSONObject.self, from: data)
-        if case .featureCollection = geojson {} else { XCTFail() }
+        if case .featureCollection = geojson {} else {
+            XCTFail("GeoJSON should decode as a feature collection.")
+        }
     }
 
     func testUnkownFeatureCollection() {
         let data = try! Fixture.geojsonData(from: "featurecollection")!
         let geojson = try! JSONDecoder().decode(GeoJSONObject.self, from: data)
-        if case .featureCollection = geojson {} else { XCTFail() }
+        if case .featureCollection = geojson {} else {
+            XCTFail("GeoJSON should decode as a feature collection.")
+        }
     }
 
     func testPerformanceDecodeFeatureCollection() {

diff --git a/Tests/MapboxMapsTests/Foundation/GeoJSON/LineStringTests.swift b/Tests/MapboxMapsTests/Foundation/GeoJSON/LineStringTests.swift
@@ -2,7 +2,7 @@ import XCTest
 import CoreLocation
 @testable import MapboxMaps
 
-// swiftlint:disable explicit_top_level_acl explicit_acl force_try force_cast
+// swiftlint:disable explicit_top_level_acl explicit_acl force_try
 let metersPerMile: CLLocationDistance = 1_609.344
 
 class LineStringTests: XCTestCase {

diff --git a/Tests/MapboxMapsTests/Foundation/GeoJSON/PointTests.swift b/Tests/MapboxMapsTests/Foundation/GeoJSON/PointTests.swift
@@ -3,7 +3,7 @@ import CoreLocation
 @testable import MapboxMaps
 
 // Disabling rules against force casting for test file.
-// swiftlint:disable explicit_top_level_acl explicit_acl force_try force_cast
+// swiftlint:disable explicit_top_level_acl explicit_acl force_try
 class PointTests: XCTestCase {
 
     func testPointFeature() {
@@ -29,6 +29,8 @@ class PointTests: XCTestCase {
         let geojson = try! JSONDecoder().decode(GeoJSONObject.self, from: data)
 
         if case let .feature(feature) = geojson,
-           case .point = feature.geometry {} else { XCTFail() }
+           case .point = feature.geometry {} else {
+            XCTFail("GeoJSON should decode as a point feature.")
+        }
     }
 }
diff --git a/Tests/MapboxMapsTests/Foundation/GeoJSON/PolygonTests.swift b/Tests/MapboxMapsTests/Foundation/GeoJSON/PolygonTests.swift
@@ -3,7 +3,7 @@ import CoreLocation
 @testable import MapboxMaps
 
 // Disabling rules against force casting for test file.
-// swiftlint:disable explicit_top_level_acl explicit_acl force_try force_cast
+// swiftlint:disable explicit_top_level_acl explicit_acl force_try
 class PolygonTests: XCTestCase {
 
     func testPolygonFeature() {

diff --git a/Tests/MapboxMapsTests/Style/GeoJSONSourceDataTests.swift b/Tests/MapboxMapsTests/Style/GeoJSONSourceDataTests.swift
@@ -44,7 +44,9 @@ class GeoJSONSourceDataTests: XCTestCase {
         if let validData = decodedSource?.data,
            case let GeoJSONSourceData.feature(feature) = validData {
             XCTAssert(feature.identifier != nil)
-            if case .point = feature.geometry {} else { XCTFail() }
+            if case .point = feature.geometry {} else {
+                XCTFail("Geometry of the decoded feature should be a point.")
+            }
         } else {
             XCTFail("Failed to read decoded geojson fixture.")
         }