Skip to content

Conversation

@ceyonur
Copy link
Collaborator

@ceyonur ceyonur commented Apr 20, 2024

Adds matrix to tests and use race for all UTs

@ceyonur ceyonur marked this pull request as ready for review April 24, 2024 17:54
@ceyonur ceyonur requested a review from darioush as a code owner April 24, 2024 17:54
@ceyonur ceyonur self-assigned this Apr 24, 2024
}

func TestAtomicSyncerResumeNewRootCheckpoint(t *testing.T) {
t.Skip("FLAKY")

Choose a reason for hiding this comment

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

this is fixed in master

@ceyonur ceyonur merged commit 47f7b4d into sync-subnet-evm Apr 24, 2024
@ceyonur ceyonur deleted the race-tests-ci branch April 24, 2024 18:09
darioush pushed a commit that referenced this pull request Apr 25, 2024
* add matrix and race to UTs

* remove unnecessary runners

* clean script

* lazy read bonus blocks

* increase timeout

* increase frequency

* apply fix fow windows

* increase timeout

* disable fail fast

* increase frequency and timeout

* use latest

* fix coma

* better log

* fix require

* Fix formatting

* fix eventually formats

* test out new wg

* Revert "test out new wg"

This reverts commit 5d4a0a8.

* check tx indexes after each block accept

* mark as flaky

* disable blobpool

* add IsSubscribed for testing

* remove subscribed when return

* use atomic bool

* attempt to fix flaky indexing test
darioush pushed a commit that referenced this pull request Apr 27, 2024
* format as subnet-evm

* sync subnet-evm up to c77ba56

* reformat as coreth

* fix tests

* Tx indexing fix (#1131)

* fix unindexor in state sync

* fix lint

* fix flaky test

* Update core/blockchain_test.go

Co-authored-by: Darioush Jalali <[email protected]>
Signed-off-by: Ceyhun Onur <[email protected]>

* Update core/test_blockchain.go

Co-authored-by: Darioush Jalali <[email protected]>
Signed-off-by: Ceyhun Onur <[email protected]>

* Update core/test_blockchain.go

Co-authored-by: Darioush Jalali <[email protected]>
Signed-off-by: Ceyhun Onur <[email protected]>

* Update core/test_blockchain.go

Co-authored-by: Darioush Jalali <[email protected]>
Signed-off-by: Ceyhun Onur <[email protected]>

* fix reviews

* readd delay for index init

* Update core/rawdb/accessors_state_sync.go

Co-authored-by: Darioush Jalali <[email protected]>
Signed-off-by: Ceyhun Onur <[email protected]>

* tx indexer fix: avoids using sleep in test (#1151)

* avoids using sleep in test

* track all goroutines

* trying harder

* add some debug information in case of fail

* Update core/blockchain.go

Signed-off-by: Darioush Jalali <[email protected]>

* Update core/blockchain.go

Signed-off-by: Darioush Jalali <[email protected]>

---------

Signed-off-by: Darioush Jalali <[email protected]>

---------

Signed-off-by: Ceyhun Onur <[email protected]>
Signed-off-by: Darioush Jalali <[email protected]>
Co-authored-by: Darioush Jalali <[email protected]>

* fix require in test

* snapshot: fix race in abort/generate (#1155)

* skip pkg cache and update version (#1158)

* skip pkg cache and update version

* remove goconst linter

* ignore tests for goconst

* use less blocks to gen

* add matrix and race to UTs (#530)

* add matrix and race to UTs

* remove unnecessary runners

* clean script

* lazy read bonus blocks

* increase timeout

* increase frequency

* apply fix fow windows

* increase timeout

* disable fail fast

* increase frequency and timeout

* use latest

* fix coma

* better log

* fix require

* Fix formatting

* fix eventually formats

* test out new wg

* Revert "test out new wg"

This reverts commit 5d4a0a8.

* check tx indexes after each block accept

* mark as flaky

* disable blobpool

* add IsSubscribed for testing

* remove subscribed when return

* use atomic bool

* attempt to fix flaky indexing test

* fix test

* return if sub nil

* Beacon root check (#529)

* add parent beacon root checks

* add beacon root tests

* revert code change

* remove comment

* nits for sync-subnet-evm (#539)

---------

Signed-off-by: Ceyhun Onur <[email protected]>
Signed-off-by: Darioush Jalali <[email protected]>
Co-authored-by: Darioush Jalali <[email protected]>
oxbee pushed a commit to taurusgroup/coreth that referenced this pull request Nov 6, 2024
* format as subnet-evm

* sync subnet-evm up to c77ba56

* reformat as coreth

* fix tests

* Tx indexing fix (ava-labs#1131)

* fix unindexor in state sync

* fix lint

* fix flaky test

* Update core/blockchain_test.go

Co-authored-by: Darioush Jalali <[email protected]>
Signed-off-by: Ceyhun Onur <[email protected]>

* Update core/test_blockchain.go

Co-authored-by: Darioush Jalali <[email protected]>
Signed-off-by: Ceyhun Onur <[email protected]>

* Update core/test_blockchain.go

Co-authored-by: Darioush Jalali <[email protected]>
Signed-off-by: Ceyhun Onur <[email protected]>

* Update core/test_blockchain.go

Co-authored-by: Darioush Jalali <[email protected]>
Signed-off-by: Ceyhun Onur <[email protected]>

* fix reviews

* readd delay for index init

* Update core/rawdb/accessors_state_sync.go

Co-authored-by: Darioush Jalali <[email protected]>
Signed-off-by: Ceyhun Onur <[email protected]>

* tx indexer fix: avoids using sleep in test (ava-labs#1151)

* avoids using sleep in test

* track all goroutines

* trying harder

* add some debug information in case of fail

* Update core/blockchain.go

Signed-off-by: Darioush Jalali <[email protected]>

* Update core/blockchain.go

Signed-off-by: Darioush Jalali <[email protected]>

---------

Signed-off-by: Darioush Jalali <[email protected]>

---------

Signed-off-by: Ceyhun Onur <[email protected]>
Signed-off-by: Darioush Jalali <[email protected]>
Co-authored-by: Darioush Jalali <[email protected]>

* fix require in test

* snapshot: fix race in abort/generate (ava-labs#1155)

* skip pkg cache and update version (ava-labs#1158)

* skip pkg cache and update version

* remove goconst linter

* ignore tests for goconst

* use less blocks to gen

* add matrix and race to UTs (ava-labs#530)

* add matrix and race to UTs

* remove unnecessary runners

* clean script

* lazy read bonus blocks

* increase timeout

* increase frequency

* apply fix fow windows

* increase timeout

* disable fail fast

* increase frequency and timeout

* use latest

* fix coma

* better log

* fix require

* Fix formatting

* fix eventually formats

* test out new wg

* Revert "test out new wg"

This reverts commit 5d4a0a8.

* check tx indexes after each block accept

* mark as flaky

* disable blobpool

* add IsSubscribed for testing

* remove subscribed when return

* use atomic bool

* attempt to fix flaky indexing test

* fix test

* return if sub nil

* Beacon root check (ava-labs#529)

* add parent beacon root checks

* add beacon root tests

* revert code change

* remove comment

* nits for sync-subnet-evm (ava-labs#539)

---------

Signed-off-by: Ceyhun Onur <[email protected]>
Signed-off-by: Darioush Jalali <[email protected]>
Co-authored-by: Darioush Jalali <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants