Update catch2 tags assigned to object store tests (#6788)

* More object store test tag updates
* Fix malformed tag and cleaned up other tags
* missed a [baas] tag
* Removed a few missed [local] tags

test/object-store/audit.cpp

@@ -282,7 +282,7 @@ struct TestClock {
 
 } // namespace
 
-TEST_CASE("audit object serialization") {
+TEST_CASE("audit object serialization", "[sync][pbs][audit]") {
     TestSyncManager test_session;
     SyncTestFile config(test_session.app(), "parent");
     config.cache = false;
@@ -1079,7 +1079,7 @@ TEST_CASE("audit object serialization") {
     }
 }
 
-TEST_CASE("audit management") {
+TEST_CASE("audit management", "[sync][pbs][audit]") {
     TestClock clock;
 
     TestSyncManager test_session;
@@ -1496,7 +1496,7 @@ TEST_CASE("audit management") {
 #endif
 }
 
-TEST_CASE("audit realm sharding") {
+TEST_CASE("audit realm sharding", "[sync][pbs][audit]") {
     // Don't start the server immediately so that we're forced to accumulate
     // a lot of local unuploaded data.
     TestSyncManager test_session{{}, {.start_immediately = false}};
@@ -1667,7 +1667,7 @@ static void generate_event(std::shared_ptr<Realm> realm, int call = 0)
     audit->end_scope(scope, assert_no_error);
 }
 
-TEST_CASE("audit integration tests") {
+TEST_CASE("audit integration tests", "[sync][pbs][audit][baas]") {
     // None of these tests need a deterministic clock, but the server rounding
     // timestamps to milliseconds can result in events not having monotonically
     // increasing timestamps with an actual clock.

test/object-store/backup.cpp

@@ -45,7 +45,7 @@
 
 using namespace realm;
 
-TEST_CASE("Automated backup") {
+TEST_CASE("Automated backup", "[backup]") {
     TestFile config;
     std::string copy_from_file_name = test_util::get_test_resource_path() + "test_backup-olden-and-golden.realm";
     config.path = test_util::get_test_path_prefix() + "test_backup.realm";

test/object-store/benchmarks/client_reset.cpp

@@ -160,7 +160,7 @@ struct BenchmarkLocalClientReset : public reset_utils::TestClientReset {
     ClientResyncMode m_mode;
 };
 
-TEST_CASE("client reset", "[benchmark]") {
+TEST_CASE("client reset", "[sync][pbs][benchmark][client reset]") {
     const std::string valid_pk_name = "_id";
     const std::string partition_value = "partition_foo";
     Property partition_prop = {"realm_id", PropertyType::String | PropertyType::Nullable};

test/object-store/benchmarks/object.cpp

@@ -73,7 +73,7 @@ struct TestContext : CppContext {
     }
 };
 
-TEST_CASE("Benchmark index change calculations", "[benchmark]") {
+TEST_CASE("Benchmark index change calculations", "[benchmark][index]") {
     _impl::CollectionChangeBuilder c;
 
     auto all_modified = [](ObjKey) {
@@ -147,7 +147,7 @@ TEST_CASE("Benchmark index change calculations", "[benchmark]") {
     }
 }
 
-TEST_CASE("Benchmark object", "[benchmark]") {
+TEST_CASE("Benchmark object", "[benchmark][object]") {
     using namespace std::string_literals;
     using AnyVec = std::vector<std::any>;
     using AnyDict = std::map<std::string, std::any>;
@@ -774,7 +774,7 @@ TEST_CASE("Benchmark object", "[benchmark]") {
     }
 }
 
-TEST_CASE("Benchmark object notification delivery", "[benchmark]") {
+TEST_CASE("Benchmark object notification delivery", "[benchmark][notifications]") {
     _impl::RealmCoordinator::assert_no_open_realms();
 
     InMemoryTestFile config;

test/object-store/benchmarks/results.cpp

@@ -33,7 +33,7 @@
 
 using namespace realm;
 
-TEST_CASE("Benchmark results", "[benchmark]") {
+TEST_CASE("Benchmark results", "[benchmark][results]") {
     InMemoryTestFile config;
     config.schema = Schema{
         {"object",
@@ -159,7 +159,7 @@ TEST_CASE("Benchmark results", "[benchmark]") {
     }
 }
 
-TEST_CASE("Benchmark results notifier", "[benchmark]") {
+TEST_CASE("Benchmark results notifier", "[benchmark][results]") {
     InMemoryTestFile config;
 
     SECTION("100 strongly connected tables") {
@@ -335,7 +335,7 @@ TEST_CASE("Benchmark results notifier", "[benchmark]") {
     }
 }
 
-TEST_CASE("aggregates") {
+TEST_CASE("aggregates", "[benchmark][aggregate]") {
     InMemoryTestFile config;
     config.schema = Schema{
         {"object",
@@ -420,7 +420,7 @@ TEST_CASE("aggregates") {
     };
 }
 
-TEST_CASE("Benchmark sectioned results", "[benchmark]") {
+TEST_CASE("Benchmark sectioned results", "[benchmark][results]") {
     InMemoryTestFile config;
     config.automatic_change_notifications = false;
     config.schema = Schema{{"object", {{"value", PropertyType::Int}}}};

test/object-store/c_api/c_api.cpp

@@ -5099,7 +5099,7 @@ static void sync_error_handler(void* p, realm_sync_session_t*, const realm_sync_
     userdata_p->error_catagory = error.error_code.category_name;
 }
 
-TEST_CASE("C API - async_open", "[c_api][sync]") {
+TEST_CASE("C API - async_open", "[sync][pbs][c_api]") {
     TestSyncManager init_sync_manager;
     SyncTestFile test_config(init_sync_manager.app(), "default");
     test_config.cache = false;
@@ -5193,7 +5193,7 @@ struct BCTOState {
 };
 
 
-TEST_CASE("C API - binding callback thread observer", "[c_api][sync]") {
+TEST_CASE("C API - binding callback thread observer", "[sync][c_api]") {
     auto bcto_user_data = BCTOState();
 
     auto bcto_free_userdata = [](realm_userdata_t userdata) {
@@ -5293,7 +5293,7 @@ std::atomic<std::size_t> error_handler_counter{0};
 std::atomic<std::size_t> before_client_reset_counter{0};
 std::atomic<std::size_t> after_client_reset_counter{0};
 
-TEST_CASE("C API - client reset", "[c_api][client-reset]") {
+TEST_CASE("C API - client reset", "[sync][pbs][c_api][client reset][baas]") {
     reset_utils::Partition partition{"realm_id", random_string(20)};
     Property partition_prop = {partition.property_name, PropertyType::String | PropertyType::Nullable};
     Schema schema{
@@ -5475,7 +5475,7 @@ static void realm_app_user2(void* p, realm_user_t* user, const realm_app_error_t
     }
 }
 
-TEST_CASE("C API app: link_user integration w/c_api transport", "[c_api][sync][app]") {
+TEST_CASE("C API app: link_user integration w/c_api transport", "[sync][app][c_api][baas]") {
     struct TestTransportUserData {
         TestTransportUserData()
             : logger(std::make_unique<util::StderrLogger>(realm::util::Logger::Level::TEST_LOGGING_LEVEL))
@@ -5702,7 +5702,7 @@ TEST_CASE("C API app: link_user integration w/c_api transport", "[c_api][sync][a
     realm_release(http_transport);
 }
 
-TEST_CASE("app: flx-sync compensating writes C API support", "[c_api][flx][sync]") {
+TEST_CASE("app: flx-sync compensating writes C API support", "[sync][flx][c_api][baas]") {
     using namespace realm::app;
     FLXSyncTestHarness harness("c_api_comp_writes");
     create_user_and_log_in(harness.app());
@@ -5783,7 +5783,7 @@ TEST_CASE("app: flx-sync compensating writes C API support", "[c_api][flx][sync]
     REQUIRE_THAT(errors[1].reason, Catch::Matchers::ContainsSubstring("object is outside of the current query view"));
 }
 
-TEST_CASE("app: flx-sync basic tests", "[c_api][flx][sync]") {
+TEST_CASE("app: flx-sync basic tests", "[sync][flx][c_api][baas]") {
     using namespace realm::app;
 
     auto make_schema = [] {
@@ -6156,7 +6156,7 @@ TEST_CASE("app: flx-sync basic tests", "[c_api][flx][sync]") {
     });
 }
 
-TEST_CASE("C API app: websocket provider", "[c_api][sync][app]") {
+TEST_CASE("C API app: websocket provider", "[sync][app][c_api][baas]") {
     using namespace realm::app;
     using namespace realm::sync;
     using namespace realm::sync::websocket;

test/object-store/collection_change_indices.cpp

@@ -26,7 +26,7 @@
 
 using namespace realm;
 
-TEST_CASE("collection_change: insert()") {
+TEST_CASE("collection_change: insert()", "[collection change]") {
     _impl::CollectionChangeBuilder c;
 
     SECTION("adds the row to the insertions set") {
@@ -61,7 +61,7 @@ TEST_CASE("collection_change: insert()") {
     }
 }
 
-TEST_CASE("collection_change: modify()") {
+TEST_CASE("collection_change: modify()", "[collection change]") {
     _impl::CollectionChangeBuilder c;
 
     SECTION("marks the row as modified") {
@@ -105,7 +105,7 @@ TEST_CASE("collection_change: modify()") {
     }
 }
 
-TEST_CASE("collection_change: erase()") {
+TEST_CASE("collection_change: erase()", "[collection change]") {
     _impl::CollectionChangeBuilder c;
 
     SECTION("adds the row to the deletions set") {
@@ -161,7 +161,7 @@ TEST_CASE("collection_change: erase()") {
     }
 }
 
-TEST_CASE("collection_change: clear()") {
+TEST_CASE("collection_change: clear()", "[collection change]") {
     _impl::CollectionChangeBuilder c;
 
     SECTION("removes all insertions") {
@@ -193,7 +193,7 @@ TEST_CASE("collection_change: clear()") {
     }
 }
 
-TEST_CASE("collection_change: move()") {
+TEST_CASE("collection_change: move()", "[collection change]") {
     _impl::CollectionChangeBuilder c;
 
     SECTION("adds the move to the list of moves") {
@@ -264,7 +264,7 @@ TEST_CASE("collection_change: move()") {
     }
 }
 
-TEST_CASE("collection_change: calculate() table order") {
+TEST_CASE("collection_change: calculate() table order", "[collection change]") {
     _impl::CollectionChangeBuilder c;
 
     auto all_modified = [](auto) {
@@ -362,7 +362,7 @@ TEST_CASE("collection_change: calculate() table order") {
 #endif
 }
 
-TEST_CASE("collection_change: calculate() sorted") {
+TEST_CASE("collection_change: calculate() sorted", "[collection change]") {
     _impl::CollectionChangeBuilder c;
 
     auto all_modified = [](auto) {
@@ -675,7 +675,7 @@ TEST_CASE("collection_change: calculate() sorted") {
     }
 }
 
-TEST_CASE("collection_change: merge()") {
+TEST_CASE("collection_change: merge()", "[collection change]") {
     _impl::CollectionChangeBuilder c;
 
     SECTION("is a no-op if the new set is empty") {

test/object-store/dictionary.cpp

@@ -1375,7 +1375,7 @@ TEST_CASE("dictionary snapshot null", "[dictionary]") {
     REQUIRE(snapshot.get_any(0) == Mixed{});
 }
 
-TEST_CASE("dictionary aggregate", "[dictionary]") {
+TEST_CASE("dictionary aggregate", "[dictionary][aggregate]") {
     InMemoryTestFile config;
     config.schema = Schema{
         {"DictionaryObject",
@@ -1401,7 +1401,7 @@ TEST_CASE("dictionary aggregate", "[dictionary]") {
     REQUIRE(*sum == 16);
 }
 
-TEST_CASE("callback with empty keypatharray") {
+TEST_CASE("callback with empty keypatharray", "[dictionary]") {
     InMemoryTestFile config;
     config.schema = Schema{
         {"object", {{"links", PropertyType::Dictionary | PropertyType::Object | PropertyType::Nullable, "target"}}},

test/object-store/frozen_objects.cpp

@@ -46,7 +46,7 @@
 using namespace realm;
 using util::any_cast;
 
-TEST_CASE("Construct frozen Realm") {
+TEST_CASE("Construct frozen Realm", "[frozen]") {
     TestFile config;
     config.schema_version = 1;
     config.schema = Schema{
@@ -62,7 +62,7 @@ TEST_CASE("Construct frozen Realm") {
     }
 }
 
-TEST_CASE("Freeze Realm", "[freeze_realm]") {
+TEST_CASE("Freeze Realm", "[frozen]") {
     TestFile config;
     config.schema_version = 1;
     config.schema = Schema{
@@ -112,7 +112,7 @@ TEST_CASE("Freeze Realm", "[freeze_realm]") {
     }
 }
 
-TEST_CASE("Freeze Results", "[freeze_results]") {
+TEST_CASE("Freeze Results", "[frozen]") {
     TestFile config;
     config.schema_version = 1;
     config.schema = Schema{{"object",
@@ -400,7 +400,7 @@ TEST_CASE("Freeze Results", "[freeze_results]") {
     }
 }
 
-TEST_CASE("Freeze List", "[freeze_list]") {
+TEST_CASE("Freeze List", "[frozen]") {
 
     TestFile config;
     config.schema_version = 1;
@@ -476,7 +476,7 @@ TEST_CASE("Freeze List", "[freeze_list]") {
     }
 }
 
-TEST_CASE("Reclaim Frozen", "[reclaim_frozen]") {
+TEST_CASE("Reclaim Frozen", "[frozen]") {
 
 #ifdef REALM_DEBUG
     constexpr int num_pending_transactions = 10;
@@ -589,7 +589,7 @@ TEST_CASE("Reclaim Frozen", "[reclaim_frozen]") {
     realm->commit_transaction();
 }
 
-TEST_CASE("Freeze Object", "[freeze_object]") {
+TEST_CASE("Freeze Object", "[frozen]") {
 
     TestFile config;
     config.schema_version = 1;
@@ -654,7 +654,7 @@ TEST_CASE("Freeze Object", "[freeze_object]") {
     }
 }
 
-TEST_CASE("Freeze dictionary", "[freeze_dictionary]") {
+TEST_CASE("Freeze dictionary", "[frozen]") {
 
     TestFile config;
     config.schema_version = 1;

test/object-store/geospatial.cpp

@@ -95,7 +95,7 @@ struct TestContext : CppContext {
     }
 };
 
-TEST_CASE("geospatial") {
+TEST_CASE("geospatial", "[geospatial]") {
     using namespace std::string_literals;
 
     Schema schema{