From c6407580d20b30396991cf9c6b95a1722925f555 Mon Sep 17 00:00:00 2001 From: Thomas Rasch Date: Fri, 26 Jul 2024 09:47:21 +0200 Subject: [PATCH] Code fixes --- Sources/GISTools/Algorithms/LineOverlap.swift | 2 +- .../Algorithms/LineOverlapTests.swift | 18 +++++++++--------- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/Sources/GISTools/Algorithms/LineOverlap.swift b/Sources/GISTools/Algorithms/LineOverlap.swift index efab8f1..1244e80 100644 --- a/Sources/GISTools/Algorithms/LineOverlap.swift +++ b/Sources/GISTools/Algorithms/LineOverlap.swift @@ -177,7 +177,7 @@ extension GeoJson { guard comparison != .notEqual else { continue } result.append(SegmentOverlapResult( - kind: comparison, + overlap: comparison, segment: segment, other: match)) } diff --git a/Tests/GISToolsTests/Algorithms/LineOverlapTests.swift b/Tests/GISToolsTests/Algorithms/LineOverlapTests.swift index f126e13..ff90a31 100644 --- a/Tests/GISToolsTests/Algorithms/LineOverlapTests.swift +++ b/Tests/GISToolsTests/Algorithms/LineOverlapTests.swift @@ -11,7 +11,7 @@ final class LineOverlapTests: XCTestCase { XCTAssertEqual(overlappingSegments.count, 2) - XCTAssertEqual(overlappingSegments.map(\.kind), [.equal, .equal]) + XCTAssertEqual(overlappingSegments.map(\.overlap), [.equal, .equal]) XCTAssertEqual(overlappingSegments[0].segment.second, overlappingSegments[1].segment.first) XCTAssertEqual(overlappingSegments[0].segment.first, Coordinate3D(latitude: -30.0, longitude: 125.0)) XCTAssertEqual(overlappingSegments[1].segment.second, Coordinate3D(latitude: -35.0, longitude: 145.0)) @@ -25,7 +25,7 @@ final class LineOverlapTests: XCTestCase { XCTAssertEqual(overlappingSegments.count, 2) - XCTAssertEqual(overlappingSegments.map(\.kind), [.equal, .equal]) + XCTAssertEqual(overlappingSegments.map(\.overlap), [.equal, .equal]) XCTAssertEqual(overlappingSegments[0].segment.second, overlappingSegments[1].segment.first) XCTAssertEqual(overlappingSegments[0].segment.first, Coordinate3D(latitude: -30.0, longitude: 125.0)) XCTAssertEqual(overlappingSegments[1].segment.second, Coordinate3D(latitude: -35.0, longitude: 145.0)) @@ -39,7 +39,7 @@ final class LineOverlapTests: XCTestCase { XCTAssertEqual(overlappingSegments.count, 2) - XCTAssertEqual(overlappingSegments.map(\.kind), [.equal, .equal]) + XCTAssertEqual(overlappingSegments.map(\.overlap), [.equal, .equal]) XCTAssertEqual(overlappingSegments[0].segment.second, overlappingSegments[1].segment.first) XCTAssertEqual(overlappingSegments[0].segment.first, Coordinate3D(latitude: -30.0, longitude: 125.0)) XCTAssertEqual(overlappingSegments[1].segment.second, Coordinate3D(latitude: -35.0, longitude: 145.0)) @@ -53,7 +53,7 @@ final class LineOverlapTests: XCTestCase { let overlappingSegments = polygon1.overlappingSegments(with: polygon2) XCTAssertEqual(overlappingSegments.count, 6) - XCTAssertEqual(Set(overlappingSegments.map(\.kind)), [.equalReversed]) + XCTAssertEqual(Set(overlappingSegments.map(\.overlap)), [.equalReversed]) let firstSegments = result.lineStrings[0].lineSegments let secondSegments = result.lineStrings[1].lineSegments @@ -79,7 +79,7 @@ final class LineOverlapTests: XCTestCase { let overlappingSegments = polygon1.overlappingSegments(with: polygon2) XCTAssertEqual(overlappingSegments.count, 4) - XCTAssertEqual(Set(overlappingSegments.map(\.kind)), [.otherOnThis]) + XCTAssertEqual(Set(overlappingSegments.map(\.overlap)), [.otherOnThis]) let firstSegments = result.lineStrings[0].lineSegments let secondSegments = result.lineStrings[1].lineSegments @@ -96,7 +96,7 @@ final class LineOverlapTests: XCTestCase { let overlappingSegments = polygon1.overlappingSegments(with: polygon2, tolerance: 5000.0) XCTAssertEqual(overlappingSegments.count, 1) - XCTAssertEqual(Set(overlappingSegments.map(\.kind)), [.otherOnThis]) + XCTAssertEqual(Set(overlappingSegments.map(\.overlap)), [.otherOnThis]) XCTAssertEqual(overlappingSegments.map(\.other), result.lineSegments) } @@ -120,8 +120,8 @@ final class LineOverlapTests: XCTestCase { XCTAssertEqual(overlappingSegments1.count, 1) XCTAssertEqual(overlappingSegments2.count, 1) - XCTAssertEqual(Set(overlappingSegments1.map(\.kind)), [.thisOnOther]) - XCTAssertEqual(Set(overlappingSegments2.map(\.kind)), [.otherOnThis]) + XCTAssertEqual(Set(overlappingSegments1.map(\.overlap)), [.thisOnOther]) + XCTAssertEqual(Set(overlappingSegments2.map(\.overlap)), [.otherOnThis]) XCTAssertEqual(overlappingSegments1.map(\.segment), result.lineSegments) XCTAssertEqual(overlappingSegments2.map(\.other), result.lineSegments) @@ -213,7 +213,7 @@ final class LineOverlapTests: XCTestCase { ])! let overlapping = lineString.overlappingSegments(with: nil) - overlapping.forEach({ print("\($0.kind): \($0.segment.coordinates), other: \($0.other.coordinates)") }) + overlapping.forEach({ print("\($0.overlap): \($0.segment.coordinates), other: \($0.other.coordinates)") }) XCTAssertEqual(overlapping.count, 6) }