diff options
| author |  janakr <janakr@google.com> | 2017-07-19 22:40:20 +0200 |
|---|---|---|
| committer |  Klaus Aehlig <aehlig@google.com> | 2017-07-20 10:30:02 +0200 |
| commit | 5e606e6113902a5ddfe4f5e54fb72a0fd2f4c3e3 (patch) | |
| tree | 5e6922a118ecba9dbdab3577d3fe099f7ee0e09a /src/test/java/com/google/devtools/build/lib/pkgcache/BuildFileModificationTest.java | |
| parent | 2cb56d53efb3964f1bd3ab3cb19f43ae7a2fdce0 (diff) | |
Reduce to one SequencedSkyframeExecutor#create method. The others were minor convenience methods that didn't (IMO) carry their weight.
PiperOrigin-RevId: 162528716
Diffstat (limited to 'src/test/java/com/google/devtools/build/lib/pkgcache/BuildFileModificationTest.java')
| -rw-r--r-- | src/test/java/com/google/devtools/build/lib/pkgcache/BuildFileModificationTest.java | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/pkgcache/BuildFileModificationTest.java b/src/test/java/com/google/devtools/build/lib/pkgcache/BuildFileModificationTest.java index 94aca28b1a..67dd5795ac 100644 --- a/src/test/java/com/google/devtools/build/lib/pkgcache/BuildFileModificationTest.java +++ b/src/test/java/com/google/devtools/build/lib/pkgcache/BuildFileModificationTest.java @@ -25,6 +25,7 @@ import com.google.devtools.build.lib.analysis.util.AnalysisMock; import com.google.devtools.build.lib.cmdline.PackageIdentifier; import com.google.devtools.build.lib.packages.NoSuchPackageException; import com.google.devtools.build.lib.packages.Package; +import com.google.devtools.build.lib.skyframe.BazelSkyframeExecutorConstants; import com.google.devtools.build.lib.skyframe.DiffAwareness; import com.google.devtools.build.lib.skyframe.PrecomputedValue; import com.google.devtools.build.lib.skyframe.SequencedSkyframeExecutor; @@ -73,7 +74,7 @@ public class BuildFileModificationTest extends FoundationTestCase { BlazeDirectories directories = new BlazeDirectories(outputBase, outputBase, rootDirectory, analysisMock.getProductName()); skyframeExecutor = - SequencedSkyframeExecutor.createForTesting( + SequencedSkyframeExecutor.create( analysisMock .getPackageFactoryBuilderForTesting() .build(ruleClassProvider, scratch.getFileSystem()), @@ -86,7 +87,11 @@ public class BuildFileModificationTest extends FoundationTestCase { AnalysisMock.get().getSkyFunctions(), ImmutableList.<PrecomputedValue.Injected>of(), ImmutableList.<SkyValueDirtinessChecker>of(), - analysisMock.getProductName()); + PathFragment.EMPTY_FRAGMENT, + analysisMock.getProductName(), + BazelSkyframeExecutorConstants.CROSS_REPOSITORY_LABEL_VIOLATION_STRATEGY, + BazelSkyframeExecutorConstants.BUILD_FILES_BY_PRIORITY, + BazelSkyframeExecutorConstants.ACTION_ON_IO_EXCEPTION_READING_BUILD_FILE); OptionsParser parser = OptionsParser.newOptionsParser( PackageCacheOptions.class, SkylarkSemanticsOptions.class); analysisMock.getInvocationPolicyEnforcer().enforce(parser); |