Skip to content

Commit

Permalink
fix!: Update maximum and default page_size for ListRequisitions and L…
Browse files Browse the repository at this point in the history
…istEventGroups (#1916)

Closes #1915
  • Loading branch information
SanjayVas authored Nov 18, 2024
1 parent cb75127 commit 6f9b14b
Show file tree
Hide file tree
Showing 6 changed files with 10 additions and 10 deletions.
2 changes: 1 addition & 1 deletion MODULE.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ bazel_dep(
)
bazel_dep(
name = "cross-media-measurement-api",
version = "0.69.0",
version = "0.70.0",
repo_name = "wfa_measurement_proto",
)
bazel_dep(
Expand Down
6 changes: 3 additions & 3 deletions MODULE.bazel.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Original file line number Diff line number Diff line change
Expand Up @@ -450,8 +450,8 @@ class EventGroupsService(private val internalEventGroupsStub: InternalEventGroup
}

companion object {
private const val DEFAULT_PAGE_SIZE = 50
private const val MAX_PAGE_SIZE = 1000
private const val DEFAULT_PAGE_SIZE = 10
private const val MAX_PAGE_SIZE = 500
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -82,8 +82,8 @@ import org.wfanet.measurement.internal.kingdom.refuseRequisitionRequest
import org.wfanet.measurement.internal.kingdom.requisitionRefusal as internalRequisitionRefusal
import org.wfanet.measurement.internal.kingdom.streamRequisitionsRequest

private const val DEFAULT_PAGE_SIZE = 50
private const val MAX_PAGE_SIZE = 1000
private const val DEFAULT_PAGE_SIZE = 10
private const val MAX_PAGE_SIZE = 500

class RequisitionsService(private val internalRequisitionStub: RequisitionsCoroutineStub) :
RequisitionsCoroutineImplBase() {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1128,7 +1128,7 @@ class EventGroupsServiceTest {
}
assertThat(streamEventGroupsRequest)
.comparingExpectedFieldsOnly()
.isEqualTo(streamEventGroupsRequest { limit = 51 })
.isEqualTo(streamEventGroupsRequest { limit = 11 })
}

@Test
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@ import org.wfanet.measurement.kingdom.deploy.gcloud.spanner.common.RequisitionSt

private val UPDATE_TIME: Timestamp = Instant.ofEpochSecond(123).toProtoTime()

private const val DEFAULT_LIMIT = 50
private const val DEFAULT_LIMIT = 10

private const val WILDCARD_NAME = "dataProviders/-"

Expand Down

0 comments on commit 6f9b14b

Please sign in to comment.