Merge pull request #1159 from tgstation/1127-TestMergingRefactor [TGSDeploy]

Adds additional null checking around test merging
This commit is contained in:
Jordan Brown
2020-12-04 13:15:31 -05:00
committed by GitHub
2 changed files with 4 additions and 1 deletions
+1 -1
View File
@@ -3,7 +3,7 @@
<!-- Integration tests will ensure they match across the board -->
<Import Project="ControlPanelVersion.props" />
<PropertyGroup>
<TgsCoreVersion>4.6.0</TgsCoreVersion>
<TgsCoreVersion>4.6.1</TgsCoreVersion>
<TgsConfigVersion>2.1.1</TgsConfigVersion>
<TgsApiVersion>7.4.0</TgsApiVersion>
<TgsClientVersion>8.4.0</TgsClientVersion>
@@ -744,6 +744,9 @@ namespace Tgstation.Server.Host.Controllers
revInfoWereLookingFor = dbPull
.Where(testRevInfo =>
{
if (testRevInfo.PrimaryTestMerge == null)
return false;
var testMergeMatch = model.NewTestMerges.Any(testTestMerge =>
{
var numberMatch = testRevInfo.PrimaryTestMerge.Number == testTestMerge.Number;