Verify store registration consistency across tracing and factory configurations
Los mantenedores suelen responder en 7 días
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 48/100
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- C#
- Estrellas
- 512
- Forks
- 99
- Merge medio
- 5 d 6 h
- PR fusionados (30 d)
- 5
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de Eventuous/eventuous
-
MongoDB integration tests fail on newer GitHub Actions runner kernelsPosiblemente ocupada @nmummau la tomó hace 3 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Eventuous/eventuous#626 · 2 comentarios ·
Los mantenedores suelen responder en 7 días
-
`dotnet format style --verify-no-changes` fails with naming violations and unnecessary importsPosiblemente ocupada @nmummau la tomó hace 3 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
Eventuous/eventuous#624 · 1 comentario ·
Los mantenedores suelen responder en 7 días
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
Eventuous/eventuous#627 · 1 comentario ·
Los mantenedores suelen responder en 7 días
-
SQL Server: `append_events` does not need the `@inserted` table variablePosiblemente ocupada @nmummau la tomó hace 9 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 20/100
Eventuous/eventuous#619 · 1 comentario ·
Los mantenedores suelen responder en 7 días
-
SQL Server: stream reads do one key lookup per event, and `IDX_EventsStream` duplicates `UQ_StreamIdAndStreamPosition`Posiblemente ocupada @nmummau la tomó hace 9 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 18/100
Eventuous/eventuous#618 · 1 comentario ·
Los mantenedores suelen responder en 7 días
Todos los issues de Eventuous/eventuous
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Nimblesite/SharpLsp#468 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
elastic/ecs-dotnet#575 ·
-
V4 alert-rule reads have no scope gate, so any authenticated credential can list alert rulesAbiertoalerts bug effort:S P2
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
nightscout/nocturne#2511 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 2 días
-
Crash in v3.00Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
arkane-systems/mousejiggler#140 ·