From 1e4dfae463b53a6907d98b9a82123a10783e1a69 Mon Sep 17 00:00:00 2001 From: Romazes Date: Wed, 14 Feb 2024 01:16:03 +0200 Subject: [PATCH] remove: Explicit attribute in tests --- .../CoinAPIDataDownloaderTests.cs | 2 +- .../CoinAPIHistoryProviderTests.cs | 2 +- .../CoinAPISymbolMapperTests.cs | 2 +- .../CoinApiDataQueueHandlerTest.cs | 13 +++++-------- 4 files changed, 8 insertions(+), 11 deletions(-) diff --git a/QuantConnect.CoinAPI.Tests/CoinAPIDataDownloaderTests.cs b/QuantConnect.CoinAPI.Tests/CoinAPIDataDownloaderTests.cs index 6b90538..669d48a 100644 --- a/QuantConnect.CoinAPI.Tests/CoinAPIDataDownloaderTests.cs +++ b/QuantConnect.CoinAPI.Tests/CoinAPIDataDownloaderTests.cs @@ -19,7 +19,7 @@ namespace QuantConnect.CoinAPI.Tests { - [TestFixture, Explicit("")] + [TestFixture] public class CoinAPIDataDownloaderTests { private CoinAPIDataDownloader _downloader; diff --git a/QuantConnect.CoinAPI.Tests/CoinAPIHistoryProviderTests.cs b/QuantConnect.CoinAPI.Tests/CoinAPIHistoryProviderTests.cs index 102c038..a7c93fd 100644 --- a/QuantConnect.CoinAPI.Tests/CoinAPIHistoryProviderTests.cs +++ b/QuantConnect.CoinAPI.Tests/CoinAPIHistoryProviderTests.cs @@ -22,7 +22,7 @@ namespace QuantConnect.CoinAPI.Tests { - [TestFixture, Explicit("")] + [TestFixture] public class CoinAPIHistoryProviderTests { private static readonly Symbol _CoinbaseBtcUsdSymbol = Symbol.Create("BTCUSD", SecurityType.Crypto, Market.Coinbase); diff --git a/QuantConnect.CoinAPI.Tests/CoinAPISymbolMapperTests.cs b/QuantConnect.CoinAPI.Tests/CoinAPISymbolMapperTests.cs index b1a8fc0..7303fc1 100644 --- a/QuantConnect.CoinAPI.Tests/CoinAPISymbolMapperTests.cs +++ b/QuantConnect.CoinAPI.Tests/CoinAPISymbolMapperTests.cs @@ -18,7 +18,7 @@ namespace QuantConnect.CoinAPI.Tests { - [TestFixture, Explicit("")] + [TestFixture] public class CoinAPISymbolMapperTests { private CoinApiSymbolMapper _coinApiSymbolMapper; diff --git a/QuantConnect.CoinAPI.Tests/CoinApiDataQueueHandlerTest.cs b/QuantConnect.CoinAPI.Tests/CoinApiDataQueueHandlerTest.cs index 474b46c..c63d45a 100644 --- a/QuantConnect.CoinAPI.Tests/CoinApiDataQueueHandlerTest.cs +++ b/QuantConnect.CoinAPI.Tests/CoinApiDataQueueHandlerTest.cs @@ -47,7 +47,7 @@ public void TearDown() } } - [Test, Explicit("")] + [Test] public void SubscribeToBTCUSDSecondOnCoinbaseDataStreamTest() { var resetEvent = new AutoResetEvent(false); @@ -61,8 +61,7 @@ public void SubscribeToBTCUSDSecondOnCoinbaseDataStreamTest() _cancellationTokenSource.Token, tick => { - // Log.Debug($"{nameof(CoinApiDataQueueHandlerTest)}: tick: {tick}"); - Log.Trace($"{nameof(CoinApiDataQueueHandlerTest)}.{nameof(ProcessFeed)}: tick {tick}"); + Log.Debug($"{nameof(CoinApiDataQueueHandlerTest)}.{nameof(SubscribeToBTCUSDSecondOnCoinbaseDataStreamTest)}: {tick}"); tradeBars.Add(tick); if (tradeBars.Count > 5) @@ -83,7 +82,7 @@ public void SubscribeToBTCUSDSecondOnCoinbaseDataStreamTest() _cancellationTokenSource.Cancel(); } - [Test, Explicit("")] + [Test] public void SubscribeToBTCUSDSecondOnDifferentMarkets() { var resetEvent = new AutoResetEvent(false); @@ -112,8 +111,7 @@ public void SubscribeToBTCUSDSecondOnDifferentMarkets() _cancellationTokenSource.Token, tick => { - // Log.Debug($"{nameof(CoinApiDataQueueHandlerTest)}: tick: {tick}"); - Log.Trace($"{nameof(CoinApiDataQueueHandlerTest)}.{nameof(ProcessFeed)}: tick {tick}"); + Log.Debug($"{nameof(CoinApiDataQueueHandlerTest)}.{nameof(SubscribeToBTCUSDSecondOnDifferentMarkets)}: {tick}"); symbolBaseData[tick.Symbol].Add(tick); }, () => @@ -167,8 +165,7 @@ public void SubscribeToBTCUSDTFutureSecondBinance() _cancellationTokenSource.Token, tick => { - //Log.Debug($"{nameof(CoinApiDataQueueHandlerTest)}: tick: {tick}"); - Log.Trace($"{nameof(CoinApiDataQueueHandlerTest)}.{nameof(ProcessFeed)}: tick {tick}"); + Log.Debug($"{nameof(CoinApiDataQueueHandlerTest)}.{nameof(SubscribeToBTCUSDTFutureSecondBinance)}: {tick}"); tickData.Add(tick); if (tickData.Count > 5)