Expose these members for testing

joegallo · joegallo · commit 3fda796f2ac4 · 2025-03-05T11:39:55.000-05:00
diff --git a/modules/ingest-common/src/main/java/org/elasticsearch/ingest/common/RegisteredDomainProcessor.java b/modules/ingest-common/src/main/java/org/elasticsearch/ingest/common/RegisteredDomainProcessor.java
@@ -83,7 +83,8 @@ public IngestDocument execute(IngestDocument document) throws Exception {
     }
 
     @Nullable
-    private static DomainInfo getRegisteredDomain(@Nullable String fqdn) {
+    // visible for testing
+    static DomainInfo getRegisteredDomain(@Nullable String fqdn) {
         if (fqdn == null) {
             return null;
         }
@@ -106,7 +107,8 @@ public String getType() {
         return TYPE;
     }
 
-    private record DomainInfo(
+    // visible for testing
+    record DomainInfo(
         String domain,
         String registeredDomain,
         String eTLD, // n.b. https://developer.mozilla.org/en-US/docs/Glossary/eTLD

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,8 @@ public IngestDocument execute(IngestDocument document) throws Exception {`
`83`	`83`	`}`
`84`	`84`
`85`	`85`	`@Nullable`
`86`		`- private static DomainInfo getRegisteredDomain(@Nullable String fqdn) {`
	`86`	`+ // visible for testing`
	`87`	`+ static DomainInfo getRegisteredDomain(@Nullable String fqdn) {`
`87`	`88`	`if (fqdn == null) {`
`88`	`89`	`return null;`
`89`	`90`	`}`
`@@ -106,7 +107,8 @@ public String getType() {`
`106`	`107`	`return TYPE;`
`107`	`108`	`}`
`108`	`109`
`109`		`- private record DomainInfo(`
	`110`	`+ // visible for testing`
	`111`	`+ record DomainInfo(`
`110`	`112`	`String domain,`
`111`	`113`	`String registeredDomain,`
`112`	`114`	`String eTLD, // n.b. https://developer.mozilla.org/en-US/docs/Glossary/eTLD`