diff --git a/java/com/google/domain/registry/model/registrar/Registrar.java b/java/com/google/domain/registry/model/registrar/Registrar.java index 206c186dd..445ec9b52 100644 --- a/java/com/google/domain/registry/model/registrar/Registrar.java +++ b/java/com/google/domain/registry/model/registrar/Registrar.java @@ -23,6 +23,7 @@ import static com.google.common.base.Predicates.notNull; import static com.google.common.base.Strings.emptyToNull; import static com.google.common.base.Strings.isNullOrEmpty; import static com.google.common.base.Strings.nullToEmpty; +import static com.google.common.collect.Sets.immutableEnumSet; import static com.google.common.io.BaseEncoding.base64; import static com.google.domain.registry.model.common.EntityGroupRoot.getCrossTldKey; import static com.google.domain.registry.model.ofy.ObjectifyService.ofy; @@ -179,13 +180,13 @@ public class Registrar extends ImmutableObject implements Buildable, Jsonifiable /** * The types for which a {@link Registrar} should be included in WHOIS and RDAP output. We exclude - * PDT and TEST. We considered excluding INTERNAL as well, but decided that troubleshooting would - * be easier with INTERNAL registrars visible. + * registrars of type TEST. We considered excluding INTERNAL as well, but decided that + * troubleshooting would be easier with INTERNAL registrars visible. */ //TODO(b/27274151): Expand documentation of this field. private static final ImmutableSet PUBLICLY_VISIBLE_TYPES = - Sets.immutableEnumSet( - Type.REAL, Type.OTE, Type.EXTERNAL_MONITORING, Type.MONITORING, Type.INTERNAL); + immutableEnumSet( + Type.REAL, Type.PDT, Type.OTE, Type.EXTERNAL_MONITORING, Type.MONITORING, Type.INTERNAL); @Parent Key parent = getCrossTldKey(); diff --git a/javatests/com/google/domain/registry/whois/WhoisServerTest.java b/javatests/com/google/domain/registry/whois/WhoisServerTest.java index d95257141..bd7030a6a 100644 --- a/javatests/com/google/domain/registry/whois/WhoisServerTest.java +++ b/javatests/com/google/domain/registry/whois/WhoisServerTest.java @@ -15,6 +15,8 @@ package com.google.domain.registry.whois; import static com.google.common.truth.Truth.assertThat; +import static com.google.domain.registry.model.registrar.Registrar.State.ACTIVE; +import static com.google.domain.registry.model.registrar.Registrar.Type.PDT; import static com.google.domain.registry.testing.DatastoreHelper.createTlds; import static com.google.domain.registry.testing.DatastoreHelper.persistResource; import static com.google.domain.registry.testing.DatastoreHelper.persistSimpleGlobalResources; @@ -85,7 +87,7 @@ public class WhoisServerTest { @Test public void testRun_domainQuery_works() throws Exception { Registrar registrar = persistResource(makeRegistrar( - "evilregistrar", "Yes Virginia