commit 3ad5645a5148054089c131478ae0404861f79f5d
author Nathan Harmata <nharmata@google.com> Fri Jun 10 16:17:45 2016 +0000
committer Dmitry Lomov <dslomov@google.com> Fri Jun 10 17:16:00 2016 +0000
tree bd60620f2d0c918fd06e0de44230fb32d6b996e5
parent 4ec1e588ec7b375714fdc33b6c07be8b081f7372

Introduce TestConstants#PACKAGE_FACTORY_FACTORY_FOR_TESTING for the sake of making it easier to create PackageFactory instances in unit tests.

--
MOS_MIGRATED_REVID=124566862

src/test/java/com/google/devtools/build/lib/packages/EnvironmentGroupTest.java

diff --git a/src/test/java/com/google/devtools/build/lib/packages/EnvironmentGroupTest.java b/src/test/java/com/google/devtools/build/lib/packages/EnvironmentGroupTest.java
index 771322a..c8e0964 100644
--- a/src/test/java/com/google/devtools/build/lib/packages/EnvironmentGroupTest.java
+++ b/src/test/java/com/google/devtools/build/lib/packages/EnvironmentGroupTest.java
@@ -22,7 +22,6 @@
 import com.google.devtools.build.lib.cmdline.Label;
 import com.google.devtools.build.lib.cmdline.PackageIdentifier;
 import com.google.devtools.build.lib.packages.util.PackageLoadingTestCase;
-import com.google.devtools.build.lib.testutil.TestRuleClassProvider;
 import com.google.devtools.build.lib.vfs.Path;
 
 import org.junit.Before;
@@ -54,9 +53,8 @@
             "    environments = [':foo', ':bar', ':baz'],",
             "    defaults = [':foo'],",
             ")");
-    PackageFactory pkgFactory = new PackageFactory(TestRuleClassProvider.getRuleClassProvider());
     pkg =
-        pkgFactory.createPackageForTesting(
+        packageFactory.createPackageForTesting(
             PackageIdentifier.createInMainRepo("pkg"), buildfile, getPackageManager(), reporter);
 
     group = (EnvironmentGroup) pkg.getTarget("group");