@@ -111,10 +111,12 @@ TEST_F(MetadataSerdeTest, DeserializeV2Valid) {
111111 ASSERT_TRUE (schema.has_value ());
112112 EXPECT_EQ (*(schema.value ().get ()), *expected_schema);
113113
114+ // schema with ID 1
114115 auto schema_v1 = metadata->SchemaById (1 );
115116 ASSERT_TRUE (schema_v1.has_value ());
116117 EXPECT_EQ (*(schema_v1.value ().get ()), *expected_schema);
117118
119+ // schema with ID 0
118120 auto expected_schema_v0 = std::make_shared<Schema>(
119121 std::vector<SchemaField>{schema_fields.at (0 )}, /* schema_id=*/ 0 );
120122 auto schema_v0 = metadata->SchemaById (0 );
@@ -173,10 +175,13 @@ TEST_F(MetadataSerdeTest, DeserializeV2Valid) {
173175 for (size_t i = 0 ; i < expected_snapshots.size (); ++i) {
174176 EXPECT_EQ (*metadata->snapshots [i], expected_snapshots[i]);
175177 }
178+
179+ // snapshot with ID 3051729675574597004
176180 auto snapshot_v0 = metadata->SnapshotById (3051729675574597004 );
177181 ASSERT_TRUE (snapshot_v0.has_value ());
178182 EXPECT_EQ (*snapshot_v0.value (), expected_snapshots[0 ]);
179183
184+ // snapshot with ID 3055729675574597004
180185 auto snapshot_v1 = metadata->SnapshotById (3055729675574597004 );
181186 ASSERT_TRUE (snapshot_v1.has_value ());
182187 EXPECT_EQ (*snapshot_v1.value (), expected_snapshots[1 ]);
0 commit comments