Add test for PeerIdTooLong and clean up code

Cargo.toml

@@ -20,6 +20,7 @@ members = [
 	"pallets/did",
 	"pallets/did-name",
 	"pallets/identity",
+	"pallets/network-authorization",
 	"pallets/network-membership",
 	"pallets/network-score",
 	"pallets/node-authorization",
@@ -155,6 +156,8 @@ pallet-transaction-weight-runtime-api = { path = "runtimes/common/api/weight", d
 pallet-registries = { path = "pallets/registries", default-features = false }
 pallet-entries = { path = "pallets/entries", default-features = false }
 pallet-schema-accounts = { path = "pallets/schema-accounts", default-features = false }
+pallet-network-authorization = { path = "pallets/network-authorization" }
+
 
 
 # substrate dependencies

pallets/node-authorization/src/tests.rs

@@ -459,3 +459,19 @@ fn test_generate_peer_id_invalid_utf8() {
 	let invalid_node_id: NodeId = vec![0xFF, 0xFE, 0xFD];
 	assert_err!(NodeAuthorization::generate_peer_id(&invalid_node_id), Error::<Test>::InvalidUtf8);
 }
+
+// the test to check that the PeerId not too long.
+#[test]
+fn add_well_known_node_with_peer_id_too_long_should_fail() {
+    new_test_ext().execute_with(|| {
+        // Attempt to add a node with an excessively long peer ID
+        assert_noop!(
+            NodeAuthorization::add_well_known_node(
+                RuntimeOrigin::signed(1),
+                test_node(TEST_NODE_LEN), // Exceeds max length
+                15
+            ),
+            Error::<Test>::PeerIdTooLong
+        );
+    });
+}

test-utils/service/src/lib.rs

@@ -178,7 +178,7 @@ where
 				interval.tick().await;
 
 				if full_nodes.iter().all(|(id, service, _, _)| full_predicate(*id, service)) {
-					break
+					break;
 				}
 			}
 		};