[Cosmos] Extract a StoreResult test factory to replace the 20-arg positional constructor in unit tests
I maintainer di solito rispondono entro 1 giorno
@mbhaskar ci sta già lavorando.
Dal 9/6/2026.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
Summary
Follow-up from PR #48914 (review feedback from @kushagraThapar).
The 20-arg positional new StoreResult(...) constructor is repeated across many test methods (in ConsistencyWriterTest.java at lines ~667, 725, 757, 779, 834, 894, 903, 919, 947, and several more in QuorumReaderTest.java). Each invocation follows the same pattern: most arguments are constants/null/false, with only 2–3 meaningful per test (replica role, isThrottledException, isValid).
This positional-constructor accretion has been growing across multiple PRs and makes tests verbose and fragile.
Proposed change
Extract a StoreResultTestFactory (or builder) with named, intent-revealing methods, e.g.:
throttled429()valid(int globalCommittedLsn)unhealthy(replicaRole)
Benefits:
- New tests become 1-line setup instead of ~20-line setup.
- When the
StoreResultconstructor changes (e.g. adding a 21st arg), only the factory updates instead of N test sites. - Test intent becomes self-documenting (
throttled429()reads as a behavior, not a 20-arg ctor invocation).
Scope
Test-only refactor; no production behavior change. Update existing StoreResult-based tests in:
sdk/cosmos/azure-cosmos-tests/.../directconnectivity/ConsistencyWriterTest.javasdk/cosmos/azure-cosmos-tests/.../directconnectivity/QuorumReaderTest.java
Context
Reviewer noted this is "not a blocker for [PR #48914] but worth scheduling as a follow-up before the next test wave lands on top."
- Lingua principale
- Java
- Stelle
- 2.6k
- Fork
- 2.2k
- Merge medio
- 1g 22h
- PR unite (30g)
- 213
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di Azure/azure-sdk-for-java
-
mgmt, in track 1 migration guide, call out that file-based authentication method is not recommendedForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. ApertaAzure.Identity
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 74/100
Azure/azure-sdk-for-java#48894 ·
I maintainer di solito rispondono entro 1 giorno
-
[BUG] - Handler.close livelocks on FAIL_NON_SERIALIZED while holding the ReactorConnection monitorForse già presa @conniey l’ha presa 1 giorno fa. ApertaAzure.Core.AMQP Client customer-reported needs-team-attention question
Azure/azure-sdk-for-java#50879 · 2 commenti · 1 reazione · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
[Design] Support External Evaluation Policy (Invoke) in management SDKsForse già presa @weidongxu-microsoft l’ha presa 6 giorni fa. Aperta
Azure/azure-sdk-for-java#50818 · 2 reazioni · 2 assegnatari ·
I maintainer di solito rispondono entro 1 giorno
-
[BUG] azure-core-http-vertx: Streaming request bodies hang on Vert.x 5.x (NoSuchMethodError on HttpClientRequest.reset)Forse già presa @alzimmermsft l’ha presa 7 giorni fa. ApertaAzure.Core Client customer-reported needs-team-attention question
Azure/azure-sdk-for-java#50793 · 6 commenti · 1 reazione · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
[FEATURE REQ] ChangeFeedProcessor event handler fully reactive implementationForse già presa @FabianMeiswinkel l’ha presa 7 giorni fa. ApertaClient Cosmos customer-reported feature-request needs-team-attention Service Attention
Difficoltà 5/5 Più di una settimana Idoneità per principianti 12/100
Azure/azure-sdk-for-java#50758 · 3 commenti · 1 reazione · 2 assegnatari ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di Azure/azure-sdk-for-java
Issue simili
-
ChaosEngine: backup-remote doctor row passes while consumer mode hides bot memory from gitForse già presa @MohabMohie l’ha presa oggi. Apertaagentic
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
ShaftHQ/SHAFT_ENGINE#7263 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
-
bug/错误
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
FCL-Team/FoldCraftLauncher#1951 ·
I maintainer di solito rispondono entro 4 giorni
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 68/100
quarkusio/quarkus-agent-mcp#296 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
commons-app/apps-android-commons#7072 ·
I maintainer di solito rispondono entro 1 giorno