Refactor DbContext and related interfaces to be generic

[JJBussert]

This pull request introduces a generic type parameter T across multiple interfaces, classes, and interceptors in the E13.Common library to enhance type safety and flexibility. The changes primarily affect the audit context, entity interfaces, and database interceptors, requiring updates to dependent tests and sample implementations.

Core Refactoring to Add Generic Type Parameter T:

• Audit Context and BaseDbContext:

  • IAuditContext now accepts a generic type parameter T for AuditUser to support non-string user identifiers. (src/E13.Common.Data.Db/IAuditContext.cs, src/E13.Common.Data.Db/IAuditContext.csL9-R11)
  • BaseDbContext is updated to BaseDbContext<T> to align with the new IAuditContext<T> interface. (src/E13.Common.Data.Db/BaseDbContext.cs, [1] [2]

• Entity Interfaces:

  • Interfaces like ICreatable, IDeletable, IModifiable, and others now accept a generic type parameter T for user-related fields (e.g., CreatedBy, DeletedBy). (src/E13.Common.Domain/ICreatable.cs, [1]; src/E13.Common.Domain/IDeletable.cs, [2]

Interceptor Updates:

• Database Interceptors:
  • CreatableInterceptor, ModifiableInterceptor, and SoftDeleteInterceptor are updated to use IAuditContext<T> and the generic entity interfaces (e.g., ICreatable<T>). (src/E13.Common.Data.Db/Interceptors/CreatableInterceptor.cs, [1]; src/E13.Common.Data.Db/Interceptors/ModifiableInterceptor.cs, [2]

Test Updates:

• Test Contexts and Assertions:
  • Updated test classes and methods to use BaseDbContext<string> and generic interceptors. (test/E13.Common.Data.Db.Tests/BaseDbContext_ICreatableTests.cs, test/E13.Common.Data.Db.Tests/BaseDbContext_ICreatableTests.csL20-R30)
  • Adjusted assertions to reference BaseDbContext<string>.UnknownUser for compatibility with the new generic implementation. (test/E13.Common.Data.Db.Tests/BaseDbContext_ICreatableTests.cs, test/E13.Common.Data.Db.Tests/BaseDbContext_ICreatableTests.csL59-R59)

Sample Code Adjustments:

• Sample Entities and Context:
  • Updated sample implementations, such as TestCreatable and TestDbContext, to use the generic interfaces and BaseDbContext<T>. (test/E13.Common.Data.Db.Tests/Sample/TestCreatable.cs, [1]; test/E13.Common.Data.Db.Tests/Sample/TestDbContext.cs, [2]

[Copilot]

Pull Request Overview

This PR refactors the DbContext and related interfaces to be generic by introducing a type parameter T across audit, entity, interceptor, and testing components to improve type safety and flexibility.

• Updated interfaces (ICreatable, IModifiable, IDeletable, etc.) and the DbContext to include a generic type parameter T.
• Updated database interceptors and tests to use the generic implementations.
• Adjusted sample implementations to align with the new generic definitions.

Reviewed Changes

Copilot reviewed 22 out of 22 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
test/E13.Common.Data.Db.Tests/Sample/TestModifiable.cs	Updated to implement IModifiable.
test/E13.Common.Data.Db.Tests/Sample/TestInvalidDeletable.cs	Updated to implement IDeletable.
test/E13.Common.Data.Db.Tests/Sample/TestEffectable.cs	Updated to implement IEffectable.
test/E13.Common.Data.Db.Tests/Sample/TestDeletable.cs	Updated to implement IDeletable.
test/E13.Common.Data.Db.Tests/Sample/TestDbContext.cs	Changed to implement BaseDbContext and modified the SaveChanges call in AddTestData.
test/E13.Common.Data.Db.Tests/Sample/TestCreatable.cs	Updated to implement ICreatable.
test/E13.Common.Data.Db.Tests/BaseDbContext_*Tests.cs	Updated interceptors and assertions to use the generic versions.
src/E13.Common.Domain/*	Updated various entity interfaces to accept generic type parameter T with appropriate nullability.
src/E13.Common.Data.Db/Interceptors/*	Updated interceptors to work with the generic audit context and entity interfaces.
src/E13.Common.Data.Db/IAuditContext.cs	Modified interface to include generic type parameter T.
src/E13.Common.Data.Db/BaseDbContext.cs	Refactored to be generic (BaseDbContext) and updated property/setter definitions.

Comments suppressed due to low confidence (1)

test/E13.Common.Data.Db.Tests/Sample/TestDbContext.cs:72

• Switching from SaveChanges_NoTagEntries() to SaveChanges() in AddTestData() may trigger additional tagging logic that was previously bypassed for arranging invalid data. Consider confirming if the new behavior is intended or if the test should still bypass those checks.

SaveChanges();