You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Recently this SDK had a bug where the transaction extra pages field, apep, was being ignored when decoding a transaction. In order to prevent bugs like this from happening again, we can create a test that automatically verifies each field persists during a roundtrip of encoding + decoding.
Scope
In order to do this, we'll need a list of each encoded transaction field key name. This must be updated with every new txn field.
Then, there are two options:
Make an object with every single field from that list present, then run it through a roundtrip of Transaction.get_obj_for_encoding and Transaction. from_obj_for_encoding and verify all fields are still present.
For each field from the list, make an object with ONLY that field present, then run it through a roundtrip of Transaction.get_obj_for_encoding and Transaction. from_obj_for_encoding and verify that field is still present.
The text was updated successfully, but these errors were encountered:
Summary
Recently this SDK had a bug where the transaction extra pages field,
apep
, was being ignored when decoding a transaction. In order to prevent bugs like this from happening again, we can create a test that automatically verifies each field persists during a roundtrip of encoding + decoding.Scope
In order to do this, we'll need a list of each encoded transaction field key name. This must be updated with every new txn field.
Then, there are two options:
Transaction.get_obj_for_encoding
andTransaction. from_obj_for_encoding
and verify all fields are still present.Transaction.get_obj_for_encoding
andTransaction. from_obj_for_encoding
and verify that field is still present.The text was updated successfully, but these errors were encountered: