diff --git a/actions/docs-verifier/src/RedirectionVerifier/RedirectTargetVerifier.cs b/actions/docs-verifier/src/RedirectionVerifier/RedirectTargetVerifier.cs index 6a3817c3..a1782002 100644 --- a/actions/docs-verifier/src/RedirectionVerifier/RedirectTargetVerifier.cs +++ b/actions/docs-verifier/src/RedirectionVerifier/RedirectTargetVerifier.cs @@ -39,6 +39,8 @@ await WriteErrorAsync( return false; } + await writer.WriteLineAsync($"Checking redirect targets in '{redirectionFilePath}'..."); + OpenPublishingRedirectionReader reader = new(redirectionFilePath); ImmutableArray redirections = await reader.MapConfigurationAsync(); if (redirections.IsDefaultOrEmpty) diff --git a/actions/docs-verifier/src/RedirectionVerifier/RedirectionHelpers.cs b/actions/docs-verifier/src/RedirectionVerifier/RedirectionHelpers.cs index b8dd2e44..0a2c9921 100644 --- a/actions/docs-verifier/src/RedirectionVerifier/RedirectionHelpers.cs +++ b/actions/docs-verifier/src/RedirectionVerifier/RedirectionHelpers.cs @@ -15,11 +15,7 @@ public static async Task> GetRedirectionFileNames() // If no redirection files are found in the OPS config, just use the default name. ImmutableArray redirectionFileNames = await GetRedirectionFilesAsync() ?? [".openpublishing.redirection.json"]; - Console.WriteLine($"The following {redirectionFileNames.Length} redirection files are registered:"); - foreach (string filename in redirectionFileNames) - { - Console.WriteLine(filename); - } + Console.WriteLine($"Found {redirectionFileNames.Length} registered redirection files."); return redirectionFileNames; } diff --git a/actions/docs-verifier/tests/GitHub.UnitTests/RedirectTargetVerifierTests.cs b/actions/docs-verifier/tests/GitHub.UnitTests/RedirectTargetVerifierTests.cs index 3f2ecd97..bc1ae9d5 100644 --- a/actions/docs-verifier/tests/GitHub.UnitTests/RedirectTargetVerifierTests.cs +++ b/actions/docs-verifier/tests/GitHub.UnitTests/RedirectTargetVerifierTests.cs @@ -19,7 +19,6 @@ public async Task WriteResultsAsyncReturnsTrueForValidLearnUrlPath() _ => Task.FromResult(HttpStatusCode.OK)); Assert.True(result); - Assert.Equal(string.Empty, writer.ToString()); } finally { @@ -46,7 +45,6 @@ public async Task WriteResultsAsyncSkipsNonLearnUrlTargets() }); Assert.True(result); - Assert.Equal(string.Empty, writer.ToString()); Assert.False(statusProviderCalled); } finally