diff options
| author |  John Cater <jcater@google.com> | 2017-11-21 07:23:28 -0800 |
|---|---|---|
| committer |  Copybara-Service <copybara-piper@google.com> | 2017-11-21 07:24:56 -0800 |
| commit | 2d5035685a6180e6abfa7e93d7ee4b7ec0632510 (patch) | |
| tree | 8c77eed55891b24c8577f691af9c25f41e2fd32b /src/test/java/com/google/devtools/build/lib/analysis/platform | |
| parent | 48c9393a4485bcc8388d27658c6c9452d53abcfb (diff) | |
Change Platform.remoteExecutionProperties to be a String, not a dict.
This is needed for the ability to synthesize the correct remote
execution protos from a platform rule.
Part of #4128.
Change-Id: I7fa8acf45642a4df4e2beb1ba9c57c2536670486
PiperOrigin-RevId: 176504885
Diffstat (limited to 'src/test/java/com/google/devtools/build/lib/analysis/platform')
| -rw-r--r-- | src/test/java/com/google/devtools/build/lib/analysis/platform/PlatformInfoTest.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/analysis/platform/PlatformInfoTest.java b/src/test/java/com/google/devtools/build/lib/analysis/platform/PlatformInfoTest.java index e8dd2fd1bc..fa0add7958 100644 --- a/src/test/java/com/google/devtools/build/lib/analysis/platform/PlatformInfoTest.java +++ b/src/test/java/com/google/devtools/build/lib/analysis/platform/PlatformInfoTest.java @@ -100,7 +100,7 @@ public class PlatformInfoTest extends BuildViewTestCase { .setLabel(makeLabel("//platform/plat1")) .addConstraint(value1) .addConstraint(value2) - .addRemoteExecutionProperty("key", "val") // execution properties are ignored. + .setRemoteExecutionProperties("key=val") // execution properties are ignored. .build()) .addEqualityGroup( // Different label. @@ -161,7 +161,7 @@ public class PlatformInfoTest extends BuildViewTestCase { ConstraintValueInfo constraintValue = ConstraintValueInfo.create(constraintSetting, makeLabel("//constraint:foo")); assertThat(provider.constraints()).containsExactly(constraintValue); - assertThat(provider.remoteExecutionProperties()).isEmpty(); + assertThat(provider.remoteExecutionProperties()).isNull(); } @Test |