test(report): capture all EIP-712 field screens

Author: BitHighlander

scripts/generate-test-report.py

@@ -2935,7 +2935,7 @@ def _arg_shown(a):
        'f2cee375...912090f and messageHash c52c0ee5...4b371e, both published, both matched on '
        'hardware and in the emulator.',
        ['Domain name', 'Domain version', 'chainId', 'verifyingContract (42 chars, in full)',
-        'Cow / wallet', 'Bob / wallet', 'contents']),
+        'From name: Cow', 'From wallet', 'To name: Bob', 'To wallet', 'contents']),
       ('TD2', 'test_msg_eip712_streaming', 'test_array_of_structs_walks',
        'An array of structs walks and signs',
        'Arrays hash WITHOUT a typeHash prefix -- enc(array) is the keccak of the concatenated '

tests/test_msg_eip712_streaming.py

@@ -118,6 +118,10 @@ def _walk(self, doc, max_steps=400):
         resp = self.client.call_raw(msg)
         for _ in range(max_steps):
             if isinstance(resp, proto.ButtonRequest):
+                # This is a manual, device-driven call_raw() loop, so no
+                # callback_ButtonRequest() will capture the field being
+                # approved. Retain it while that exact field is still active.
+                self.client.capture_oled()
                 self.client.debug.press_yes()
                 resp = self.client.call_raw(proto.ButtonAck())
             elif isinstance(resp, eth.EthereumTypedDataStructRequest):
@@ -140,6 +144,8 @@ def setUp(self):
         self.requires_structured_eip712()
         self.setup_mnemonic_nopin_nopassphrase()
         self.client.apply_policy('AdvancedMode', 1)
+        # The report entries describe typed-data fields, not the policy prompt.
+        self.client.reset_screenshots()
 
     def test_spec_example_matches_the_published_hashes(self):
         """The device's own hashes equal the EIP-712 reference implementation's.