Add a comment to the added test

radixdlt · Dec 13, 2024 · 7978a88 · 7978a88
1 parent 9061fa8
commit 7978a88
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/.vscode/settings.json b/.vscode/settings.json
@@ -36,6 +36,7 @@
         "testcases",
         "thiserror",
         "typeshare",
+        "uncategorized",
         "unhashed",
         "uniffi",
         "unstake",

diff --git a/crates/radix-engine-toolkit/tests/transaction_types.rs b/crates/radix-engine-toolkit/tests/transaction_types.rs
@@ -2503,6 +2503,11 @@ fn account_locker_is_recognized_as_general_transaction() {
     assert_eq!(dynamic_analysis.new_entities, NewEntities::default());
 }
 
+// This test ensures that a transaction intent manifest is classified as uncategorized in the Gateway
+// The wallet currently uses the classification of the transaction intent manifest as the classification of the transaction
+// For an MVP, we want the wallet to show transactions with subintents as Complex/Uncategorized, so this
+// test checks that. In future, we may wish to revisit this and change this restriction, but we should make
+// sure that the transactions still display reasonably in the wallet.
 #[test]
 fn manifest_with_yield_to_child_has_no_classifications() {
     // Arrange