Skip to content

Commit

Permalink
remove: Explicit attribute in tests
Browse files Browse the repository at this point in the history
  • Loading branch information
Romazes committed Feb 13, 2024
1 parent aea3b12 commit 1e4dfae
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 11 deletions.
2 changes: 1 addition & 1 deletion QuantConnect.CoinAPI.Tests/CoinAPIDataDownloaderTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@

namespace QuantConnect.CoinAPI.Tests
{
[TestFixture, Explicit("")]
[TestFixture]
public class CoinAPIDataDownloaderTests
{
private CoinAPIDataDownloader _downloader;
Expand Down
2 changes: 1 addition & 1 deletion QuantConnect.CoinAPI.Tests/CoinAPIHistoryProviderTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down
2 changes: 1 addition & 1 deletion QuantConnect.CoinAPI.Tests/CoinAPISymbolMapperTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@

namespace QuantConnect.CoinAPI.Tests
{
[TestFixture, Explicit("")]
[TestFixture]
public class CoinAPISymbolMapperTests
{
private CoinApiSymbolMapper _coinApiSymbolMapper;
Expand Down
13 changes: 5 additions & 8 deletions QuantConnect.CoinAPI.Tests/CoinApiDataQueueHandlerTest.cs
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ public void TearDown()
}
}

[Test, Explicit("")]
[Test]
public void SubscribeToBTCUSDSecondOnCoinbaseDataStreamTest()
{
var resetEvent = new AutoResetEvent(false);
Expand All @@ -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)
Expand All @@ -83,7 +82,7 @@ public void SubscribeToBTCUSDSecondOnCoinbaseDataStreamTest()
_cancellationTokenSource.Cancel();
}

[Test, Explicit("")]
[Test]
public void SubscribeToBTCUSDSecondOnDifferentMarkets()
{
var resetEvent = new AutoResetEvent(false);
Expand Down Expand Up @@ -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);
},
() =>
Expand Down Expand Up @@ -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)
Expand Down

0 comments on commit 1e4dfae

Please sign in to comment.