Fix such that legacy document ids without document type maps to default bucket space.

1 files changed, 8 insertions, 0 deletions

diff --git a/storage/src/tests/storageserver/configurable_bucket_resolver_test.cpp b/storage/src/tests/storageserver/configurable_bucket_resolver_test.cpp
index 4d80192d9e9..c10db0a1acd 100644
--- a/storage/src/tests/storageserver/configurable_bucket_resolver_test.cpp
+++ b/storage/src/tests/storageserver/configurable_bucket_resolver_test.cpp
@@ -20,6 +20,7 @@ struct ConfigurableBucketResolverTest : CppUnit::TestFixture {
     CPPUNIT_TEST(known_bucket_space_is_resolved_from_document_id);
     CPPUNIT_TEST(unknown_bucket_space_in_id_throws_exception);
     CPPUNIT_TEST(can_create_resolver_from_bucket_space_config);
+    CPPUNIT_TEST(legacy_document_id_without_document_type_maps_to_default_space);
     CPPUNIT_TEST_SUITE_END();
 
     using BucketSpaceMapping = ConfigurableBucketResolver::BucketSpaceMapping;
@@ -47,6 +48,7 @@ struct ConfigurableBucketResolverTest : CppUnit::TestFixture {
     void known_bucket_space_is_resolved_from_document_id();
     void unknown_bucket_space_in_id_throws_exception();
     void can_create_resolver_from_bucket_space_config();
+    void legacy_document_id_without_document_type_maps_to_default_space();
 };
 
 CPPUNIT_TEST_SUITE_REGISTRATION(ConfigurableBucketResolverTest);
@@ -133,5 +135,11 @@ void ConfigurableBucketResolverTest::can_create_resolver_from_bucket_space_confi
                          resolver->bucketFromId(DocumentId("id::baz::xyz")).getBucketSpace());
 }
 
+void ConfigurableBucketResolverTest::legacy_document_id_without_document_type_maps_to_default_space() {
+    auto resolver = create_simple_resolver();
+    CPPUNIT_ASSERT_EQUAL(document::FixedBucketSpaces::default_space(),
+                         resolver.bucketFromId(DocumentId("userdoc:baz:1234:baz")).getBucketSpace());
+}
+
 }