diff options
author | cparsons <cparsons@google.com> | 2018-07-30 12:44:17 -0700 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-07-30 12:46:29 -0700 |
commit | df8bf902e642ad8e8244e7fd58f1c36a6dfadb6c (patch) | |
tree | 8a7c8f6f4bebcc5c9e1d938efde8da37bd8ea908 /src/test/java/com/google | |
parent | ccb745699953365425c1931c05413d9911e051a8 (diff) |
Automated rollback of commit be88b8539b30d1c36c68b0f6acfe1e9ae010db3b.
*** Reason for rollback ***
Concerns were raised about this breaking change, so rolling back to be safe.
*** Original change description ***
Throw a rule error when a skylark rule implementation returns multiple providers of the same type.
RELNOTES: A rule error is now thrown if a Skylark rule implementation function returns multiple providers of the same type.
PiperOrigin-RevId: 206629186
Diffstat (limited to 'src/test/java/com/google')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/skylark/SkylarkIntegrationTest.java | 19 |
1 files changed, 0 insertions, 19 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/skylark/SkylarkIntegrationTest.java b/src/test/java/com/google/devtools/build/lib/skylark/SkylarkIntegrationTest.java index badce86c03..75d79294f5 100644 --- a/src/test/java/com/google/devtools/build/lib/skylark/SkylarkIntegrationTest.java +++ b/src/test/java/com/google/devtools/build/lib/skylark/SkylarkIntegrationTest.java @@ -1185,25 +1185,6 @@ public class SkylarkIntegrationTest extends BuildViewTestCase { } @Test - public void testConflictingProviderKeys() throws Exception { - scratch.file( - "test/extension.bzl", - "my_provider = provider()", - "other_provider = provider()", - "def _impl(ctx):", - " return struct(providers = [my_provider(x = 1), other_provider(), my_provider()])", - "my_rule = rule(_impl)" - ); - - checkError( - "test", - "r", - "Multiple conflicting returned providers with key my_provider", - "load(':extension.bzl', 'my_rule')", - "my_rule(name = 'r')"); - } - - @Test public void testRecursionDetection() throws Exception { reporter.removeHandler(failFastHandler); scratch.file( |