Skip to content
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

Jump to bottom

Fix flaky CI tests #9010

Closed
9 of 17 tasks
amaury1093 opened this issue Mar 26, 2021 · 2 comments
Closed
9 of 17 tasks

Fix flaky CI tests #9010

amaury1093 opened this issue Mar 26, 2021 · 2 comments
Assignees
@facundomedica
Labels
T: CI T: Tests

Comments

@amaury1093
Copy link
Contributor

amaury1093 commented Mar 26, 2021 •
edited by julienrbrt

Summary of Bug

A couple of tests are flaky. We should fix them.

Version

04246ef

Steps to Reproduce

I noticed several tests that are randomly failing (and more often in the past couple of weeks, but that's a subjective feeling).

These flaky tests can be tackled separately.

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@amaury1093 amaury1093 mentioned this issue Apr 13, 2021
Merged
9 tasks
This was referenced Apr 28, 2021
Merged
Merged
mergify bot pushed a commit that referenced this issue Apr 28, 2021
x/gov/keeper: fix flaky TestPaginatedVotesQuery (#9223)
6ad84c5 
When testing with -race, sometimes the random source generate the same
string for consecutive calls, causing duplicated voter address. So the
number of votes in DB is not 20.

To fix this, we ensure unique addresses are generated, by using a map
for tracking which one was produced, and skip the duplicated address and
generated new one. Testing with:

	go test -race -v -count=1000 -run=TestPaginatedVotesQuery

now passes.

Updates #9010
mergify bot pushed a commit that referenced this issue Apr 28, 2021
x/gov/keeper: fix flaky TestPaginatedVotesQuery (#9223)
3f70ac2 
When testing with -race, sometimes the random source generate the same
string for consecutive calls, causing duplicated voter address. So the
number of votes in DB is not 20.

To fix this, we ensure unique addresses are generated, by using a map
for tracking which one was produced, and skip the duplicated address and
generated new one. Testing with:

	go test -race -v -count=1000 -run=TestPaginatedVotesQuery

now passes.

Updates #9010

(cherry picked from commit 6ad84c5)

# Conflicts:
#	x/gov/keeper/querier_test.go
amaury1093 added a commit that referenced this issue Apr 29, 2021
@mergify @amaury1093
x/gov/keeper: fix flaky TestPaginatedVotesQuery (backport #9223) (#9225)
75f84b5 
* x/gov/keeper: fix flaky TestPaginatedVotesQuery (#9223)

When testing with -race, sometimes the random source generate the same
string for consecutive calls, causing duplicated voter address. So the
number of votes in DB is not 20.

To fix this, we ensure unique addresses are generated, by using a map
for tracking which one was produced, and skip the duplicated address and
generated new one. Testing with:

	go test -race -v -count=1000 -run=TestPaginatedVotesQuery

now passes.

Updates #9010

(cherry picked from commit 6ad84c5)

# Conflicts:
#	x/gov/keeper/querier_test.go

* fi merge conflict

Co-authored-by: Cuong Manh Le <cuong@orijtech.com>
Co-authored-by: Amaury M <1293565+amaurym@users.noreply.github.com>
@github-actions github-actions bot added the stale label Jul 24, 2021
@amaury1093 amaury1093 added pinned and removed stale labels Jul 30, 2021
@amaury1093 amaury1093 reopened this Jul 30, 2021
@github-actions github-actions bot added the stale label Sep 20, 2021
@amaury1093 amaury1093 mentioned this issue Oct 27, 2021
Merged
19 tasks
@julienrbrt julienrbrt self-assigned this Apr 23, 2022
@julienrbrt julienrbrt removed the stale label Apr 23, 2022
@julienrbrt julienrbrt mentioned this issue Apr 23, 2022
Closed
@amaury1093 amaury1093 mentioned this issue Apr 28, 2022
Merged
19 tasks
@tac0turtle tac0turtle removed the pinned label May 9, 2022
@p0mvn
Copy link
Member

p0mvn commented May 10, 2022

Please consider adding here to track: #11923

@facundomedica facundomedica mentioned this issue May 11, 2022
Closed
4 tasks
@julienrbrt julienrbrt removed their assignment May 13, 2022
@rootulp rootulp mentioned this issue Jul 18, 2022
Merged
@julienrbrt julienrbrt mentioned this issue Jan 14, 2023
Closed
@prettymuchbryce prettymuchbryce mentioned this issue Jan 30, 2023
Open
@facundomedica facundomedica self-assigned this Feb 2, 2023
@tac0turtle
Copy link
Member

closing this since ci is less flaky with these tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@facundomedica facundomedica

Labels
T: CI T: Tests
Projects
No open projects
Cosmos SDK Maintenance
Status: Done
Milestone
No milestone
Development

No branches or pull requests
6 participants
@amaury1093 @alexanderbez @facundomedica @tac0turtle @julienrbrt @p0mvn