fix another test

Author: eric-maynard

quarkus/service/src/test/java/org/apache/polaris/service/quarkus/catalog/IcebergCatalogTest.java

@@ -1695,8 +1695,8 @@ public void testFileIOWrapper() {
 
     table.updateProperties().set("foo", "bar").commit();
     Assertions.assertThat(measured.getInputBytes())
-        .as("A table was read and written")
-        .isGreaterThan(0);
+        .as("A table was read and written, but no trip to storage was made")
+        .isEqualTo(0);
 
     Assertions.assertThat(catalog.dropTable(TABLE)).as("Table deletion should succeed").isTrue();
     TaskEntity taskEntity =
@@ -1860,7 +1860,7 @@ public void testTableOperationsDoesNotRefreshAfterCommit(boolean updateMetadataO
     IcebergCatalog.BasePolarisTableOperations ops = Mockito.spy(realOps);
 
     try (MockedStatic<TableMetadataParser> mocked =
-             Mockito.mockStatic(TableMetadataParser.class, Mockito.CALLS_REAL_METHODS)) {
+        Mockito.mockStatic(TableMetadataParser.class, Mockito.CALLS_REAL_METHODS)) {
       TableMetadata base1 = ops.current();
       mocked.verify(
           () -> TableMetadataParser.read(Mockito.any(), Mockito.anyString()), Mockito.times(1));

service/common/src/main/java/org/apache/polaris/service/catalog/iceberg/IcebergCatalog.java

@@ -1495,11 +1495,11 @@ public void doCommit(TableMetadata base, TableMetadata metadata) {
 
       // We diverge from `BaseMetastoreTableOperations` in the below code block
       if (updateMetadataOnCommit) {
-        currentMetadata = TableMetadata
-            .buildFrom(metadata)
-            .withMetadataLocation(newLocation)
-            .discardChanges()
-            .build();
+        currentMetadata =
+            TableMetadata.buildFrom(metadata)
+                .withMetadataLocation(newLocation)
+                .discardChanges()
+                .build();
         currentMetadataLocation = newLocation;
       }