Abstracted beeps for testing.

Author: DerekGooding

ConsoleHero.Test/BeepHelperTests.cs

@@ -0,0 +1,72 @@
+using ConsoleHero.Helpers;
+using ConsoleHero.Interfaces;
+using Moq;
+
+namespace ConsoleHero.Test;
+
+[TestClass]
+public class BeepHelperTests
+{
+#pragma warning disable CS8618 //Allow nullable. TestInitialize ensures.
+    private Mock<IPlatformHelper> _platformHelperMock;
+    private BeepHelper _beepHelper;
+#pragma warning restore CS8618
+
+    [TestInitialize]
+    public void TestInitialize()
+    {
+        _platformHelperMock = new Mock<IPlatformHelper>();
+        _beepHelper = new BeepHelper(_platformHelperMock.Object);
+    }
+
+    [TestMethod]
+    public void Beep_NoParameters_WritesBellCharacter()
+    {
+        using ConsoleOutput consoleOutput = new();
+
+        BeepHelper.Beep();
+
+        // Assert
+        Assert.AreEqual("\a", consoleOutput.GetOutput());
+    }
+
+    [TestMethod]
+    public void Beep_LinuxPlatform_StartsProcess()
+    {
+        _platformHelperMock.Setup(p => p.IsWindows).Returns(false);
+        _platformHelperMock.Setup(p => p.IsLinux).Returns(true);
+
+        // Act
+        _beepHelper.Beep(1000, 500);
+
+        // Assert
+        // Add appropriate assertions for process calls if applicable.
+    }
+
+    [TestMethod]
+    public void Beep_MacOSPlatform_StartsProcess()
+    {
+        _platformHelperMock.Setup(p => p.IsWindows).Returns(false);
+        _platformHelperMock.Setup(p => p.IsOSX).Returns(true);
+
+        // Act
+        _beepHelper.Beep(1000, 500);
+
+        // Assert
+        // Add appropriate assertions for process calls if applicable.
+    }
+
+    [TestMethod]
+    public void Beep_NonWindowsPlatformWithException_WritesBellCharacter()
+    {
+        _platformHelperMock.Setup(p => p.IsWindows).Returns(false);
+        _platformHelperMock.Setup(p => p.IsLinux).Returns(true);
+        using ConsoleOutput consoleOutput = new();
+
+        // Act
+        _beepHelper.Beep(1000, 500);
+
+        // Assert
+        Assert.AreEqual("\a", consoleOutput.GetOutput());
+    }
+}

ConsoleHero.Test/HelperTests.cs renamed to ConsoleHero.Test/ColorHelperTests.cs

@@ -3,7 +3,7 @@
 namespace ConsoleHero.Test;
 
 [TestClass]
-public class HelperTests
+public class ColorHelperTests
 {
     private readonly StringWriter _consoleOutput = new();
 

ConsoleHero.Test/ConsoleOutput.cs

@@ -0,0 +1,22 @@
+namespace ConsoleHero.Test;
+public class ConsoleOutput : IDisposable
+{
+    private readonly StringWriter _stringWriter;
+    private readonly TextWriter _originalOutput;
+
+    public ConsoleOutput()
+    {
+        _stringWriter = new StringWriter();
+        _originalOutput = Console.Out;
+        Console.SetOut(_stringWriter);
+    }
+
+    public string GetOutput() => _stringWriter.ToString();
+
+    public void Dispose()
+    {
+        GC.SuppressFinalize(this);
+        Console.SetOut(_originalOutput);
+        _stringWriter.Dispose();
+    }
+}

ConsoleHero/AssemblyInfo.cs

@@ -1,3 +1,4 @@
 using System.Runtime.CompilerServices;
 
-[assembly: InternalsVisibleTo("ConsoleHero.Test")]
+[assembly: InternalsVisibleTo("ConsoleHero.Test")]
+[assembly: InternalsVisibleTo("DynamicProxyGenAssembly2")]

ConsoleHero/Helpers/BeepHelper.cs

@@ -1,18 +1,21 @@
-using System.Runtime.InteropServices;
+using ConsoleHero.Interfaces;
 
 namespace ConsoleHero.Helpers;
 
-internal static class BeepHelper
+internal class BeepHelper(IPlatformHelper plateformHelper)
 {
+    private readonly IPlatformHelper _platformHelper = plateformHelper;
+
     internal static void Beep() => Write("\a");
 
-    internal static void Beep(int frequency, int duration)
+    [System.Diagnostics.CodeAnalysis.SuppressMessage("Interoperability", "CA1416:Validate platform compatibility", Justification = "Ensured by _platformHelper")]
+    internal void Beep(int frequency, int duration)
     {
-        if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
+        if (_platformHelper.IsWindows)
         {
             Console.Beep(frequency, duration);
         }
-        else if (RuntimeInformation.IsOSPlatform(OSPlatform.Linux) || RuntimeInformation.IsOSPlatform(OSPlatform.OSX))
+        else if (_platformHelper.IsLinux || _platformHelper.IsOSX)
         {
             try
             {

ConsoleHero/Interfaces/IPlateformHelper.cs

@@ -0,0 +1,7 @@
+namespace ConsoleHero.Interfaces;
+internal interface IPlatformHelper
+{
+    bool IsWindows { get; }
+    bool IsLinux { get; }
+    bool IsOSX { get; }
+}

ConsoleHero/Interfaces/PlatformHelper.cs

@@ -0,0 +1,10 @@
+using System.Runtime.InteropServices;
+
+namespace ConsoleHero.Interfaces;
+
+internal class PlatformHelper : IPlatformHelper
+{
+    public bool IsWindows => RuntimeInformation.IsOSPlatform(OSPlatform.Windows);
+    public bool IsLinux => RuntimeInformation.IsOSPlatform(OSPlatform.Linux);
+    public bool IsOSX => RuntimeInformation.IsOSPlatform(OSPlatform.OSX);
+}

ConsoleHero/Tune.cs

@@ -1,4 +1,5 @@
 using ConsoleHero.Helpers;
+using ConsoleHero.Interfaces;
 
 namespace ConsoleHero;
 
@@ -9,6 +10,7 @@ namespace ConsoleHero;
 /// </summary>
 public record Tune : INode
 {
+    internal BeepHelper _beepHelper = new(new PlatformHelper());
     internal Tune() { }
     internal List<Note> Notes { get; set; } = [];
     internal bool Wait { get; set; } = true;
@@ -28,7 +30,7 @@ internal void Play()
 
         foreach (Note item in Notes)
         {
-            BeepHelper.Beep(item.NoteTone, item.NoteDuration);
+            _beepHelper.Beep(item.NoteTone, item.NoteDuration);
         }
     }