Failing dynamo create requests when a range is specified since it is ignored.

client-dynamodb-v2/src/main/kotlin/app/cash/backfila/client/dynamodbv2/internal/DynamoDbBackfillOperator.kt

@@ -29,6 +29,11 @@ class DynamoDbBackfillOperator<I : Any, P : Any>(
     val config = parametersOperator.constructBackfillConfig(request)
     backfill.validate(config)
 
+    require(
+      request.range == null ||
+        (request.range.start == null && request.range.end == null),
+    ) { "Range is an invalid input for this Dynamo Backfila client" }
+
     var table = dynamoDbClient.describeTable {
       it.tableName(backfill.dynamoDbTable.tableName())
     }.table()

client-dynamodb-v2/src/test/kotlin/app/cash/backfila/client/dynamodbv2/DynamoDbBackfillTest.kt

@@ -59,6 +59,23 @@ class DynamoDbBackfillTest {
     }.hasMessageContaining("Validate failed")
   }
 
+  @Test
+  fun `adding a range fails creation`() {
+    testData.addThriller()
+
+    assertThatCode {
+      backfila.createWetRun<MakeTracksExplicitBackfill>(rangeStart = "start")
+    }.hasMessageContaining("Range is an invalid input for this Dynamo Backfila client")
+
+    assertThatCode {
+      backfila.createWetRun<MakeTracksExplicitBackfill>(rangeEnd = "end")
+    }.hasMessageContaining("Range is an invalid input for this Dynamo Backfila client")
+
+    assertThatCode {
+      backfila.createWetRun<MakeTracksExplicitBackfill>(rangeStart = "start", rangeEnd = "end")
+    }.hasMessageContaining("Range is an invalid input for this Dynamo Backfila client")
+  }
+
   class MakeTracksExplicitBackfill @Inject constructor(
     dynamoDb: DynamoDbClient,
     private val dynamoDbEnhancedClient: DynamoDbEnhancedClient,

client-dynamodb/build.gradle.kts

@@ -36,6 +36,12 @@ dependencies {
   testImplementation(project(":backfila-embedded"))
   testImplementation(project(":client-testing"))
 
+  if (org.apache.tools.ant.taskdefs.condition.Os.isArch("aarch64")) {
+    // Without this, we can't compile on Apple Silicon currently. This is likely not necessary to
+    // have longterm, so we should remove it when platform fixes things across Square.
+    testImplementation("io.github.ganadist.sqlite4java:libsqlite4java-osx-aarch64:1.0.392")
+  }
+
   // ****************************************
   // For TESTING purposes only. We only want Misk for easy testing.
   // DO NOT turn these into regular dependencies.

client-dynamodb/src/main/kotlin/app/cash/backfila/client/dynamodb/internal/DynamoDbBackfillOperator.kt

@@ -28,6 +28,11 @@ class DynamoDbBackfillOperator<I : Any, P : Any>(
     val config = parametersOperator.constructBackfillConfig(request)
     backfill.validate(config)
 
+    require(
+      request.range == null ||
+        (request.range.start == null && request.range.end == null),
+    ) { "Range is an invalid input for this Dynamo Backfila client" }
+
     val tableMapper = dynamoDb.newTableMapper<I, Any, Any>(backfill.itemType.java)
     val tableDescription = tableMapper.describeTable()
     if (backfill.mustHaveProvisionedBillingMode()) {

client-dynamodb/src/test/kotlin/app/cash/backfila/client/dynamodb/DynamoDbBackfillTest.kt

@@ -53,6 +53,23 @@ class DynamoDbBackfillTest {
     }.hasMessageContaining("Validate failed")
   }
 
+  @Test
+  fun `adding a range fails creation`() {
+    testData.addThriller()
+
+    assertThatCode {
+      backfila.createWetRun<MakeTracksExplicitBackfill>(rangeStart = "start")
+    }.hasMessageContaining("Range is an invalid input for this Dynamo Backfila client")
+
+    assertThatCode {
+      backfila.createWetRun<MakeTracksExplicitBackfill>(rangeEnd = "end")
+    }.hasMessageContaining("Range is an invalid input for this Dynamo Backfila client")
+
+    assertThatCode {
+      backfila.createWetRun<MakeTracksExplicitBackfill>(rangeStart = "start", rangeEnd = "end")
+    }.hasMessageContaining("Range is an invalid input for this Dynamo Backfila client")
+  }
+
   class MakeTracksExplicitBackfill @Inject constructor(
     dynamoDb: DynamoDBMapper,
   ) : UpdateInPlaceDynamoDbBackfill<TrackItem, MakeTracksExplicitBackfill.ExplicitParameters>(dynamoDb) {