Skip to content

Restructure more of the tests #283

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
TraGicCode merged 2 commits into from
Apr 14, 2026
+35 −74
Merged

Restructure more of the tests #283

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
ad39f01
Restructure more of the tests
TraGicCode Apr 14, 2026
685bbe3
fix format issue
TraGicCode Apr 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 21 additions & 0 deletions tests/BuslyCLI.Console.Tests/Commands/CommandTestBase.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
using BuslyCLI.Infrastructure;
using Microsoft.Extensions.DependencyInjection;
using Spectre.Console.Cli.Extensions.DependencyInjection;
using Spectre.Console.Cli.Testing;

namespace BuslyCLI.Console.Tests.Commands;

public abstract class CommandTestBase
{
protected CommandAppTester Sut { get; private set; } = null!;

[SetUp]
public void Setup()
{
var registrations = new ServiceCollection();
registrations.AddBuslyCLIServices();
using var registrar = new DependencyInjectionRegistrar(registrations);
Sut = new CommandAppTester(registrar);
Sut.Configure(AppConfiguration.GetSpectreCommandConfiguration());
}
}
21 changes: 3 additions & 18 deletions tests/BuslyCLI.Console.Tests/Commands/Transport/CurrentTransportTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,25 +1,10 @@
using BuslyCLI.Console.Tests.TestHelpers;
using BuslyCLI.Infrastructure;
using Microsoft.Extensions.DependencyInjection;
using Spectre.Console.Cli.Extensions.DependencyInjection;
using Spectre.Console.Cli.Testing;
using Spectre.Console.Testing;

namespace BuslyCLI.Console.Tests.Commands.Transport;

public class CurrentTransportTests
public class CurrentTransportTests : CommandTestBase
{
private CommandAppTester _sut;

[SetUp]
public void Setup()
{
var registrations = new ServiceCollection();
registrations.AddBuslyCLIServices();
using var registrar = new DependencyInjectionRegistrar(registrations);
_sut = new CommandAppTester(registrar);
_sut.Configure(AppConfiguration.GetSpectreCommandConfiguration());
}

[Test]
public void ShouldOutputCurrentTransport()
Expand All @@ -36,7 +21,7 @@ public void ShouldOutputCurrentTransport()
using var configFile = new TestableNServiceBusConfigurationFile(yamlFile);

// Act
var result = _sut.Run("transport", "current", "--config", configFile.FilePath);
var result = Sut.Run("transport", "current", "--config", configFile.FilePath);

// Assert
Assert.That(result.ExitCode, Is.EqualTo(0));
Expand All @@ -61,7 +46,7 @@ public void ShouldOutputCurrentTransportNotConfigured()
using var configFile = new TestableNServiceBusConfigurationFile(yamlFile);

// Act
var result = _sut.Run("transport", "current", "--config", configFile.FilePath);
var result = Sut.Run("transport", "current", "--config", configFile.FilePath);

// Assert
Assert.That(result.ExitCode, Is.EqualTo(0));
Expand Down
21 changes: 3 additions & 18 deletions tests/BuslyCLI.Console.Tests/Commands/Transport/DeleteTransportTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,25 +1,10 @@
using BuslyCLI.Console.Tests.TestHelpers;
using BuslyCLI.Infrastructure;
using Microsoft.Extensions.DependencyInjection;
using Spectre.Console.Cli.Extensions.DependencyInjection;
using Spectre.Console.Cli.Testing;
using Spectre.Console.Testing;

namespace BuslyCLI.Console.Tests.Commands.Transport;

public class DeleteTransportTests
public class DeleteTransportTests : CommandTestBase
{
private CommandAppTester _sut;

[SetUp]
public void Setup()
{
var registrations = new ServiceCollection();
registrations.AddBuslyCLIServices();
using var registrar = new DependencyInjectionRegistrar(registrations);
_sut = new CommandAppTester(registrar);
_sut.Configure(AppConfiguration.GetSpectreCommandConfiguration());
}

[Test]
public void ShouldBeIdempotentWhenDeletingNonExistingTransport()
Expand All @@ -37,7 +22,7 @@ public void ShouldBeIdempotentWhenDeletingNonExistingTransport()
var nonExistingTransport = Guid.NewGuid().ToString();

// Act
var result = _sut.Run("transport", "delete", nonExistingTransport, "--config", configFile.FilePath);
var result = Sut.Run("transport", "delete", nonExistingTransport, "--config", configFile.FilePath);

Assert.That(result.ExitCode, Is.EqualTo(0));
Assert.That(result.Output, Is.EqualTo($"Cannot delete transport {nonExistingTransport} since it doesn't exist in the config file."));
Expand All @@ -58,7 +43,7 @@ public void ShouldDeleteTransport()
using var configFile = new TestableNServiceBusConfigurationFile(yamlFile);

// Act
var result = _sut.Run("transport", "delete", "local-learning", "--config", configFile.FilePath);
var result = Sut.Run("transport", "delete", "local-learning", "--config", configFile.FilePath);

Assert.That(result.ExitCode, Is.EqualTo(0));
Assert.That(result.Output, Is.EqualTo(
Expand Down
23 changes: 4 additions & 19 deletions tests/BuslyCLI.Console.Tests/Commands/Transport/ListTransportTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,25 +1,10 @@
using BuslyCLI.Console.Tests.TestHelpers;
using BuslyCLI.Infrastructure;
using Microsoft.Extensions.DependencyInjection;
using Spectre.Console.Cli.Extensions.DependencyInjection;
using Spectre.Console.Cli.Testing;
using Spectre.Console.Testing;

namespace BuslyCLI.Console.Tests.Commands.Transport;

public class ListTransportTests
public class ListTransportTests : CommandTestBase
{
private CommandAppTester _sut;

[SetUp]
public void Setup()
{
var registrations = new ServiceCollection();
registrations.AddBuslyCLIServices();
using var registrar = new DependencyInjectionRegistrar(registrations);
_sut = new CommandAppTester(registrar);
_sut.Configure(AppConfiguration.GetSpectreCommandConfiguration());
}

[Test]
public void ShouldOutputAnEmptyGridWhenConfigFileIsEmptyYaml()
Expand All @@ -29,7 +14,7 @@ public void ShouldOutputAnEmptyGridWhenConfigFileIsEmptyYaml()
---
""";
using var configFile = new TestableNServiceBusConfigurationFile(yamlFile);
var result = _sut.Run("transport", "list", "--config", configFile.FilePath);
var result = Sut.Run("transport", "list", "--config", configFile.FilePath);

Assert.That(result.ExitCode, Is.EqualTo(0));
Assert.That(result.Output, Is.EqualTo("CURRENT NAME TRANSPORT-TYPE"));
Expand All @@ -44,7 +29,7 @@ public void ShouldOutputAnEmptyGridWhenTransportArrayIsEmpty()
transports:
""";
using var configFile = new TestableNServiceBusConfigurationFile(yamlFile);
var result = _sut.Run("transport", "list", "--config", configFile.FilePath);
var result = Sut.Run("transport", "list", "--config", configFile.FilePath);

Assert.That(result.ExitCode, Is.EqualTo(0));
Assert.That(result.Output, Is.EqualTo("CURRENT NAME TRANSPORT-TYPE"));
Expand All @@ -64,7 +49,7 @@ public void ShouldOutputASingleTransport()
using var configFile = new TestableNServiceBusConfigurationFile(yamlFile);

// Act
var result = _sut.Run("transport", "list", "--config", configFile.FilePath);
var result = Sut.Run("transport", "list", "--config", configFile.FilePath);

// Assert
Assert.That(result.ExitCode, Is.EqualTo(0));
Expand Down
23 changes: 4 additions & 19 deletions tests/BuslyCLI.Console.Tests/Commands/Transport/SetTransportTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,25 +1,10 @@
using BuslyCLI.Console.Tests.TestHelpers;
using BuslyCLI.Infrastructure;
using Microsoft.Extensions.DependencyInjection;
using Spectre.Console.Cli.Extensions.DependencyInjection;
using Spectre.Console.Cli.Testing;
using Spectre.Console.Testing;

namespace BuslyCLI.Console.Tests.Commands.Transport;

public class SetTransportTests
public class SetTransportTests : CommandTestBase
{
private CommandAppTester _sut;

[SetUp]
public void Setup()
{
var registrations = new ServiceCollection();
registrations.AddBuslyCLIServices();
using var registrar = new DependencyInjectionRegistrar(registrations);
_sut = new CommandAppTester(registrar);
_sut.Configure(AppConfiguration.GetSpectreCommandConfiguration());
}

[Test]
public void ShouldOutputAMessageWhenTransportDoesNotExist()
Expand All @@ -37,7 +22,7 @@ public void ShouldOutputAMessageWhenTransportDoesNotExist()
var nonExistingTransport = $"{Guid.NewGuid():N}";

// Act
var result = _sut.Run("transport", "set", nonExistingTransport, "--config", configFile.FilePath);
var result = Sut.Run("transport", "set", nonExistingTransport, "--config", configFile.FilePath);

// Assert
Assert.That(result.ExitCode, Is.EqualTo(0));
Expand Down Expand Up @@ -66,7 +51,7 @@ public void ShouldOutputAMessageWhenTransportIsSet()
using var configFile = new TestableNServiceBusConfigurationFile(yamlFile);

// Act
var result = _sut.Run("transport", "set", "local-learning2", "--config", configFile.FilePath);
var result = Sut.Run("transport", "set", "local-learning2", "--config", configFile.FilePath);

// Assert
Assert.That(result.ExitCode, Is.EqualTo(0));
Expand All @@ -92,7 +77,7 @@ public void ShouldBeIdempotentWhenSettingTransportToTheAlreadyConfiguredTranspor
using var configFile = new TestableNServiceBusConfigurationFile(yamlFile);

// Act
var result = _sut.Run("transport", "set", "local-learning", "--config", configFile.FilePath);
var result = Sut.Run("transport", "set", "local-learning", "--config", configFile.FilePath);

// Assert
Assert.That(result.ExitCode, Is.EqualTo(0));
Expand Down