Skip to content

Commit

Permalink
Fix bad runtime identifer source for CheckExecutePermission test
Browse files Browse the repository at this point in the history
Work done for #196
  • Loading branch information
atruskie committed Mar 27, 2020
1 parent 4c02f91 commit 6d12298
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion tests/Acoustics.Test/Shared/AppConfigHelperTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,7 @@ string Invoke()
}
}

[RuntimeIdentifierSpecificDataTestMethod(RidType.Pseudo)]
[RuntimeIdentifierSpecificDataTestMethod(RidType.Compiled)]
[DataRow(WinX64, false)]
[DataRow(OsxX64, null)]
[DataRow(LinuxX64, null)]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,9 @@ public override TestResult[] Execute(ITestMethod testMethod)
var actualRid = this.RuntimeIdentifierSource switch
{
RidType.Actual => actual,
RidType.Compiled => compiled,
RidType.Compiled => string.IsNullOrEmpty(compiled)
? throw new ArgumentException($"BuildMetadata.CompiledRuntimeIdentifer was null or empty. Cannot use as {nameof(this.RuntimeIdentifierSource)}.")
: compiled,
RidType.Pseudo => pseudo,
_ => throw new InvalidOperationException($"RidType {this.RuntimeIdentifierSource} is not supported"),
};
Expand Down

0 comments on commit 6d12298

Please sign in to comment.