From 04c2f5e1bb10f56dc7639990f6f88ff1ee1ce301 Mon Sep 17 00:00:00 2001 From: Colin Dawson Date: Tue, 12 Nov 2024 18:52:03 +0000 Subject: [PATCH] Added unit testing to cover the LX200 not being able to park. --- .../TelescopeUnitTests.cs | 39 +++++++------------ 1 file changed, 13 insertions(+), 26 deletions(-) diff --git a/Meade.net.Telescope.UnitTests/TelescopeUnitTests.cs b/Meade.net.Telescope.UnitTests/TelescopeUnitTests.cs index 7add67e..9648436 100644 --- a/Meade.net.Telescope.UnitTests/TelescopeUnitTests.cs +++ b/Meade.net.Telescope.UnitTests/TelescopeUnitTests.cs @@ -911,12 +911,14 @@ namespace Meade.net.Telescope.UnitTests Assert.That(result, Is.EqualTo(expected)); } - [Test] - public void CanPark_Get_ReturnsTrue() + [TestCase("Autostar", "43Eg", true)] + [TestCase("LX200 Classic", "", false)] + public void CanPark_Get_ReturnsTrue(string productName, string firmware, bool canPark) { + ConnectTelescope(productName, firmware, $"{_testProperties.AlignmentMode}N0"); var result = _telescope.CanPark; - Assert.That(result, Is.True); + Assert.That(result, Is.EqualTo(canPark)); } [TestCase("Autostar", "43Eg")] @@ -1621,30 +1623,15 @@ namespace Meade.net.Telescope.UnitTests Assert.That(_telescope.AtPark, Is.True); } - //[Test] - //public void Park_WhenLX200NotParked_ThenSendsParkCommand() - //{ - // var alt = 1.0; - // var altAsDM = "01*00"; - // _utilMock.Setup(x => x.DegreesToDM(alt, "*", "", 2)).Returns(altAsDM); + [Test] + public void Park_WhenLX200ClassicNotParked_ThenSendsParkCommand() + { + ConnectTelescope(TelescopeList.LX200CLASSIC); + Assert.That(_telescope.AtPark, Is.False); + _sharedResourcesWrapperMock.Verify(x => x.SendBlind(_traceLoggerMock.Object, "hP", false), Times.Never); - // var az = 180; - // var azAsDM = "180*00"; - // _utilMock.Setup(x => x.DegreesToDM(az, "*", "", 2)).Returns(azAsDM); - - // //_sharedResourcesWrapperMock.Setup(x => x.SendBool(_traceLoggerMock.Object, "Sa+77*30", false)).Returns(true); - // //_sharedResourcesWrapperMock.Setup(x => x.SendBool(_traceLoggerMock.Object, "Sz180*00", false)).Returns(true); - - // _sharedResourcesWrapperMock.Setup(x => x.SendBool(_traceLoggerMock.Object, "Sa+01*00", false)).Returns(true); - // _sharedResourcesWrapperMock.Setup(x => x.SendBool(_traceLoggerMock.Object, "Sz180*00", false)).Returns(true); - - // ConnectTelescope(TelescopeList.LX200CLASSIC); - // Assert.That(_telescope.AtPark, Is.False); - - // _telescope.Park(); - - // Assert.That(_telescope.AtPark, Is.True); - //} + Assert.Throws(() => { _telescope.Park(); }); + } [Test] public void Park_WhenParked_ThenDoesNothing()