Skip to content

Commit

Permalink
Fix tests
Browse files Browse the repository at this point in the history
  • Loading branch information
sisuresh committed Aug 8, 2023
1 parent e5deeaa commit d4ad454
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions src/transactions/test/InvokeHostFunctionTests.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -999,7 +999,7 @@ TEST_CASE("contract storage", "[tx][soroban]")
SECTION("restore contract instance and wasm")
{
// Restore Instance and WASM
restoreOp(contractKeys, 68);
restoreOp(contractKeys, 54);

// Instance should now be useable
putWithFootprint(
Expand Down Expand Up @@ -1037,7 +1037,7 @@ TEST_CASE("contract storage", "[tx][soroban]")
SECTION("restore contract wasm, not instance")
{
// Only restore WASM
restoreOp({contractKeys[1]}, 65);
restoreOp({contractKeys[1]}, 51);

// invocation should fail
putWithFootprint(
Expand All @@ -1056,7 +1056,7 @@ TEST_CASE("contract storage", "[tx][soroban]")
SECTION("lifetime extensions")
{
// Restore Instance and WASM
restoreOp(contractKeys, 68);
restoreOp(contractKeys, 54);

auto instanceBumpAmount = 10'000;
auto wasmBumpAmount = 15'000;
Expand All @@ -1065,7 +1065,7 @@ TEST_CASE("contract storage", "[tx][soroban]")
bumpOp(instanceBumpAmount, {contractKeys[0]}, 4);

// bump WASM
bumpOp(wasmBumpAmount, {contractKeys[1]}, 171);
bumpOp(wasmBumpAmount, {contractKeys[1]}, 135);

checkKeyExpirationLedger(contractKeys[0], ledgerSeq,
ledgerSeq + instanceBumpAmount);
Expand Down Expand Up @@ -1239,7 +1239,7 @@ TEST_CASE("contract storage", "[tx][soroban]")
1;

// Bump instance and WASM so that they don't expire during the test
bumpOp(10'000, contractKeys, 97);
bumpOp(10'000, contractKeys, 77);

put("key", 0, ContractDataDurability::PERSISTENT);
checkContractDataExpirationLedger(
Expand Down

5 comments on commit d4ad454

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from dmkozh
at sisuresh@d4ad454

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging sisuresh/stellar-core/upgrade-test = d4ad454 into auto

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sisuresh/stellar-core/upgrade-test = d4ad454 merged ok, testing candidate = 1a29f89

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = 1a29f89

Please sign in to comment.