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.

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

feat: addresses module #291

Merged
merged 32 commits into from
Oct 8, 2024
Merged
Changes from 1 commit
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
a8bcd0e
Initial proto
cbrit Mar 13, 2024
1814cf1
Generate addressmap proto bindings
cbrit Mar 13, 2024
6bc712c
Change module name to addresses to avoid key prefix collision
cbrit Mar 13, 2024
8d36899
WIP - addresses module
cbrit Mar 14, 2024
4b374a1
WIP - address module builds
cbrit Mar 14, 2024
d415759
Unit tests and refactoring
cbrit Mar 15, 2024
c62622e
CLI queries and tests
cbrit Mar 15, 2024
3ed4f58
tx CLI commands and unit tests
cbrit Mar 15, 2024
1dbb93c
Add addresses gen state to setup_test
cbrit Mar 15, 2024
abe2f37
Rename default genesis state method
cbrit Mar 15, 2024
87e5901
Addresses integration test
cbrit Mar 15, 2024
19729af
Fixing linting errors
cbrit Mar 15, 2024
426819d
More linting fixes
cbrit Mar 15, 2024
f1f06a8
More linting
cbrit Mar 15, 2024
8a6e97a
A few tweaks
cbrit Mar 15, 2024
9cffc23
Test pagination in query unit test
cbrit Mar 18, 2024
7b51fab
Review items
cbrit Mar 18, 2024
cb2fc5a
Merge branch 'main' into collin/addresses
cbrit Mar 18, 2024
0b5ec42
Merge branch 'main' into collin/addresses
cbrit Aug 7, 2024
b6f11d5
Compiler errors, proto errors. WIP - need to bump go version to 1.22
cbrit Aug 7, 2024
5e35ce0
Fix addresses test
cbrit Aug 7, 2024
113970c
Fix addresses test
cbrit Aug 7, 2024
7e89e5e
Make linter happy
cbrit Aug 19, 2024
bf31ad9
Merge branch 'main' into collin/addresses
cbrit Sep 25, 2024
8e3d78f
Unhappy path integration tests for x/addresses
cbrit Sep 25, 2024
34fdfc0
Unhandled error
cbrit Sep 25, 2024
1673648
Improved keeper unit tests
cbrit Sep 25, 2024
8ac8e30
More unit tests
cbrit Sep 25, 2024
eb632dd
Make CI linter happy
cbrit Sep 26, 2024
3143444
more linter stuff
cbrit Sep 26, 2024
cf334d5
Update proto-builder image version, and add missing directives to x/a…
cbrit Oct 7, 2024
e4cfdbe
Merge branch 'main' into collin/addresses
cbrit Oct 8, 2024
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
Fix addresses test
cbrit committed Aug 7, 2024
commit 113970c98f8726eda72421edae2588a0b8e195c0
8 changes: 8 additions & 0 deletions integration_tests/addresses_test.go
Original file line number Diff line number Diff line change
@@ -70,5 +70,13 @@ func (s *IntegrationTestSuite) TestAddresses() {

return len(queryRes.AddressMappings) == 0
}, 20*time.Second, 4*time.Second, "address mapping not deleted")

_, err = addressesQueryClient.QueryAddressMappingByEVMAddress(context.Background(), &types.QueryAddressMappingByEVMAddressRequest{EvmAddress: evmAddress.Hex()})
s.Require().Error(err)
s.Require().Contains(err.Error(), "code = NotFound")

_, err = addressesQueryClient.QueryAddressMappingByCosmosAddress(context.Background(), &types.QueryAddressMappingByCosmosAddressRequest{CosmosAddress: cosmosAddress.String()})
s.Require().Error(err)
s.Require().Contains(err.Error(), "code = NotFound")
})
}

Unchanged files with check annotations Beta

// ProposalContents returns all the distribution content functions used to
// simulate governance proposals.
func (am AppModule) ProposalContents(_ module.SimulationState) []sim.WeightedProposalContent {

Check failure on line 160 in x/addresses/module.go

GitHub Actions / golangci-lint

SA1019: sim.WeightedProposalContent is deprecated: Use WeightedProposalMsg instead. (staticcheck)
return nil
}