Skip to content

Commit 36b7927

Browse files
committed
style: fix whitespace issues across multiple files
1 parent baa2d9e commit 36b7927

File tree

6 files changed

+25
-25
lines changed

6 files changed

+25
-25
lines changed

src/NLWebNet/Extensions/ServiceCollectionExtensions.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ public static IServiceCollection AddNLWebNet<TDataBackend>(this IServiceCollecti
105105
/// <param name="configureOptions">Optional configuration callback</param>
106106
/// <param name="configureMultiBackend">Multi-backend configuration callback</param>
107107
/// <returns>The service collection for chaining</returns>
108-
public static IServiceCollection AddNLWebNetMultiBackend(this IServiceCollection services,
108+
public static IServiceCollection AddNLWebNetMultiBackend(this IServiceCollection services,
109109
Action<NLWebOptions>? configureOptions = null,
110110
Action<MultiBackendOptions>? configureMultiBackend = null)
111111
{

src/NLWebNet/Services/BackendManager.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ public BackendManager(
3434
// otherwise fall back to generic names for backward compatibility
3535
var backendArray = _backends.ToArray();
3636
var configuredEndpoints = _options.Endpoints?.Where(e => e.Value.Enabled).ToList() ?? new List<KeyValuePair<string, BackendEndpointOptions>>();
37-
37+
3838
if (configuredEndpoints.Count > 0 && configuredEndpoints.Count == backendArray.Length)
3939
{
4040
// Use configured endpoint identifiers
@@ -54,7 +54,7 @@ public BackendManager(
5454
}
5555

5656
// Set write backend - for now use the first backend if writeEndpoint is configured
57-
if (!string.IsNullOrEmpty(_options.WriteEndpoint) &&
57+
if (!string.IsNullOrEmpty(_options.WriteEndpoint) &&
5858
_backendsByName.TryGetValue(_options.WriteEndpoint, out var writeBackend))
5959
{
6060
_writeBackend = writeBackend;
@@ -64,7 +64,7 @@ public BackendManager(
6464
_writeBackend = backendArray[0]; // Default to first backend
6565
}
6666

67-
_logger.LogInformation("BackendManager initialized with {BackendCount} backends, WriteEndpoint: {WriteEndpoint}",
67+
_logger.LogInformation("BackendManager initialized with {BackendCount} backends, WriteEndpoint: {WriteEndpoint}",
6868
_backendsByName.Count, _options.WriteEndpoint ?? "default");
6969
}
7070

@@ -82,7 +82,7 @@ public async Task<IEnumerable<NLWebResult>> SearchAsync(string query, string? si
8282
return await firstBackend.SearchAsync(query, site, maxResults, cancellationToken);
8383
}
8484

85-
_logger.LogDebug("Starting parallel search across {BackendCount} backends for query: {Query}",
85+
_logger.LogDebug("Starting parallel search across {BackendCount} backends for query: {Query}",
8686
_backends.Count(), query);
8787

8888
var results = new ConcurrentBag<NLWebResult>();

src/NLWebNet/Services/ResultGenerator.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ public async Task<IEnumerable<NLWebResult>> GenerateListAsync(string query, stri
5959
_logger.LogDebug("Generating list results for query: {Query}", query);
6060

6161
IEnumerable<NLWebResult> results;
62-
62+
6363
if (_backendManager != null)
6464
{
6565
// Use multi-backend manager

tests/NLWebNet.Tests/Extensions/MultiBackendExtensionsTests.cs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ public void AddNLWebNetMultiBackend_RegistersRequiredServices()
2424

2525
// Assert
2626
var serviceProvider = services.BuildServiceProvider();
27-
27+
2828
// Check that all required services are registered
2929
Assert.IsNotNull(serviceProvider.GetService<INLWebService>());
3030
Assert.IsNotNull(serviceProvider.GetService<IQueryProcessor>());
@@ -50,9 +50,9 @@ public void AddNLWebNetMultiBackend_WithMultiBackendDisabled_UsesBackwardCompati
5050
// Assert
5151
var serviceProvider = services.BuildServiceProvider();
5252
var multiBackendOptions = serviceProvider.GetRequiredService<IOptions<MultiBackendOptions>>();
53-
53+
5454
Assert.IsFalse(multiBackendOptions.Value.Enabled);
55-
55+
5656
// Should still be able to get the main service
5757
var nlWebService = serviceProvider.GetService<INLWebService>();
5858
Assert.IsNotNull(nlWebService);
@@ -78,7 +78,7 @@ public void AddNLWebNetMultiBackend_ConfiguresMultiBackendOptions()
7878
var serviceProvider = services.BuildServiceProvider();
7979
var options = serviceProvider.GetRequiredService<IOptions<NLWebOptions>>();
8080
var multiBackendOptions = serviceProvider.GetRequiredService<IOptions<MultiBackendOptions>>();
81-
81+
8282
Assert.AreEqual(QueryMode.Summarize, options.Value.DefaultMode);
8383
Assert.IsTrue(multiBackendOptions.Value.Enabled);
8484
Assert.IsFalse(multiBackendOptions.Value.EnableParallelQuerying);
@@ -98,7 +98,7 @@ public void AddNLWebNetMultiBackend_WithoutConfiguration_UsesDefaults()
9898
var serviceProvider = services.BuildServiceProvider();
9999
var options = serviceProvider.GetRequiredService<IOptions<NLWebOptions>>();
100100
var multiBackendOptions = serviceProvider.GetRequiredService<IOptions<MultiBackendOptions>>();
101-
101+
102102
// Should use default values
103103
Assert.AreEqual(QueryMode.List, options.Value.DefaultMode);
104104
Assert.IsFalse(multiBackendOptions.Value.Enabled); // Default is false for backward compatibility

tests/NLWebNet.Tests/Integration/MultiBackendIntegrationTests.cs

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ public async Task EndToEnd_MultiBackendSearch_WorksCorrectly()
1414
{
1515
// Arrange - Set up a complete multi-backend service configuration
1616
var services = new ServiceCollection();
17-
17+
1818
services.AddNLWebNetMultiBackend(
1919
options =>
2020
{
@@ -61,7 +61,7 @@ public async Task EndToEnd_MultiBackendSearch_WorksCorrectly()
6161
// Verify write backend is accessible
6262
var writeBackend = backendManager.GetWriteBackend();
6363
Assert.IsNotNull(writeBackend, "Should have a write backend available");
64-
64+
6565
var capabilities = writeBackend.GetCapabilities();
6666
Assert.IsNotNull(capabilities, "Write backend should have capabilities");
6767
}
@@ -71,7 +71,7 @@ public async Task EndToEnd_MultiBackendDisabled_FallsBackToSingleBackend()
7171
{
7272
// Arrange - Set up multi-backend service but with multi-backend disabled
7373
var services = new ServiceCollection();
74-
74+
7575
services.AddNLWebNetMultiBackend(
7676
options =>
7777
{
@@ -98,7 +98,7 @@ public async Task EndToEnd_MultiBackendDisabled_FallsBackToSingleBackend()
9898
Assert.IsNotNull(response);
9999
Assert.AreEqual("test-002", response.QueryId);
100100
Assert.IsNull(response.Error, "Response should not have an error");
101-
101+
102102
// Verify configuration
103103
var options = serviceProvider.GetRequiredService<IOptions<NLWebOptions>>();
104104
Assert.IsFalse(options.Value.MultiBackend.Enabled, "Multi-backend should be disabled");
@@ -109,7 +109,7 @@ public async Task EndToEnd_StreamingResponse_WorksWithMultiBackend()
109109
{
110110
// Arrange
111111
var services = new ServiceCollection();
112-
112+
113113
services.AddNLWebNetMultiBackend(options =>
114114
{
115115
options.EnableStreaming = true;
@@ -134,7 +134,7 @@ public async Task EndToEnd_StreamingResponse_WorksWithMultiBackend()
134134
responseCount++;
135135
Assert.IsNotNull(response);
136136
Assert.AreEqual("test-003", response.QueryId);
137-
137+
138138
// Break after a few responses to avoid long test
139139
if (responseCount >= 3) break;
140140
}
@@ -147,7 +147,7 @@ public async Task EndToEnd_DeduplicationAcrossBackends_WorksCorrectly()
147147
{
148148
// Arrange
149149
var services = new ServiceCollection();
150-
150+
151151
services.AddNLWebNetMultiBackend(options =>
152152
{
153153
options.MultiBackend.Enabled = true;
@@ -163,16 +163,16 @@ public async Task EndToEnd_DeduplicationAcrossBackends_WorksCorrectly()
163163
// Assert
164164
var resultList = results.ToList();
165165
var uniqueUrls = resultList.Select(r => r.Url).Distinct().Count();
166-
167-
Assert.AreEqual(resultList.Count, uniqueUrls,
166+
167+
Assert.AreEqual(resultList.Count, uniqueUrls,
168168
"Results should be deduplicated - no duplicate URLs");
169-
169+
170170
if (resultList.Count > 1)
171171
{
172172
// Verify results are sorted by score
173173
var scores = resultList.Select(r => r.Score).ToList();
174174
var sortedScores = scores.OrderByDescending(s => s).ToList();
175-
CollectionAssert.AreEqual(sortedScores, scores,
175+
CollectionAssert.AreEqual(sortedScores, scores,
176176
"Results should be sorted by relevance score");
177177
}
178178
}

tests/NLWebNet.Tests/Services/BackendManagerTests.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ public void Initialize()
1919
_logger = new TestLogger<BackendManager>();
2020
_backend1 = new MockDataBackend(new TestLogger<MockDataBackend>());
2121
_backend2 = new MockDataBackend(new TestLogger<MockDataBackend>());
22-
22+
2323
_options = new MultiBackendOptions
2424
{
2525
Enabled = true,
@@ -219,7 +219,7 @@ public void GetBackendInfo_UsesConfiguredEndpointNames_WhenEndpointsProvided()
219219

220220
// Verify that configured endpoint names are used instead of generic backend_0, backend_1
221221
var backendIds = infoList.Select(info => info.Id).OrderBy(id => id).ToList();
222-
CollectionAssert.AreEqual(new[] { "primary_backend", "secondary_backend" }, backendIds,
222+
CollectionAssert.AreEqual(new[] { "primary_backend", "secondary_backend" }, backendIds,
223223
"Backend IDs should use configured endpoint names");
224224

225225
// Verify write endpoint identification works with configured names
@@ -245,7 +245,7 @@ public void GetBackendInfo_FallsBackToGenericNames_WhenNoEndpointsConfigured()
245245

246246
// Verify that generic names are used as fallback
247247
var backendIds = infoList.Select(info => info.Id).OrderBy(id => id).ToList();
248-
CollectionAssert.AreEqual(new[] { "backend_0", "backend_1" }, backendIds,
248+
CollectionAssert.AreEqual(new[] { "backend_0", "backend_1" }, backendIds,
249249
"Backend IDs should fall back to generic names when no endpoints configured");
250250
}
251251
}

0 commit comments

Comments
 (0)