Add XML docs and SourceLink metadata; enforce CS1591 #341

Added XML documentation comments to key public members in TestBedWithDI, OutputLogger, and OutputLoggerProvider. Updated the project file to remove suppression of CS1591 warnings, enforce documentation completeness, and include SourceLink and repository metadata for improved NuGet package quality and traceability.

Author: Arash-Sabet

src/Abstracts/TestBedWithDI.cs

@@ -10,6 +10,11 @@ namespace Xunit.Microsoft.DependencyInjection.Abstracts;
 public abstract class TestBedWithDI<TFixture> : TestBed<TFixture>
 	where TFixture : TestBedFixture
 {
+	/// <summary>
+	/// Initializes the test bed and performs property injection for members marked with <see cref="InjectAttribute"/>.
+	/// </summary>
+	/// <param name="testOutputHelper">The xUnit test output helper.</param>
+	/// <param name="fixture">The fixture instance supplying services.</param>
 	protected TestBedWithDI(ITestOutputHelper testOutputHelper, TFixture fixture)
 		: base(testOutputHelper, fixture) =>
 		// Resolve and inject dependencies for derived classes

src/Logging/OutputLogger.cs

@@ -16,9 +16,15 @@ public OutputLogger(ITestOutputHelper testOutputHelper)
 	{
 	}
 
+	/// <summary>
+	/// Begins a logical operation scope. Returns a disposable that performs no action.
+	/// </summary>
 	public IDisposable? BeginScope<TState>(TState state) where TState : notnull
 		=> new NoOpDisposable();
 
+	/// <summary>
+	/// Always returns true; all log levels are enabled for forwarding to test output.
+	/// </summary>
 	public bool IsEnabled(LogLevel logLevel)
 		=> true;
 

src/Logging/OutputLoggerProvider.cs

@@ -8,8 +8,14 @@ public class OutputLoggerProvider(ITestOutputHelper testOutputHelper) : ILoggerP
 {
 	private readonly ITestOutputHelper _testOutputHelper = testOutputHelper;
 
+	/// <summary>
+	/// Creates a new <see cref="OutputLogger"/> for the specified category.
+	/// </summary>
 	public ILogger CreateLogger(string categoryName)
 		=> new OutputLogger(categoryName, _testOutputHelper);
 
+	/// <summary>
+	/// Disposes the provider (no-op other than suppressing finalization).
+	/// </summary>
 	public void Dispose() => GC.SuppressFinalize(this);
 }

src/Xunit.Microsoft.DependencyInjection.csproj

@@ -7,10 +7,18 @@
 	<PackageReadmeFile>README.md</PackageReadmeFile>
   <!-- Enable XML documentation output for NuGet consumers -->
   <GenerateDocumentationFile>true</GenerateDocumentationFile>
-  <!-- Suppress warning for missing XML comments on publicly visible types if any remain -->
-  <NoWarn>$(NoWarn);CS1591</NoWarn>
-  <!-- Include repository URL placeholders if desired (can be updated later) -->
+  <!-- Enforce completeness: do NOT suppress CS1591 (removed) -->
   <DocumentationFile>bin\$(Configuration)\$(TargetFramework)\Xunit.Microsoft.DependencyInjection.xml</DocumentationFile>
+  <!-- Repository / SourceLink metadata -->
+  <RepositoryUrl>https://github.com/Umplify/xunit-dependency-injection</RepositoryUrl>
+  <RepositoryType>git</RepositoryType>
+  <PublishRepositoryUrl>true</PublishRepositoryUrl>
+  <EmbedUntrackedSources>true</EmbedUntrackedSources>
+  <IncludeSymbols>true</IncludeSymbols>
+  <SymbolPackageFormat>snupkg</SymbolPackageFormat>
+  <EnableSourceLink>true</EnableSourceLink>
+  <!-- Optional: mark builds as CI to embed commit information when built on CI -->
+  <ContinuousIntegrationBuild>true</ContinuousIntegrationBuild>
   </PropertyGroup>
   <ItemGroup>
     <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="9.0.8" />
@@ -20,9 +28,12 @@
     <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="9.0.8" />
     <PackageReference Include="xunit.v3.extensibility.core" Version="3.0.1" />
     <PackageReference Include="Microsoft.Extensions.Configuration.EnvironmentVariables" Version="9.0.8" />
+  <PackageReference Include="Microsoft.SourceLink.GitHub" Version="8.0.0" PrivateAssets="All" />
   </ItemGroup>
   <ItemGroup>
 	<None Include="..\README.md" Pack="true" PackagePath="\" />
+  <!-- Explicitly include XML documentation file in package -->
+  <None Include="bin\$(Configuration)\$(TargetFramework)\Xunit.Microsoft.DependencyInjection.xml" Pack="true" PackagePath="\" Condition="Exists('bin\\$(Configuration)\\$(TargetFramework)\\Xunit.Microsoft.DependencyInjection.xml')" />
   </ItemGroup>
   <ItemGroup>
     <Folder Include="Abstracts\" />