Skip to content

Commit 4636ae1

Browse files
authored
Fix debugging tests (#3326)
<!-- Reference any GitHub issues resolved by this PR --> ## Introduced changes <!-- A brief description of the changes --> - Fix docs snippets related to debugging - Fix debugging tests ## Checklist <!-- Make sure all of these are complete --> - [x] Linked relevant issue - [x] Updated relevant documentation - [x] Added relevant tests - [x] Performed self-review of the code - [x] Added changes to `CHANGELOG.md`
1 parent 01b33f6 commit 4636ae1

File tree

2 files changed

+25
-25
lines changed

2 files changed

+25
-25
lines changed

crates/forge/tests/e2e/debugging.rs

Lines changed: 21 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ fn test_output(trace_message_fn: fn(&str) -> String) -> String {
5555
note: run with `SNFORGE_BACKTRACE=1` environment variable to display a backtrace
5656
{debugging_trace_fail}
5757
58-
[PASS] trace_info_integrationtest::test_trace::test_debugging_trace_success (l1_gas: ~0, l1_data_gas: ~288, l2_gas: ~1600000)
58+
[PASS] trace_info_integrationtest::test_trace::test_debugging_trace_success (l1_gas: ~[..], l1_data_gas: ~[..], l2_gas: ~[..])
5959
{debugging_trace_pass}
6060
6161
Running 0 test(s) from src/
@@ -75,49 +75,49 @@ fn detailed_debugging_trace_message(test_name: &str) -> String {
7575
├─ [selector] execute_calls
7676
│ ├─ [contract name] SimpleContract
7777
│ ├─ [entry point type] External
78-
│ ├─ [calldata] array![RecursiveCall {{ contract_address: ContractAddress(0x10a2fac439604ce4129fe7c205b711e8141e12e2e52e08f7f898fe7ac13f0a), payload: array![RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}, RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}] }}, RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}]
79-
│ ├─ [storage address] 0x7b29abec6baad44d169ee10b37c9a1eae834d71887607f60d2f90836f6eb973
80-
│ ├─ [caller address] 0x1724987234973219347210837402
78+
│ ├─ [calldata] array![RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}, RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}] }}, RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}]
79+
│ ├─ [storage address] [..]
80+
│ ├─ [caller address] [..]
8181
│ ├─ [call type] Call
82-
│ ├─ [call result] success: array![RecursiveCall {{ contract_address: ContractAddress(0x10a2fac439604ce4129fe7c205b711e8141e12e2e52e08f7f898fe7ac13f0a), payload: array![RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}, RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}] }}, RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}]
82+
│ ├─ [call result] success: array![RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}, RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}] }}, RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}]
8383
│ ├─ [selector] execute_calls
8484
│ │ ├─ [contract name] SimpleContract
8585
│ │ ├─ [entry point type] External
86-
│ │ ├─ [calldata] array![RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}, RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}]
87-
│ │ ├─ [storage address] 0x10a2fac439604ce4129fe7c205b711e8141e12e2e52e08f7f898fe7ac13f0a
88-
│ │ ├─ [caller address] 0x7b29abec6baad44d169ee10b37c9a1eae834d71887607f60d2f90836f6eb973
86+
│ │ ├─ [calldata] array![RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}, RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}]
87+
│ │ ├─ [storage address] [..]
88+
│ │ ├─ [caller address] [..]
8989
│ │ ├─ [call type] Call
90-
│ │ ├─ [call result] success: array![RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}, RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}]
90+
│ │ ├─ [call result] success: array![RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}, RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}]
9191
│ │ ├─ [selector] execute_calls
9292
│ │ │ ├─ [contract name] SimpleContract
9393
│ │ │ ├─ [entry point type] External
9494
│ │ │ ├─ [calldata] array![]
95-
│ │ │ ├─ [storage address] 0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda
96-
│ │ │ ├─ [caller address] 0x10a2fac439604ce4129fe7c205b711e8141e12e2e52e08f7f898fe7ac13f0a
95+
│ │ │ ├─ [storage address] [..]
96+
│ │ │ ├─ [caller address] [..]
9797
│ │ │ ├─ [call type] Call
9898
│ │ │ └─ [call result] success: array![]
9999
│ │ └─ [selector] execute_calls
100100
│ │ ├─ [contract name] SimpleContract
101101
│ │ ├─ [entry point type] External
102102
│ │ ├─ [calldata] array![]
103-
│ │ ├─ [storage address] 0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda
104-
│ │ ├─ [caller address] 0x10a2fac439604ce4129fe7c205b711e8141e12e2e52e08f7f898fe7ac13f0a
103+
│ │ ├─ [storage address] [..]
104+
│ │ ├─ [caller address] [..]
105105
│ │ ├─ [call type] Call
106106
│ │ └─ [call result] success: array![]
107107
│ └─ [selector] execute_calls
108108
│ ├─ [contract name] SimpleContract
109109
│ ├─ [entry point type] External
110110
│ ├─ [calldata] array![]
111-
│ ├─ [storage address] 0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda
112-
│ ├─ [caller address] 0x7b29abec6baad44d169ee10b37c9a1eae834d71887607f60d2f90836f6eb973
111+
│ ├─ [storage address] [..]
112+
│ ├─ [caller address] [..]
113113
│ ├─ [call type] Call
114114
│ └─ [call result] success: array![]
115115
└─ [selector] fail
116116
├─ [contract name] SimpleContract
117117
├─ [entry point type] External
118118
├─ [calldata] array![0x1, 0x2, 0x3, 0x4, 0x5]
119-
├─ [storage address] 0x7b29abec6baad44d169ee10b37c9a1eae834d71887607f60d2f90836f6eb973
120-
├─ [caller address] 0x1724987234973219347210837402
119+
├─ [storage address] [..]
120+
├─ [caller address] [..]
121121
├─ [call type] Call
122122
└─ [call result] panic: (0x1, 0x2, 0x3, 0x4, 0x5)
123123
"}
@@ -128,12 +128,12 @@ fn standard_debugging_trace_message(test_name: &str) -> String {
128128
[test name] trace_info_integrationtest::test_trace::test_debugging_trace_{test_name}
129129
├─ [selector] execute_calls
130130
│ ├─ [contract name] SimpleContract
131-
│ ├─ [calldata] array![RecursiveCall {{ contract_address: ContractAddress(0x10a2fac439604ce4129fe7c205b711e8141e12e2e52e08f7f898fe7ac13f0a), payload: array![RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}, RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}] }}, RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}]
132-
│ ├─ [call result] success: array![RecursiveCall {{ contract_address: ContractAddress(0x10a2fac439604ce4129fe7c205b711e8141e12e2e52e08f7f898fe7ac13f0a), payload: array![RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}, RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}] }}, RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}]
131+
│ ├─ [calldata] array![RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}, RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}] }}, RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}]
132+
│ ├─ [call result] success: array![RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}, RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}] }}, RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}]
133133
│ ├─ [selector] execute_calls
134134
│ │ ├─ [contract name] SimpleContract
135-
│ │ ├─ [calldata] array![RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}, RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}]
136-
│ │ ├─ [call result] success: array![RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}, RecursiveCall {{ contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }}]
135+
│ │ ├─ [calldata] array![RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}, RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}]
136+
│ │ ├─ [call result] success: array![RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}, RecursiveCall {{ contract_address: ContractAddress([..]), payload: array![] }}]
137137
│ │ ├─ [selector] execute_calls
138138
│ │ │ ├─ [contract name] SimpleContract
139139
│ │ │ ├─ [calldata] array![]

docs/src/snforge-advanced-features/debugging.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -39,12 +39,12 @@ $ snforge test --trace-verbosity standard
3939
[test name] trace_info_integrationtest::test_trace::test_debugging_trace_success
4040
├─ [selector] execute_calls
4141
│ ├─ [contract name] SimpleContract
42-
│ ├─ [calldata] array![RecursiveCall { contract_address: ContractAddress(0x10a2fac439604ce4129fe7c205b711e8141e12e2e52e08f7f898fe7ac13f0a), payload: array![RecursiveCall { contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }, RecursiveCall { contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }] }, RecursiveCall { contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }]
43-
│ ├─ [call result] success: array![RecursiveCall { contract_address: ContractAddress(0x10a2fac439604ce4129fe7c205b711e8141e12e2e52e08f7f898fe7ac13f0a), payload: array![RecursiveCall { contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }, RecursiveCall { contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }] }, RecursiveCall { contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }]
42+
│ ├─ [calldata] array![RecursiveCall { contract_address: ContractAddress([..]), payload: array![RecursiveCall { contract_address: ContractAddress([..]), payload: array![] }, RecursiveCall { contract_address: ContractAddress([..]), payload: array![] }] }, RecursiveCall { contract_address: ContractAddress([..]), payload: array![] }]
43+
│ ├─ [call result] success: array![RecursiveCall { contract_address: ContractAddress([..]), payload: array![RecursiveCall { contract_address: ContractAddress([..]), payload: array![] }, RecursiveCall { contract_address: ContractAddress([..]), payload: array![] }] }, RecursiveCall { contract_address: ContractAddress([..]), payload: array![] }]
4444
│ ├─ [selector] execute_calls
4545
│ │ ├─ [contract name] SimpleContract
46-
│ │ ├─ [calldata] array![RecursiveCall { contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }, RecursiveCall { contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }]
47-
│ │ ├─ [call result] success: array![RecursiveCall { contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }, RecursiveCall { contract_address: ContractAddress(0x28f58bf524dc0adcf7468c67d7ffdac1e5d885d347c6a498978f538984dbda), payload: array![] }]
46+
│ │ ├─ [calldata] array![RecursiveCall { contract_address: ContractAddress([..]), payload: array![] }, RecursiveCall { contract_address: ContractAddress([..]), payload: array![] }]
47+
│ │ ├─ [call result] success: array![RecursiveCall { contract_address: ContractAddress([..]), payload: array![] }, RecursiveCall { contract_address: ContractAddress([..]), payload: array![] }]
4848
│ │ ├─ [selector] execute_calls
4949
│ │ │ ├─ [contract name] SimpleContract
5050
│ │ │ ├─ [calldata] array![]

0 commit comments

Comments
 (0)