chore(*): Improve coverage (#485)

* improve coverage: iota-names controller

* improve coverage: subnames config

* review suggestions

Author: Alex6323

packages/iota-names/tests/controller_tests.move

@@ -464,6 +464,17 @@ fun test_set_user_data() {
     assert_eq(*data.get(&DATA_KEY_AVATAR.to_string()), b"value_avatar".to_string());
     assert_eq(*data.get(&utf8(DATA_KEY_IPFS)), b"value_ipfs".to_string());
 
+    set_user_data_util(
+        scenario,
+        FIRST_ADDRESS,
+        DATA_KEY_AVATAR.to_string(),
+        b"value_new_avatar".to_string(),
+        0,
+    );
+    let data = &scenario.get_user_data(NAME.to_string());
+    assert_eq(data.size(), 2);
+    assert_eq(*data.get(&DATA_KEY_AVATAR.to_string()), b"value_new_avatar".to_string());
+
     scenario_val.end();
 }
 
@@ -748,4 +759,4 @@ fun test_reverse_reset_when_target_address_changes() {
 
     destroy(uid);
     scenario_val.end();
-}
+}

packages/subnames/tests/unit_tests.move

@@ -8,7 +8,7 @@ module iota_names_subnames::unit_tests;
 
 use iota_names::name::{Self, new as new_name, parent};
 use std::string::utf8;
-use iota_names_subnames::config::{assert_is_valid_subname, default};
+use iota_names_subnames::config::{assert_is_valid_subname, default, new as new_config};
 
 // === Validity of subname | parent lengths (based on string) ===
 #[test]
@@ -105,16 +105,16 @@ fun test_invalid_parent_sln_failure() {
 fun test_invalid_child_label_size_failure() {
     assert_is_valid_subname(
         &new_name(utf8(b"sub.test.iota")),
-        &new_name(utf8(b"ob.example.iota")),
+        &new_name(utf8(b"ob.test.iota")),
         &default(),
     );
 }
 
 #[test, expected_failure(abort_code = iota_names_subnames::config::ENotSupportedTLN)]
 fun test_not_supported_tln_failure() {
     assert_is_valid_subname(
-        &new_name(utf8(b"sub.sub.example.move")),
         &new_name(utf8(b"sub.example.move")),
+        &new_name(utf8(b"sub.sub.example.move")),
         &default(),
     );
 }
@@ -140,3 +140,12 @@ fun derive_parent_from_child() {
         0,
     );
 }
+
+#[test]
+fun test_custom_config() {
+    assert_is_valid_subname(
+        &new_name(utf8(b"a.move")),
+        &new_name(utf8(b"b.a.move")),
+        &new_config(vector[b"move".to_string()], 3, 1, 60 * 60 * 1000),
+    );
+}