Skip to content

Commit

Permalink
Fixing a couple tests
Browse files Browse the repository at this point in the history
  • Loading branch information
daveaglick committed Sep 23, 2019
1 parent 5ad3a43 commit f98da78
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 5 deletions.
3 changes: 2 additions & 1 deletion src/core/Statiq.Testing/Execution/TestPipelineCollection.cs
Expand Up @@ -7,7 +7,8 @@ namespace Statiq.Testing
{
public class TestPipelineCollection : IPipelineCollection
{
private readonly Dictionary<string, IPipeline> _pipelines = new Dictionary<string, IPipeline>();
private readonly Dictionary<string, IPipeline> _pipelines =
new Dictionary<string, IPipeline>(StringComparer.OrdinalIgnoreCase);

public IPipeline Add(string name)
{
Expand Down
Expand Up @@ -61,9 +61,9 @@ public async Task NoPipelinesWarning()
&& x.FormattedMessage == "No pipelines are configured or specified.");
}

[TestCase("Trace", 6)]
[TestCase("Debug", 5)]
[TestCase("Information", 4)]
[TestCase("Trace", 19)] // Includes module start/finish
[TestCase("Debug", 18)] // Include modules start/finish
[TestCase("Information", 5)] // Includes pipeline finish
[TestCase("Warning", 3)]
[TestCase("Error", 2)]
[TestCase("Critical", 1)]
Expand Down Expand Up @@ -92,7 +92,7 @@ public async Task SetsLogLevel(string logLevel, int expected)

// Then
exitCode.ShouldBe((int)ExitCode.Normal);
provider.Messages.Count(x => x.FormattedMessage.StartsWith("Foo")).ShouldBe(expected);
provider.Messages.Count(x => x.FormattedMessage.StartsWith("Foo/Process")).ShouldBe(expected);
}

[Test]
Expand Down

0 comments on commit f98da78

Please sign in to comment.