Core: Fix BaseMetastoreCatalog#registerTable to return full table name (

apache#6589)

* Core: Fix BaseMetastoreCatalog#registerTable to return full table name

registerTable should return full table name, just like how loadTable and
create API returns

* Empty

core/src/main/java/org/apache/iceberg/BaseMetastoreCatalog.java

@@ -83,7 +83,7 @@ public Table registerTable(TableIdentifier identifier, String metadataFileLocati
     TableMetadata metadata = TableMetadataParser.read(ops.io(), metadataFile);
     ops.commit(null, metadata);
 
-    return new BaseTable(ops, identifier.toString());
+    return new BaseTable(ops, fullTableName(name(), identifier));
   }
 
   @Override

core/src/test/java/org/apache/iceberg/jdbc/TestJdbcCatalog.java

@@ -47,6 +47,7 @@
 import org.apache.iceberg.SortOrder;
 import org.apache.iceberg.Table;
 import org.apache.iceberg.TableOperations;
+import org.apache.iceberg.TestHelpers;
 import org.apache.iceberg.Transaction;
 import org.apache.iceberg.catalog.CatalogTests;
 import org.apache.iceberg.catalog.Namespace;
@@ -778,7 +779,9 @@ public void testRegisterTable() {
     catalog.dropTable(identifier, false);
     TableOperations ops = ((HasTableOperations) registeringTable).operations();
     String metadataLocation = ((JdbcTableOperations) ops).currentMetadataLocation();
-    Assertions.assertThat(catalog.registerTable(identifier, metadataLocation)).isNotNull();
+    Table registeredTable = catalog.registerTable(identifier, metadataLocation);
+    Assertions.assertThat(registeredTable).isNotNull();
+    TestHelpers.assertSerializedAndLoadedMetadata(registeringTable, registeredTable);
     Assertions.assertThat(catalog.loadTable(identifier)).isNotNull();
     Assertions.assertThat(catalog.dropTable(identifier)).isTrue();
   }