Verify store registration consistency across tracing and factory configurations
I maintainer di solito rispondono entro 7 giorni
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 48/100
Direzione di ricerca
Start with src/Extensions/src/Eventuous.Extensions.DependencyInjection/Registrations/Stores.cs and the dependency injection test project. Read the registration overloads and run the existing registration tests to learn the intended singleton and existing-registration behavior. Add host-level tests using a stateful in-memory store for tracing and factory combinations; done means writes and reads through the registered interfaces share the intended store and the required behaviors are covered.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Problem
The store-registration code in Stores.cs has approximately 12% line coverage in the October 5, 2026 .NET 10 coverage run. It contains separate paths for tracing, implementation-type registrations, and factory registrations. You can view a test coverage report I generated here: index.html
A host can build successfully while its reader and writer resolve to different backing stores or its tracing wrappers resolve incorrectly. Registration tests should therefore verify working reads and writes, not just successful service resolution.
Source: Stores.cs.
Proposed tests
- Build a minimal host using
AddEventStore<T>and verify that an event written throughIEventWritercan be read throughIEventReaderand accessed throughIEventStore. - Exercise both implementation-type and factory registration overloads with tracing enabled and disabled.
- Verify the corresponding read/write behavior for
AddEventReaderWriter<T>. - Exercise standalone reader and writer registrations in a working host.
- Verify singleton and factory-invocation behavior according to the intended contract. Tracing wrappers may be different objects, but must use the intended underlying store.
- Verify the documented behavior when registrations already exist or registration methods are called more than once.
Acceptance criteria
- The tracing/registration-overload combinations are covered with actual event writes and reads.
- Reader, writer, and store interfaces observe consistent backing-store state where registered as one store.
- Factory invocation counts and service lifetimes match the intended singleton contract.
- Existing-registration behavior is explicitly tested and documented.
- Any process-wide diagnostics settings are restored after each test and isolated from parallel tests.
- Tests use a stateful in-memory store and require no Docker containers.
Validation
Run the dependency-injection test project and collect focused coverage for store registrations. Add a regression test before correcting any inconsistency discovered by the host-level tests.
- Lingua principale
- C#
- Stelle
- 512
- Fork
- 99
- Merge medio
- 5g 6h
- PR unite (30g)
- 5
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Nessun 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 Eventuous/eventuous
-
MongoDB integration tests fail on newer GitHub Actions runner kernelsForse già presa @nmummau l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
Eventuous/eventuous#626 · 2 commenti ·
I maintainer di solito rispondono entro 7 giorni
-
`dotnet format style --verify-no-changes` fails with naming violations and unnecessary importsForse già presa @nmummau l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 80/100
Eventuous/eventuous#624 · 1 commento ·
I maintainer di solito rispondono entro 7 giorni
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
Eventuous/eventuous#627 · 1 commento ·
I maintainer di solito rispondono entro 7 giorni
-
SQL Server: `append_events` does not need the `@inserted` table variableForse già presa @nmummau l’ha presa 6 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 20/100
Eventuous/eventuous#619 · 1 commento ·
I maintainer di solito rispondono entro 7 giorni
-
SQL Server: stream reads do one key lookup per event, and `IDX_EventsStream` duplicates `UQ_StreamIdAndStreamPosition`Forse già presa @nmummau l’ha presa 6 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 18/100
Eventuous/eventuous#618 · 1 commento ·
I maintainer di solito rispondono entro 7 giorni
Tutte le issue di Eventuous/eventuous
Issue simili
-
Format request: .wbpzApertaenhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
KnifeLemon/Filee#294 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
gavin-wolfe/wolfden#104 ·
I maintainer di solito rispondono entro 1 giorno
-
[BUG] Still libmicrohttpd dependency is not removed properly from External-Dependency-Info.txt fileForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertatype/bug
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
area:framework
Difficoltà 1/5 1-3 ore Idoneità per principianti 72/100
nunit/docs#1262 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
validate_environment_path strips executable extensions from directory names, not only from the end of the pathForse già presa @auraseb l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
Unity-Technologies/ml-agents#6332 ·