Skip to content

Commit

Permalink
value_tests.rs: Remove comments that mention legacy traits.
Browse files Browse the repository at this point in the history
  • Loading branch information
Lorak-mmk committed Jan 30, 2025
1 parent ee0bbb4 commit fa66d14
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions scylla-cql/src/frame/value_tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -884,8 +884,7 @@ fn cqlvalue_serialization() {
]
);

// Unlike the legacy Value trait, SerializeValue takes case of reordering
// the fields
// SerializeValue takes case of reordering the fields
let udt = CqlValue::UserDefinedType {
keyspace: "ks".to_string(),
name: "t".to_string(),
Expand Down Expand Up @@ -1256,9 +1255,7 @@ fn tuple_value_list() {

#[test]
fn map_value_list() {
// The legacy ValueList would serialize this as a list of named values,
// whereas the new SerializeRow will order the values by their names.

// SerializeRow will order the values by their names.
// Note that the alphabetical order of the keys is "ala", "kota", "ma",
// but the impl sorts properly.
let row = BTreeMap::from_iter([("ala", 1), ("ma", 2), ("kota", 3)]);
Expand Down

0 comments on commit fa66d14

Please sign in to comment.