diff --git a/FocuserTestConsole/FocuserTestConsole.csproj b/FocuserTestConsole/FocuserTestConsole.csproj index a0e33a3..0b2500d 100644 --- a/FocuserTestConsole/FocuserTestConsole.csproj +++ b/FocuserTestConsole/FocuserTestConsole.csproj @@ -10,7 +10,7 @@ Properties ASCOM.MeadeGeneric ASCOM.MeadeGeneric.Test - v4.5 + v4.8 512 diff --git a/FocuserTestConsole/app.config b/FocuserTestConsole/app.config index c5e1dae..786a845 100644 --- a/FocuserTestConsole/app.config +++ b/FocuserTestConsole/app.config @@ -1,3 +1,3 @@ - + diff --git a/Meade.net.Focuser.UnitTests/FocuserUnitTests.cs b/Meade.net.Focuser.UnitTests/FocuserUnitTests.cs index a898637..e5699dc 100644 --- a/Meade.net.Focuser.UnitTests/FocuserUnitTests.cs +++ b/Meade.net.Focuser.UnitTests/FocuserUnitTests.cs @@ -15,6 +15,7 @@ namespace Meade.net.Focuser.UnitTests { private Mock _utilMock; private Mock _sharedResourcesWrapperMock; + private Mock _traceLoggerMock; private ProfileProperties _profileProperties; @@ -40,11 +41,13 @@ namespace Meade.net.Focuser.UnitTests _utilMock = new Mock(); + _traceLoggerMock = new Mock(); + _sharedResourcesWrapperMock = new Mock(); _sharedResourcesWrapperMock.Setup(x => x.ReadProfile()).Returns(() => _profileProperties); - _focuser = new ASCOM.Meade.net.Focuser(_utilMock.Object, _sharedResourcesWrapperMock.Object); + _focuser = new ASCOM.Meade.net.Focuser(_utilMock.Object, _sharedResourcesWrapperMock.Object, _traceLoggerMock.Object); } private void ConnectFocuser() @@ -117,7 +120,7 @@ namespace Meade.net.Focuser.UnitTests _focuser.CommandBlind(expectedMessage, raw); - _sharedResourcesWrapperMock.Verify(x => x.SendBlind(expectedMessage, raw), Times.Once); + _sharedResourcesWrapperMock.Verify(x => x.SendBlind(_traceLoggerMock.Object, expectedMessage, raw), Times.Once); } [Test] @@ -134,13 +137,13 @@ namespace Meade.net.Focuser.UnitTests public void CommandBool_WhenConnected_ThenSendsExpectedMessage(bool raw) { string expectedMessage = "test blind Message"; - _sharedResourcesWrapperMock.Setup(x => x.SendBool(expectedMessage, raw)).Returns(true); + _sharedResourcesWrapperMock.Setup(x => x.SendBool(_traceLoggerMock.Object, expectedMessage, raw)).Returns(true); ConnectFocuser(); var result = _focuser.CommandBool(expectedMessage, raw); - _sharedResourcesWrapperMock.Verify(x => x.SendBool(expectedMessage, raw), Times.Once); + _sharedResourcesWrapperMock.Verify(x => x.SendBool(_traceLoggerMock.Object, expectedMessage, raw), Times.Once); Assert.That(result, Is.True); } @@ -161,11 +164,11 @@ namespace Meade.net.Focuser.UnitTests ConnectFocuser(); - _sharedResourcesWrapperMock.Setup(x => x.SendString(sendMessage, true)).Returns(() => expectedMessage); + _sharedResourcesWrapperMock.Setup(x => x.SendString(_traceLoggerMock.Object, sendMessage, true)).Returns(() => expectedMessage); var actualMessage = _focuser.CommandString(sendMessage, true); - _sharedResourcesWrapperMock.Verify(x => x.SendString(sendMessage, true), Times.Once); + _sharedResourcesWrapperMock.Verify(x => x.SendString(_traceLoggerMock.Object, sendMessage, true), Times.Once); Assert.That(actualMessage, Is.EqualTo(expectedMessage)); } @@ -321,7 +324,7 @@ namespace Meade.net.Focuser.UnitTests _focuser.Halt(); - _sharedResourcesWrapperMock.Verify(x => x.SendBlind("FQ", false), Times.AtLeastOnce); + _sharedResourcesWrapperMock.Verify(x => x.SendBlind(_traceLoggerMock.Object, "FQ", false), Times.AtLeastOnce); } [Test] @@ -411,13 +414,13 @@ namespace Meade.net.Focuser.UnitTests if (position < 0) { - _sharedResourcesWrapperMock.Verify(x => x.SendBlind("F-", false), Times.Once); - _sharedResourcesWrapperMock.Verify(x => x.SendBlind("F+", false), Times.Never); + _sharedResourcesWrapperMock.Verify(x => x.SendBlind(_traceLoggerMock.Object, "F-", false), Times.Once); + _sharedResourcesWrapperMock.Verify(x => x.SendBlind(_traceLoggerMock.Object, "F+", false), Times.Never); } else { - _sharedResourcesWrapperMock.Verify(x => x.SendBlind("F-", false), Times.Never); - _sharedResourcesWrapperMock.Verify(x => x.SendBlind("F+", false), Times.Once); + _sharedResourcesWrapperMock.Verify(x => x.SendBlind(_traceLoggerMock.Object, "F-", false), Times.Never); + _sharedResourcesWrapperMock.Verify(x => x.SendBlind(_traceLoggerMock.Object, "F+", false), Times.Once); } _utilMock.Verify(x => x.WaitForMilliseconds(Math.Abs(position)), Times.Once); @@ -437,16 +440,16 @@ namespace Meade.net.Focuser.UnitTests if (position < 0) { - _sharedResourcesWrapperMock.Verify(x => x.SendBlind("F-", false), Times.Once); - _sharedResourcesWrapperMock.Verify(x => x.SendBlind("F+", false), Times.Never); + _sharedResourcesWrapperMock.Verify(x => x.SendBlind(_traceLoggerMock.Object, "F-", false), Times.Once); + _sharedResourcesWrapperMock.Verify(x => x.SendBlind(_traceLoggerMock.Object, "F+", false), Times.Never); _utilMock.Verify(x => x.WaitForMilliseconds(Math.Abs(position)), Times.Once); _utilMock.Verify(x => x.WaitForMilliseconds(Math.Abs(_profileProperties.BacklashCompensation)), Times.Never); _utilMock.Verify(x => x.WaitForMilliseconds(100), Times.Exactly(1)); } else { - _sharedResourcesWrapperMock.Verify(x => x.SendBlind("F-", false), Times.Once); - _sharedResourcesWrapperMock.Verify(x => x.SendBlind("F+", false), Times.Once); + _sharedResourcesWrapperMock.Verify(x => x.SendBlind(_traceLoggerMock.Object, "F-", false), Times.Once); + _sharedResourcesWrapperMock.Verify(x => x.SendBlind(_traceLoggerMock.Object, "F+", false), Times.Once); _utilMock.Verify(x => x.WaitForMilliseconds(Math.Abs(position) + _profileProperties.BacklashCompensation), Times.Once); _utilMock.Verify(x => x.WaitForMilliseconds(_profileProperties.BacklashCompensation), Times.Once); _utilMock.Verify(x => x.WaitForMilliseconds(100), Times.Exactly(2)); diff --git a/Meade.net.Setup/Product.wxs b/Meade.net.Setup/Product.wxs index e2e47d8..83a573a 100644 --- a/Meade.net.Setup/Product.wxs +++ b/Meade.net.Setup/Product.wxs @@ -21,13 +21,13 @@ Type="raw" /> - - = "6.4.1"]]> + + = "6.6.0"]]> - - - + + + = "#528040")]]>