Skip to content

add a non-sequencing node to localnet and test it #507

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
ClaytonNorthey92 merged 10 commits into from
Jul 2, 2025
Merged

add a non-sequencing node to localnet and test it #507

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
76b66fa
add a non-sequencing node to localnet and test it
ClaytonNorthey92 May 19, 2025
c4fd18d
removed commented out code
ClaytonNorthey92 Jun 6, 2025
4ef8663
linting fixes
ClaytonNorthey92 Jun 6, 2025
a1cd652
linting fixes
ClaytonNorthey92 Jun 6, 2025
4575c17
linting fixes
ClaytonNorthey92 Jun 6, 2025
ce61e5d
remove retries
ClaytonNorthey92 Jun 6, 2025
f02d7cd
added retries where optimism creates its own ctx timeout
ClaytonNorthey92 Jun 9, 2025
5392b5c
removed versbose log line
ClaytonNorthey92 Jun 9, 2025
a79df2a
better printing
ClaytonNorthey92 Jun 27, 2025
49b6e11
ran linter
ClaytonNorthey92 Jun 27, 2025
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
Prev Previous commit
Next Next commit
added retries where optimism creates its own ctx timeout
  • Loading branch information
@ClaytonNorthey92
ClaytonNorthey92 committed Jul 2, 2025
commit f02d7cda4ffbb5ec43794047932ef0b7add4e36f
45 changes: 36 additions & 9 deletions e2e/monitor/main_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -588,7 +588,15 @@ func bridgeEthL2ToL1(t *testing.T, ctx context.Context, l1Client *ethclient.Clie

t.Logf("assuming L2OutputOracle is at %s", ooproxy)

blockNumber, err := wait.ForOutputRootPublished(ctx, l1Client, ooproxy, receipt.BlockNumber)
var blockNumber uint64

for range 5 {
blockNumber, err = wait.ForOutputRootPublished(ctx, l1Client, ooproxy, receipt.BlockNumber)
if err == nil {
break
}
}

if err != nil {
t.Fatal(err)
}
Expand Down Expand Up @@ -678,8 +686,14 @@ func bridgeEthL2ToL1(t *testing.T, ctx context.Context, l1Client *ethclient.Clie
break
}

t.Logf("waiting for the finalization period")
if err := wait.ForFinalizationPeriod(ctx, l1Client, receipt.BlockNumber, ooproxy); err != nil {
for range 5 {
t.Logf("waiting for the finalization period")
if err := wait.ForFinalizationPeriod(ctx, l1Client, receipt.BlockNumber, ooproxy); err == nil {
break
}
}

if err != nil {
t.Fatal(err)
}

Expand Down Expand Up @@ -1004,7 +1018,14 @@ func bridgeERC20FromL2ToL1(t *testing.T, ctx context.Context, l1Address common.A

t.Logf("assuming L2OutputOracle is at %s", ooproxy)

blockNumber, err := wait.ForOutputRootPublished(ctx, l1Client, ooproxy, receipt.BlockNumber)
var blockNumber uint64

for range 5 {
blockNumber, err = wait.ForOutputRootPublished(ctx, l1Client, ooproxy, receipt.BlockNumber)
if err == nil {
break
}
}
if err != nil {
t.Fatal(err)
}
Expand Down Expand Up @@ -1094,8 +1115,14 @@ func bridgeERC20FromL2ToL1(t *testing.T, ctx context.Context, l1Address common.A
break
}

t.Logf("waiting for the finalization period")
if err := wait.ForFinalizationPeriod(ctx, l1Client, receipt.BlockNumber, ooproxy); err != nil {
for range 5 {
t.Logf("waiting for the finalization period")
if err = wait.ForFinalizationPeriod(ctx, l1Client, receipt.BlockNumber, ooproxy); err == nil {
break
}
}

if err != nil {
t.Fatal(err)
}

Expand Down Expand Up @@ -1283,18 +1310,18 @@ func assertOutputRootsAreTheSame(t *testing.T, ctx context.Context, l2Client *et
}

if err := assertResultNotError(&outputAtBlockResponseOne); err != nil {
t.Fatalf("error in response: %v", outputAtBlockResponseOne.Error)
t.Fatalf("error in response: %v", outputAtBlockResponseOne.Error)
}

if err := assertResultNotError(&outputAtBlockResponseTwo); err != nil {
t.Fatalf("error in response: %v", outputAtBlockResponseTwo.Error)
t.Fatalf("error in response: %v", outputAtBlockResponseTwo.Error)
}

assertResultNotError(&outputAtBlockResponseOne)
assertResultNotError(&outputAtBlockResponseTwo)

if diff := deep.Equal(outputAtBlockResponseOne, outputAtBlockResponseTwo); len(diff) > 0 {
t.Fatalf("output roots are not the same: %s", diff)
t.Fatalf("output roots are not the same: %s", diff)
}

tip--
Expand Down