diff options
author | 2016-11-18 10:43:51 +0000 | |
---|---|---|
committer | 2016-11-18 10:55:05 +0000 | |
commit | 84a3ed95143a14d05d9459b555fd09ad645aa707 (patch) | |
tree | 2a23a049553da7fa3b76985170acba1126085d17 /src/test/java | |
parent | 82179a1c8b24538c00ced1fd649f6ed5b2978938 (diff) |
Add # support for LabelValidator as discussed in #2006
As suggested by @damienmg in #2006 send this out to run it with internal test.
Closes #2059 .
Progress towards #374.
--
Reviewed-on: https://github.com/bazelbuild/bazel/pull/2059
MOS_MIGRATED_REVID=139562084
Diffstat (limited to 'src/test/java')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/cmdline/LabelValidatorTest.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/cmdline/LabelValidatorTest.java b/src/test/java/com/google/devtools/build/lib/cmdline/LabelValidatorTest.java index feceda346d..c5df57171f 100644 --- a/src/test/java/com/google/devtools/build/lib/cmdline/LabelValidatorTest.java +++ b/src/test/java/com/google/devtools/build/lib/cmdline/LabelValidatorTest.java @@ -22,7 +22,6 @@ import static org.junit.Assert.assertNull; import static org.junit.Assert.fail; import com.google.devtools.build.lib.cmdline.LabelValidator.PackageAndTarget; - import org.junit.Test; import org.junit.runner.RunWith; import org.junit.runners.JUnit4; @@ -96,6 +95,7 @@ public class LabelValidatorTest { assertNull(LabelValidator.validateTargetName("foo.bar")); assertNull(LabelValidator.validateTargetName("foo@bar")); assertNull(LabelValidator.validateTargetName("foo~bar")); + assertNull(LabelValidator.validateTargetName("foo#bar")); assertEquals("target names may not end with '/'", LabelValidator.validateTargetName("foo/")); |