actions/runner
1
0
Fork 0
mirror of https://github.com/actions/runner.git synced 2025-12-10 20:36:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add filetable to testing file, remove ? since we know filetable should never be non null

Browse Source
This commit is contained in:
Ethan Chiu
2020-07-08 16:25:20 -04:00
parent 635baa5295
commit 68b05f63c9
2 changed files with 4 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/Runner.Worker/ActionManifestManager.cs
View File

@@ -75,7 +75,7 @@ namespace GitHub.Runner.Worker
// Add this file to the FileTable in executionContext if it hasn't been added already
// we use > since fileID is 1 indexed
if (fileId > executionContext.FileTable?.Count)
if (fileId > executionContext.FileTable.Count)
{
executionContext.FileTable.Add(fileRelativePath);
}
@@ -298,12 +298,9 @@ namespace GitHub.Runner.Worker
}
// Add the file table from the Execution Context
if (executionContext.FileTable?.Count > 0)
for (var i = 0; i < executionContext.FileTable.Count; i++)
{
for (var i = 0; i < executionContext.FileTable.Count; i++)
{
result.GetFileId(executionContext.FileTable[i]);
}
result.GetFileId(executionContext.FileTable[i]);
}
return result;

1
src/Test/L0/Worker/ActionRunnerL0.cs
View File

@@ -379,6 +379,7 @@ namespace GitHub.Runner.Common.Tests.Worker
_ec.Setup(x => x.ExpressionFunctions).Returns(new List<IFunctionInfo>());
_ec.Setup(x => x.IntraActionState).Returns(new Dictionary<string, string>());
_ec.Setup(x => x.EnvironmentVariables).Returns(new Dictionary<string, string>());
_ec.Setup(x => x.FileTable).Returns(new List<String>());
_ec.Setup(x => x.SetGitHubContext(It.IsAny<string>(), It.IsAny<string>()));
_ec.Setup(x => x.GetGitHubContext(It.IsAny<string>())).Returns("{\"foo\":\"bar\"}");
_ec.Setup(x => x.CancellationToken).Returns(_ecTokenSource.Token);