aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/test/java/com/google/devtools/build/lib/cmdline/LabelValidatorTest.java
diff options
context:
space:
mode:
authorGravatar Meng Zhang <zhangmeng@google.com>2016-11-18 10:43:51 +0000
committerGravatar Yun Peng <pcloudy@google.com>2016-11-18 10:55:05 +0000
commit84a3ed95143a14d05d9459b555fd09ad645aa707 (patch)
tree2a23a049553da7fa3b76985170acba1126085d17 /src/test/java/com/google/devtools/build/lib/cmdline/LabelValidatorTest.java
parent82179a1c8b24538c00ced1fd649f6ed5b2978938 (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/com/google/devtools/build/lib/cmdline/LabelValidatorTest.java')
-rw-r--r--src/test/java/com/google/devtools/build/lib/cmdline/LabelValidatorTest.java2
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/"));