Merge a64b909 into aa80c88

samsmithnz · May 20, 2021 · c1c0419 · c1c0419
2 parents aa80c88 + a64b909
commit c1c0419
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/.github/workflows/dotnet.yml b/.github/workflows/dotnet.yml
@@ -63,5 +63,4 @@ jobs:
         run: |
           .\.sonar\scanner\dotnet-sonarscanner begin /k:"samsmithnz_BattleSimulator" /o:"samsmithnz-github" /d:sonar.login="${{ secrets.SONAR_TOKEN }}" /d:sonar.host.url="https://sonarcloud.io"
           dotnet build src/Battle.Logic/Battle.Logic.csproj
-          dotnet test src/Battle.Tests/Battle.Tests.csproj --configuration Debug --logger trx /p:CollectCoverage=true /p:CoverletOutput=TestResults/ /p:CoverletOutputFormat=lcov  /p:CoverletOutputFormat=lcov --settings:./src/Battle.Tests/CodeCoverage.runsettings
           .\.sonar\scanner\dotnet-sonarscanner end /d:sonar.login="${{ secrets.SONAR_TOKEN }}"
diff --git a/src/Battle.Logic/CharacterCover/Cover.cs b/src/Battle.Logic/CharacterCover/Cover.cs
@@ -193,7 +193,7 @@ public static CoverState CalculateCover(Vector3 defenderPosition, int width, int
         private static List<Vector3> FindAdjacentCover(Vector3 currentLocation, int width, int height, string[,] validTiles)
         {
             List<Vector3> result = new();
-            if (currentLocation.X > width - 1 | currentLocation.Z > height - 1)
+            if (currentLocation.X > width - 1 || currentLocation.Z > height - 1)
             {
                 throw new Exception("The character is off the map");
             }