Skip zero price and target quantity case

Author: JosueNina

Common/Algorithm/Framework/Portfolio/SignalExports/Collective2SignalExport.cs

@@ -209,6 +209,11 @@ private bool ConvertTypeOfSymbol(Symbol targetSymbol, out string typeOfSymbol)
         /// <returns>Number of shares hold of the given position</returns>
         protected int ConvertPercentageToQuantity(IAlgorithm algorithm, PortfolioTarget target)
         {
+            if (algorithm.Securities[target.Symbol].Price == 0 && target.Quantity == 0)
+            {
+                algorithm.Debug($"Warning: The price for {target.Symbol} is 0, and the target quantity is 0. Returning 0 as the calculated quantity.");
+                return 0;
+            }
             var numberShares = PortfolioTarget.Percent(algorithm, target.Symbol, target.Quantity);
             if (numberShares == null)
             {

Tests/Algorithm/Framework/Portfolio/SignalExportTargetTests.cs

@@ -279,6 +279,25 @@ public void SignalExportManagerGetsCorrectPortfolioTargetArray(SecurityType secu
             Assert.AreEqual(quantity, targetQuantity, 1);
         }
 
+        [Test]
+        public void SignalExportManagerDoesNotThrowOnZeroPrice()
+        {
+            var algorithm = new AlgorithmStub(true);
+            algorithm.SetDateTime(new DateTime(2024, 02, 16, 11, 53, 30));
+
+            var security = algorithm.AddSecurity(Symbols.SPY);
+            // Set the market price to 0 to simulate the edge case being tested
+            security.SetMarketPrice(new Tick { Value = 0 });
+
+            using var manager = new Collective2SignalExportHandler("", 0);
+            // Ensure ConvertPercentageToQuantity does not throw when price is 0
+            Assert.DoesNotThrow(() =>
+            {
+                var result = manager.ConvertPercentageToQuantity(algorithm, new PortfolioTarget(Symbols.SPY, 0));
+                Assert.AreEqual(0, result);
+            });
+        }
+
         [Test]
         public void SignalExportManagerHandlesIndexOptions()
         {