diff options
author | 2016-08-26 08:16:25 +0000 | |
---|---|---|
committer | 2016-08-26 18:40:43 +0000 | |
commit | d2fcd9d05823563f1d77eebe19e1caa334724740 (patch) | |
tree | 172eef02e880653d6c8ca3fa34387552d46e2b81 /src/test/java/com/google/devtools/build/lib/analysis/actions | |
parent | 112e02ac2396d133b669ff6f58a37a0cc3c84a37 (diff) |
Add client environment to ActionExecutionContext
As the execution of an action now also depends on the client environment,
make the latter part of the ActionExecutionContext, so that enough context
is provided to actually execute an action.
--
Change-Id: Ida7bf407ef0c0375728faba92494bfd47dcbaeb8
Reviewed-on: https://bazel-review.googlesource.com/#/c/5391
MOS_MIGRATED_REVID=131377490
Diffstat (limited to 'src/test/java/com/google/devtools/build/lib/analysis/actions')
6 files changed, 30 insertions, 23 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/analysis/actions/FileWriteActionTestCase.java b/src/test/java/com/google/devtools/build/lib/analysis/actions/FileWriteActionTestCase.java index b2ccbc4399..c2563f3920 100644 --- a/src/test/java/com/google/devtools/build/lib/analysis/actions/FileWriteActionTestCase.java +++ b/src/test/java/com/google/devtools/build/lib/analysis/actions/FileWriteActionTestCase.java @@ -20,6 +20,7 @@ import static org.junit.Assert.assertFalse; import static org.junit.Assert.assertNull; import static org.junit.Assert.assertTrue; +import com.google.common.collect.ImmutableMap; import com.google.common.collect.Sets; import com.google.devtools.build.lib.actions.Action; import com.google.devtools.build.lib.actions.ActionExecutionContext; @@ -32,10 +33,8 @@ import com.google.devtools.build.lib.exec.util.TestExecutorBuilder; import com.google.devtools.build.lib.util.io.FileOutErr; import com.google.devtools.build.lib.vfs.FileSystemUtils; import com.google.devtools.build.lib.vfs.Path; - -import org.junit.Before; - import java.util.Collection; +import org.junit.Before; public abstract class FileWriteActionTestCase extends BuildViewTestCase { @@ -56,7 +55,8 @@ public abstract class FileWriteActionTestCase extends BuildViewTestCase { @Before public final void createExecutorAndContext() throws Exception { executor = new TestExecutorBuilder(directories, binTools).build(); - context = new ActionExecutionContext(executor, null, null, new FileOutErr(), null); + context = new ActionExecutionContext(executor, null, null, new FileOutErr(), + ImmutableMap.<String, String>of(), null); } protected abstract Action createAction( diff --git a/src/test/java/com/google/devtools/build/lib/analysis/actions/LTOBackendActionTest.java b/src/test/java/com/google/devtools/build/lib/analysis/actions/LTOBackendActionTest.java index f2349e5511..2dd014bbcb 100644 --- a/src/test/java/com/google/devtools/build/lib/analysis/actions/LTOBackendActionTest.java +++ b/src/test/java/com/google/devtools/build/lib/analysis/actions/LTOBackendActionTest.java @@ -16,6 +16,7 @@ package com.google.devtools.build.lib.analysis.actions; import static com.google.common.truth.Truth.assertThat; import static org.junit.Assert.assertEquals; +import com.google.common.collect.ImmutableMap; import com.google.devtools.build.lib.actions.AbstractAction; import com.google.devtools.build.lib.actions.Action; import com.google.devtools.build.lib.actions.ActionExecutionContext; @@ -29,15 +30,13 @@ import com.google.devtools.build.lib.analysis.util.BuildViewTestCase; import com.google.devtools.build.lib.exec.util.TestExecutorBuilder; import com.google.devtools.build.lib.util.io.FileOutErr; import com.google.devtools.build.lib.vfs.PathFragment; - +import java.util.HashMap; +import java.util.Map; import org.junit.Before; import org.junit.Test; import org.junit.runner.RunWith; import org.junit.runners.JUnit4; -import java.util.HashMap; -import java.util.Map; - /** Tests {@link LTOBackendAction}. */ @RunWith(JUnit4.class) public class LTOBackendActionTest extends BuildViewTestCase { @@ -78,7 +77,8 @@ public class LTOBackendActionTest extends BuildViewTestCase { @Before public final void createExecutorAndContext() throws Exception { executor = new TestExecutorBuilder(directories, binTools).build(); - context = new ActionExecutionContext(executor, null, null, new FileOutErr(), null); + context = new ActionExecutionContext(executor, null, null, new FileOutErr(), + ImmutableMap.<String, String>of(), null); } @Test diff --git a/src/test/java/com/google/devtools/build/lib/analysis/actions/ParamFileWriteActionTest.java b/src/test/java/com/google/devtools/build/lib/analysis/actions/ParamFileWriteActionTest.java index 21b79759fd..0f6b3940df 100644 --- a/src/test/java/com/google/devtools/build/lib/analysis/actions/ParamFileWriteActionTest.java +++ b/src/test/java/com/google/devtools/build/lib/analysis/actions/ParamFileWriteActionTest.java @@ -17,6 +17,7 @@ import static com.google.common.truth.Truth.assertThat; import static org.junit.Assert.assertEquals; import com.google.common.collect.ImmutableList; +import com.google.common.collect.ImmutableMap; import com.google.devtools.build.lib.actions.Action; import com.google.devtools.build.lib.actions.ActionExecutionContext; import com.google.devtools.build.lib.actions.ActionInputHelper; @@ -35,15 +36,13 @@ import com.google.devtools.build.lib.exec.util.TestExecutorBuilder; import com.google.devtools.build.lib.util.io.FileOutErr; import com.google.devtools.build.lib.vfs.FileSystemUtils; import com.google.devtools.build.lib.vfs.PathFragment; - +import java.nio.charset.StandardCharsets; +import java.util.Collection; import org.junit.Before; import org.junit.Test; import org.junit.runner.RunWith; import org.junit.runners.JUnit4; -import java.nio.charset.StandardCharsets; -import java.util.Collection; - /** Tests for ParamFileWriteAction. */ @RunWith(JUnit4.class) public class ParamFileWriteActionTest extends BuildViewTestCase { @@ -163,6 +162,6 @@ public class ParamFileWriteActionTest extends BuildViewTestCase { Executor executor = new TestExecutorBuilder(directories, binTools).build(); return new ActionExecutionContext( - executor, null, null, new FileOutErr(), artifactExpander); + executor, null, null, new FileOutErr(), ImmutableMap.of(), artifactExpander); } } diff --git a/src/test/java/com/google/devtools/build/lib/analysis/actions/PopulateTreeArtifactActionTest.java b/src/test/java/com/google/devtools/build/lib/analysis/actions/PopulateTreeArtifactActionTest.java index d76e674e74..2d6c0fcc4f 100644 --- a/src/test/java/com/google/devtools/build/lib/analysis/actions/PopulateTreeArtifactActionTest.java +++ b/src/test/java/com/google/devtools/build/lib/analysis/actions/PopulateTreeArtifactActionTest.java @@ -18,6 +18,7 @@ import static org.junit.Assert.fail; import static org.mockito.Mockito.mock; import com.google.common.collect.ImmutableList; +import com.google.common.collect.ImmutableMap; import com.google.devtools.build.lib.actions.Action; import com.google.devtools.build.lib.actions.ActionExecutionContext; import com.google.devtools.build.lib.actions.ActionExecutionException; @@ -315,7 +316,12 @@ public class PopulateTreeArtifactActionTest extends BuildViewTestCase { .build(); return new ActionExecutionContext( - executor, null, new TestMetadataHandler(storingExpandedTreeFileArtifacts), null, null); + executor, + null, + new TestMetadataHandler(storingExpandedTreeFileArtifacts), + null, + ImmutableMap.of(), + null); } private Artifact createTreeArtifact(String rootRelativePath) { diff --git a/src/test/java/com/google/devtools/build/lib/analysis/actions/SymlinkActionTest.java b/src/test/java/com/google/devtools/build/lib/analysis/actions/SymlinkActionTest.java index 991d8b64f2..0bdf79f606 100644 --- a/src/test/java/com/google/devtools/build/lib/analysis/actions/SymlinkActionTest.java +++ b/src/test/java/com/google/devtools/build/lib/analysis/actions/SymlinkActionTest.java @@ -20,6 +20,7 @@ import static org.junit.Assert.assertFalse; import static org.junit.Assert.assertTrue; import static org.junit.Assert.fail; +import com.google.common.collect.ImmutableMap; import com.google.common.collect.Sets; import com.google.devtools.build.lib.actions.ActionExecutionContext; import com.google.devtools.build.lib.actions.ActionExecutionException; @@ -29,7 +30,6 @@ import com.google.devtools.build.lib.analysis.util.BuildViewTestCase; import com.google.devtools.build.lib.exec.util.TestExecutorBuilder; import com.google.devtools.build.lib.vfs.FileSystemUtils; import com.google.devtools.build.lib.vfs.Path; - import org.junit.Before; import org.junit.Test; import org.junit.runner.RunWith; @@ -76,7 +76,8 @@ public class SymlinkActionTest extends BuildViewTestCase { @Test public void testSymlink() throws Exception { Executor executor = new TestExecutorBuilder(directories, null).build(); - action.execute(new ActionExecutionContext(executor, null, null, null, null)); + action.execute(new ActionExecutionContext(executor, null, null, null, + ImmutableMap.<String, String>of(), null)); assertTrue(output.isSymbolicLink()); assertEquals(input, output.resolveSymbolicLinks()); assertEquals(inputArtifact, action.getPrimaryInput()); @@ -91,7 +92,8 @@ public class SymlinkActionTest extends BuildViewTestCase { action = new ExecutableSymlinkAction(NULL_ACTION_OWNER, inputArtifact, outputArtifact); assertFalse(input.isExecutable()); ActionExecutionContext actionExecutionContext = - new ActionExecutionContext(executor, null, null, null, null); + new ActionExecutionContext(executor, null, null, null, + ImmutableMap.<String, String>of(), null); try { action.execute(actionExecutionContext); fail("Expected ActionExecutionException"); diff --git a/src/test/java/com/google/devtools/build/lib/analysis/actions/TemplateExpansionActionTest.java b/src/test/java/com/google/devtools/build/lib/analysis/actions/TemplateExpansionActionTest.java index 3f423ebe29..6a174a851e 100644 --- a/src/test/java/com/google/devtools/build/lib/analysis/actions/TemplateExpansionActionTest.java +++ b/src/test/java/com/google/devtools/build/lib/analysis/actions/TemplateExpansionActionTest.java @@ -20,6 +20,7 @@ import static org.junit.Assert.assertFalse; import com.google.common.base.Joiner; import com.google.common.collect.ImmutableList; +import com.google.common.collect.ImmutableMap; import com.google.common.collect.ImmutableSet; import com.google.common.collect.Lists; import com.google.devtools.build.lib.actions.ActionExecutionContext; @@ -35,15 +36,13 @@ import com.google.devtools.build.lib.testutil.FoundationTestCase; import com.google.devtools.build.lib.util.io.FileOutErr; import com.google.devtools.build.lib.vfs.FileSystemUtils; import com.google.devtools.build.lib.vfs.Path; - +import java.nio.charset.StandardCharsets; +import java.util.List; import org.junit.Before; import org.junit.Test; import org.junit.runner.RunWith; import org.junit.runners.JUnit4; -import java.nio.charset.StandardCharsets; -import java.util.List; - /** * Tests {@link TemplateExpansionAction}. */ @@ -174,7 +173,8 @@ public class TemplateExpansionActionTest extends FoundationTestCase { } private ActionExecutionContext createContext(Executor executor) { - return new ActionExecutionContext(executor, null, null, new FileOutErr(), null); + return new ActionExecutionContext(executor, null, null, new FileOutErr(), + ImmutableMap.<String, String>of(), null); } private void executeTemplateExpansion(String expected) throws Exception { |