Update PinotConfig tests to handle new broker-url and proxy validation constraint

Author: shreyas-dview

plugin/trino-pinot/src/test/java/io/trino/plugin/pinot/TestPinotConfig.java

@@ -68,7 +68,7 @@ public void testExplicitPropertyMappings()
                 .put("pinot.max-rows-for-broker-queries", "5000")
                 .put("pinot.aggregation-pushdown.enabled", "false")
                 .put("pinot.count-distinct-pushdown.enabled", "false")
-                .put("pinot.proxy.enabled", "true")
+                .put("pinot.proxy.enabled", "false")
                 .put("pinot.target-segment-page-size", "2MB")
                 .buildOrThrow();
 
@@ -85,7 +85,7 @@ public void testExplicitPropertyMappings()
                 .setMaxRowsForBrokerQueries(5000)
                 .setAggregationPushdownEnabled(false)
                 .setCountDistinctPushdownEnabled(false)
-                .setProxyEnabled(true)
+                .setProxyEnabled(false)
                 .setTargetSegmentPageSize(DataSize.of(2, MEGABYTE));
 
         ConfigAssertions.assertFullMapping(properties, expected);
@@ -103,6 +103,18 @@ public void testInvalidCountDistinctPushdown()
                 AssertTrue.class);
     }
 
+    @Test
+    public void testInvalidBrokerProxyConfiguration()
+    {
+        assertFailsValidation(
+            new PinotConfig()
+                    .setBrokerUrl(HostAndPort.fromString("host1:1111"))
+                    .setProxyEnabled(true),
+            "validBrokerProxyConfiguration",
+            "Invalid configuration: pinot.broker-url and pinot.proxy.enabled cannot both be set.",
+            AssertTrue.class);
+    }
+
     @Test
     public void testControllerUrls()
     {