Skip to content

docs: update comments on the DAVerifier test to reflect the correct transaction type #173

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Aug 23, 2023
Merged

docs: update comments on the DAVerifier test to reflect the correct transaction type #173

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
77fffc2
docs: update comments on the DAVerifier test to reflect the correct t…
rach-id Aug 10, 2023
b977a5a
Merge branch 'master' into correct_tx_type_docs
rach-id Aug 22, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 7 additions & 7 deletions src/lib/verifier/test/DAVerifier.t.sol
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -202,11 +202,11 @@ contract DAVerifierTest is DSTest {
}
}

/// @title TestFixture contains the necessary information to create proofs for the blob
/// that was posted to Celestia. It represents the data mentioned in the comment at
/// the beginning of this file.
/// @title TestFixture contains the necessary information to create proofs for the token
/// transfer transaction that happened on Celestia. It represents the data mentioned in
/// the comment at the beginning of this file.
contract TestFixture {
/// @notice the share containing the blob that was published to Celestia.
/// @notice the share containing the token transfer transaction on Celestia.
bytes public shareData = abi.encodePacked(
hex"0000000000000000000000000000000000000000000000000000000001010000",
hex"014500000026c3020a95010a92010a1c2f636f736d6f732e62616e6b2e763162",
Expand Down Expand Up @@ -249,10 +249,10 @@ contract TestFixture {
hex"ffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff7329c7d336d0140840837fc0d8eafa2403f4f6b019b602581cd9f04e28026eae"
);

/// @notice the data root of the block containing the submitted blob.
/// @notice the data root of the block containing the token transfer transaction.
bytes32 public dataRoot = 0x55cfc29fc0cd263906122d5cb859091224495b141fc0c51529612d7ab8962950;

/// @notice the height of the block containing the submitted blob.
/// @notice the height of the block containing the submitted token transfer transaction.
uint256 public height = 3;

/// @notice the data root tuple root committing to the Celestia block.
Expand Down Expand Up @@ -282,7 +282,7 @@ contract TestFixture {
bytes32(0xff576381b02abadc50e414f6b4efcae31091cd40a5aba75f56be52d1bb2efcae)
];

/// @notice the data root tuple of the block containing the submitted blob.
/// @notice the data root tuple of the block containing the token transfer transaction.
function getDataRootTuple() public view returns (DataRootTuple memory) {
return DataRootTuple(height, dataRoot);
}
Expand Down