refactor(data): Convert Geometry to Kotlin

This commit refactors the `Geometry` interface, converting it from Java to Kotlin.

As part of this change, the `getGeometryType()` and `getGeometryObject()` methods have been replaced with the idiomatic Kotlin properties `geometryType` and `geometryObject`.

The implementing classes, `Point` and `LineString`, and their corresponding tests have been updated to align with this new property-based API.

Author: dkhawk

library/src/main/java/com/google/maps/android/data/Geometry.kt

@@ -1,5 +1,5 @@
 /*
- * Copyright 2017 Google Inc.
+ * Copyright 2025 Google LLC
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -13,27 +13,21 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
-package com.google.maps.android.data;
+package com.google.maps.android.data
 
 /**
  * An abstraction that represents a Geometry object
  *
- * @param <T> the type of Geometry object
+ * @param T the type of Geometry object
  */
-public interface Geometry<T> {
+interface Geometry<T> {
     /**
      * Gets the type of geometry
-     *
-     * @return type of geometry
      */
-    String getGeometryType();
+    val geometryType: String
 
     /**
      * Gets the stored KML Geometry object
-     *
-     * @return geometry object
      */
-    T getGeometryObject();
-
-}
+    val geometryObject: T
+}

library/src/main/java/com/google/maps/android/data/LineString.kt

@@ -35,12 +35,12 @@ open class LineString(coordinates: List<LatLng>) : Geometry<List<LatLng>> {
     /**
      * Gets the type of geometry
      */
-    override fun getGeometryType(): String = "LineString"
+    override val geometryType: String = "LineString"
 
     /**
      * Gets the geometry object
      */
-    override fun getGeometryObject(): List<LatLng> = _coordinates
+    override val geometryObject: List<LatLng> = _coordinates
 
     override fun equals(other: Any?): Boolean {
         if (this === other) return true
@@ -58,4 +58,4 @@ open class LineString(coordinates: List<LatLng>) : Geometry<List<LatLng>> {
     override fun toString(): String {
         return "LineString(coordinates=$_coordinates)"
     }
-}
+}

library/src/main/java/com/google/maps/android/data/Point.kt

@@ -35,12 +35,12 @@ open class Point(coordinates: LatLng) : Geometry<LatLng> {
     /**
      * Gets the type of geometry
      */
-    override fun getGeometryType(): String = "Point"
+    override val geometryType: String = "Point"
 
     /**
      * Gets the geometry object
      */
-    override fun getGeometryObject(): LatLng = _coordinates
+    override val geometryObject: LatLng = _coordinates
 
     override fun equals(other: Any?): Boolean {
         if (this === other) return true
@@ -58,4 +58,4 @@ open class Point(coordinates: LatLng) : Geometry<LatLng> {
     override fun toString(): String {
         return "Point(coordinates=$_coordinates)"
     }
-}
+}

library/src/test/java/com/google/maps/android/data/LineStringTest.kt

@@ -35,13 +35,13 @@ class LineStringTest {
     }
 
     @Test
-    fun `getType returns correct type`() {
+    fun `geometryType returns correct type`() {
         val lineString = createSimpleLineString()
-        assertThat(lineString.getGeometryType()).isEqualTo("LineString")
+        assertThat(lineString.geometryType).isEqualTo("LineString")
     }
 
     @Test
-    fun `getGeometryObject returns correct coordinates`() {
+    fun `geometryObject returns correct coordinates`() {
         val lineString = createSimpleLineString()
         val expectedCoordinates = listOf(
             LatLng(90.0, 60.0),
@@ -51,7 +51,7 @@ class LineStringTest {
             LatLng(90.0, 54.0),
             LatLng(86.0, 56.0)
         )
-        assertThat(lineString.getGeometryObject()).containsExactlyElementsIn(expectedCoordinates).inOrder()
+        assertThat(lineString.geometryObject).containsExactlyElementsIn(expectedCoordinates).inOrder()
     }
 
     @Test
@@ -80,4 +80,4 @@ class LineStringTest {
         val lineString = LineString(listOf(LatLng(1.0, 2.0)))
         assertThat(lineString.toString()).isEqualTo("LineString(coordinates=[lat/lng: (1.0,2.0)])")
     }
-}
+}

library/src/test/java/com/google/maps/android/data/PointTest.kt

@@ -23,16 +23,16 @@ import java.lang.reflect.InvocationTargetException
 
 class PointTest {
     @Test
-    fun `getGeometryType returns correct type`() {
+    fun `geometryType returns correct type`() {
         val point = Point(LatLng(0.0, 50.0))
-        assertThat(point.getGeometryType()).isEqualTo("Point")
+        assertThat(point.geometryType).isEqualTo("Point")
     }
 
     @Test
-    fun `getGeometryObject returns correct coordinates`() {
+    fun `geometryObject returns correct coordinates`() {
         val coordinates = LatLng(0.0, 50.0)
         val point = Point(coordinates)
-        assertThat(point.getGeometryObject()).isEqualTo(coordinates)
+        assertThat(point.geometryObject).isEqualTo(coordinates)
         assertThat(point.coordinates).isEqualTo(coordinates)
     }
 
@@ -66,4 +66,4 @@ class PointTest {
         val point = Point(LatLng(1.0, 2.0))
         assertThat(point.toString()).isEqualTo("Point(coordinates=lat/lng: (1.0,2.0))")
     }
-}
+}