Skip to content

chore: tests for batching and deduplication by field selection #1449

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 20 commits into from
Jun 2, 2022
Merged

chore: tests for batching and deduplication by field selection #1449

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
4ec8f7c
feat: support chained dataloaders
May 3, 2022
44a3640
feat: unused code
May 3, 2022
396765f
feat: renaming some fields and methods
May 4, 2022
eadd1d9
feat: renaming some fields and methods
May 4, 2022
1758f9c
feat: update coverage bc decorated methods
May 4, 2022
437bba8
fix: KotlinDataLoaderegistry
May 10, 2022
c5d8a58
fix: KotlinDataLoader test
May 10, 2022
748f90b
fix: KotlinDataLoader test
May 10, 2022
1a7cc50
feat: add copyright
May 11, 2022
39b2b56
feat: add removed files
May 11, 2022
09f7251
feat: update docs
May 16, 2022
6035002
feat: example of batching by selection fields
May 17, 2022
19b3e87
feat: add 1 more test
May 17, 2022
a310fd1
feat: add 1 more test
May 17, 2022
c32795e
feat: updade docs
May 19, 2022
da04f90
feat: update test
May 25, 2022
8364b40
feat: test case for batching by selection fields
May 27, 2022
e9074fe
chore: pull docs
May 27, 2022
9f52b30
feat: update tests
May 27, 2022
fb2b796
feat: update tests
May 27, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
feat: update tests
  • Loading branch information
samvazquez committed May 27, 2022
commit fb2b796d23ff3f4b84c954dc5e79a71f5364c264
16 changes: 8 additions & 8 deletions ...m/expediagroup/graphql/dataloader/instrumentation/fixture/repository/ProductRepository.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ import java.time.Duration
import java.util.Optional

object ProductRepository {
private val properties = listOf(
private val products = listOf(
Product(
1,
ProductSummary("Product 1"),
Expand All @@ -36,25 +36,25 @@ object ProductRepository {
fun getProducts(requests: List<ProductServiceRequest>): Flux<Optional<Product>> {
val reducedRequests = requests
.groupBy(ProductServiceRequest::id)
.mapValues { (propertyId, requests) ->
.mapValues { (productId, requests) ->
ProductServiceRequest(
propertyId,
productId,
requests.map(ProductServiceRequest::fields).flatten().distinct()
)
}.values.toList()

val results = reducedRequests.mapNotNull { propertyRequest ->
properties
.firstOrNull { it.id == propertyRequest.id }
val results = reducedRequests.mapNotNull { productRequest ->
products
.firstOrNull { it.id == productRequest.id }
?.let { property ->
Product(
property.id,
when {
propertyRequest.fields.contains("summary") -> property.summary
productRequest.fields.contains("summary") -> property.summary
else -> null
},
when {
propertyRequest.fields.contains("details") -> property.details
productRequest.fields.contains("details") -> property.details
else -> null
}
)
Expand Down