Minor unit tests fixes

Author: jhonabreul

Tests/Algorithm/AlgorithmAddDataTests.cs

@@ -65,8 +65,10 @@ public void DefaultDataFeeds_CanBeOverwritten_Successfully()
 
             // new forex - should be tradebar
             var forexQuote = algo.AddForex("EURUSD");
-            Assert.IsTrue(forexQuote.Subscriptions.Count() == 1);
+            // The quote bar subscription is kept
+            Assert.IsTrue(forexQuote.Subscriptions.Count() == 2);
             Assert.IsTrue(GetMatchingSubscription(algo, forexQuote.Symbol, typeof(TradeBar)) != null);
+            Assert.IsTrue(GetMatchingSubscription(algo, forexQuote.Symbol, typeof(QuoteBar)) != null);
 
             // reset to empty string, affects other tests because config is static
             Config.Set("security-data-feeds", "");
@@ -91,9 +93,20 @@ public void DefaultDataFeeds_AreAdded_Successfully()
 
             // equity low resolution
             var equityDaily = algo.AddSecurity(SecurityType.Equity, "goog", Resolution.Daily);
-            Assert.IsTrue(equityDaily.Subscriptions.Count() == 2);
+            Assert.IsTrue(equityDaily.Subscriptions.Count() == 3);
             Assert.IsTrue(GetMatchingSubscription(algo, equityDaily.Symbol, typeof(TradeBar)) != null);
+            Assert.IsTrue(GetMatchingSubscription(algo, equityMinute.Symbol, typeof(QuoteBar)) != null);
+
+            Assert.IsTrue(ReferenceEquals(equityMinute, equityDaily));
 
+            var equitySubscriptions = algo.SubscriptionManager.SubscriptionDataConfigService
+                .GetSubscriptionDataConfigs(equityMinute.Symbol);
+            Assert.IsTrue(equitySubscriptions.SingleOrDefault(
+                s => s.TickType == TickType.Trade && s.Type == typeof(TradeBar) && s.Resolution == Resolution.Minute) != null);
+            Assert.IsTrue(equitySubscriptions.SingleOrDefault(
+                s => s.TickType == TickType.Quote && s.Type == typeof(QuoteBar) && s.Resolution == Resolution.Minute) != null);
+            Assert.IsTrue(equitySubscriptions.SingleOrDefault(
+                s => s.TickType == TickType.Trade && s.Type == typeof(TradeBar) && s.Resolution == Resolution.Daily) != null);
 
             // option
             var option = algo.AddSecurity(SecurityType.Option, "goog");
@@ -116,16 +129,29 @@ public void DefaultDataFeeds_AreAdded_Successfully()
             Assert.IsTrue(future.Subscriptions.FirstOrDefault(x => typeof(FutureUniverse) == x.Type) != null);
 
             // Crypto high resolution
-            var cryptoMinute = algo.AddSecurity(SecurityType.Equity, "goog");
+            var cryptoMinute = algo.AddSecurity(SecurityType.Crypto, "btcusd");
             Assert.IsTrue(cryptoMinute.Subscriptions.Count() == 2);
             Assert.IsTrue(GetMatchingSubscription(algo, cryptoMinute.Symbol, typeof(TradeBar)) != null);
             Assert.IsTrue(GetMatchingSubscription(algo, cryptoMinute.Symbol, typeof(QuoteBar)) != null);
 
             // Crypto low resolution
             var cryptoHourly = algo.AddSecurity(SecurityType.Crypto, "btcusd", Resolution.Hour);
-            Assert.IsTrue(cryptoHourly.Subscriptions.Count() == 2);
+            Assert.IsTrue(cryptoHourly.Subscriptions.Count() == 4);
             Assert.IsTrue(GetMatchingSubscription(algo, cryptoHourly.Symbol, typeof(TradeBar)) != null);
             Assert.IsTrue(GetMatchingSubscription(algo, cryptoHourly.Symbol, typeof(QuoteBar)) != null);
+
+            Assert.IsTrue(ReferenceEquals(cryptoMinute, cryptoHourly));
+
+            var cryptoSubscriptions = algo.SubscriptionManager.SubscriptionDataConfigService
+                .GetSubscriptionDataConfigs(cryptoMinute.Symbol);
+            Assert.IsTrue(cryptoSubscriptions.SingleOrDefault(
+                s => s.TickType == TickType.Trade && s.Type == typeof(TradeBar) && s.Resolution == Resolution.Minute) != null);
+            Assert.IsTrue(cryptoSubscriptions.SingleOrDefault(
+                s => s.TickType == TickType.Quote && s.Type == typeof(QuoteBar) && s.Resolution == Resolution.Minute) != null);
+            Assert.IsTrue(cryptoSubscriptions.SingleOrDefault(
+                s => s.TickType == TickType.Trade && s.Type == typeof(TradeBar) && s.Resolution == Resolution.Hour) != null);
+            Assert.IsTrue(cryptoSubscriptions.SingleOrDefault(
+                s => s.TickType == TickType.Quote && s.Type == typeof(QuoteBar) && s.Resolution == Resolution.Hour) != null);
         }