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

Author: shreyas-dev

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

@@ -68,7 +68,6 @@ 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.target-segment-page-size", "2MB")
                 .buildOrThrow();
 
@@ -85,7 +84,6 @@ public void testExplicitPropertyMappings()
                 .setMaxRowsForBrokerQueries(5000)
                 .setAggregationPushdownEnabled(false)
                 .setCountDistinctPushdownEnabled(false)
-                .setProxyEnabled(true)
                 .setTargetSegmentPageSize(DataSize.of(2, MEGABYTE));
 
         ConfigAssertions.assertFullMapping(properties, expected);
@@ -103,6 +101,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()
     {