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

[Supplier] Refactor/non custodial staking tests #718

Merged
merged 53 commits into from
Aug 13, 2024
Merged
Show file tree
Hide file tree
Changes from 46 commits
Commits
Show all changes
53 commits
Select commit Hold shift + click to select a range
a18ba42
feat: Implement non-custodial staking
red-0ne Jul 30, 2024
676c2c4
chore: Address review change requests
red-0ne Aug 1, 2024
14ea326
Merge remote-tracking branch 'origin/main' into refactor/non-custodia…
red-0ne Aug 1, 2024
bae1f00
refactor: unit/integration tests
red-0ne Aug 2, 2024
b44e07b
Merge remote-tracking branch 'origin/main' into feat/non-custodial-st…
red-0ne Aug 2, 2024
5f5e98b
Merge remote-tracking branch 'origin/feat/non-custodial-staking' into…
red-0ne Aug 2, 2024
473e042
fix: Remove tabs and non-printable chars
red-0ne Aug 2, 2024
3d01f53
Merge remote-tracking branch 'origin/main' into feat/non-custodial-st…
red-0ne Aug 2, 2024
2d064fd
Merge branch 'feat/non-custodial-staking' into refactor/non-custodial…
red-0ne Aug 2, 2024
0699a35
fix: Remove shadowed variable
red-0ne Aug 2, 2024
85e6b81
Merge remote-tracking branch 'origin/main' into refactor/non-custodia…
red-0ne Aug 2, 2024
bb84ab1
Merge remote-tracking branch 'origin/main' into feat/non-custodial-st…
red-0ne Aug 2, 2024
498fc26
Merge remote-tracking branch 'origin/feat/non-custodial-staking' into…
red-0ne Aug 2, 2024
853724a
fix: All tests working
red-0ne Aug 3, 2024
6d7151e
fix: Revert merge changes
red-0ne Aug 3, 2024
20b26e7
chore: Remove imoprt newline
red-0ne Aug 3, 2024
b33a013
fix: Add missing sender address
red-0ne Aug 3, 2024
b02af2f
fix: Use higher context err variable
red-0ne Aug 3, 2024
7b3555a
Merge remote-tracking branch 'origin/feat/non-custodial-staking' into…
red-0ne Aug 3, 2024
604224d
fix: Enfore sender to be owner or operator
red-0ne Aug 3, 2024
540f5db
Merge branch 'feat/non-custodial-staking' into refactor/non-custodial…
red-0ne Aug 3, 2024
ac07b43
fix: Remove err shadow declaration
red-0ne Aug 3, 2024
79e4c39
Merge remote-tracking branch 'origin/main' into feat/non-custodial-st…
red-0ne Aug 5, 2024
473e047
fix: Err var shadowing
red-0ne Aug 6, 2024
088e675
chore: Log default operator address fallback
red-0ne Aug 6, 2024
428104b
Merge remote-tracking branch 'origin/main' into feat/non-custodial-st…
red-0ne Aug 7, 2024
ef62442
Merge branch 'feat/non-custodial-staking' into refactor/non-custodial…
red-0ne Aug 7, 2024
365ee96
fix: Add missing ctx argument
red-0ne Aug 7, 2024
d096d02
fix: Add polylog as side effect
red-0ne Aug 7, 2024
20e8ed5
fix: Address review changes, mimic Morse behavior
red-0ne Aug 8, 2024
b543120
Merge remote-tracking branch 'origin/main' into feat/non-custodial-st…
red-0ne Aug 8, 2024
fdfd9b9
fix: Duplicate supplierKeeper declaration
red-0ne Aug 8, 2024
4843c21
Merge remote-tracking branch 'origin/feat/non-custodial-staking' into…
red-0ne Aug 8, 2024
095571a
fix: Reward the owner instead of the operator
red-0ne Aug 8, 2024
611f583
Merge remote-tracking branch 'origin/feat/non-custodial-staking' into…
red-0ne Aug 8, 2024
3a7c0ba
fix: Update tests
red-0ne Aug 9, 2024
e34634b
chore: Add test external signer test case
red-0ne Aug 9, 2024
910c85b
fix: Allow owner to change its aaddress
red-0ne Aug 9, 2024
e092a4e
Merge remote-tracking branch 'origin/feat/non-custodial-staking' into…
red-0ne Aug 9, 2024
b3ac63c
fix: Wrong word added
red-0ne Aug 9, 2024
1f26789
fix: Stake auth contidional
red-0ne Aug 9, 2024
0a43775
Merge remote-tracking branch 'origin/feat/non-custodial-staking' into…
red-0ne Aug 9, 2024
85d84ec
fix: Missing owner address in test
red-0ne Aug 9, 2024
391b6db
chore: Address review change requests
red-0ne Aug 9, 2024
98b0446
Merge remote-tracking branch 'origin/main' into feat/non-custodial-st…
red-0ne Aug 9, 2024
7b78f49
Merge branch 'feat/non-custodial-staking' into refactor/non-custodial…
red-0ne Aug 9, 2024
8060f46
chore: Address review change requests
red-0ne Aug 13, 2024
2deda3b
test: Add test assertion
red-0ne Aug 13, 2024
b887a0c
fix: Error sequence
red-0ne Aug 13, 2024
8121719
chore: Remove unused method
red-0ne Aug 13, 2024
11ef9bb
Merge remote-tracking branch 'origin/feat/non-custodial-staking' into…
red-0ne Aug 13, 2024
1bd3182
fix: Rename Errors in tests
red-0ne Aug 13, 2024
e52e1d9
Merge remote-tracking branch 'origin/main' into refactor/non-custodia…
red-0ne Aug 13, 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
4 changes: 2 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -813,7 +813,7 @@ warn_message_local_stress_test: ## Print a warning message when kicking off a lo
@echo "| |"
@echo "| 1. Review the # of suppliers & gateways in 'load-testing/localnet_loadtest_manifest.yaml' |"
@echo "| 2. Update 'localnet_config.yaml' to reflect what you found in (1) |"
@echo "| DEVELOPER_TIP: If you're operating off defaults, you'll likely need to update to 3 |"
@echo "| DEVELOPER_TIP: If you're operating off defaults, you'll likely need to update to 3 |"
@echo "| |"
@echo "| TODO_DOCUMENT(@okdas): Move this into proper documentation w/ clearer explanations |"
@echo "| |"
Expand All @@ -827,7 +827,7 @@ warn_flaky_tests: ## Print a warning message that some unit tests may be flaky
@echo "| |"
@echo "| 1. Our unit / integration tests are far from perfect & some are flaky |"
@echo "| 2. If you ran 'make go_develop_and_test' and a failure occurred, try to run: |"
@echo "| 'make test_all' once or twice more |"
@echo "| 'make test_all' once or twice more |"
@echo "| 3. If the same error persists, isolate it with 'go test -v ./path/to/failing/module |"
@echo "| |"
@echo "+-----------------------------------------------------------------------------------------------+"
Expand Down
2 changes: 1 addition & 1 deletion api/poktroll/application/application.pulsar.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions api/poktroll/application/event.pulsar.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

10 changes: 5 additions & 5 deletions api/poktroll/application/tx.pulsar.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion api/poktroll/gateway/event.pulsar.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion api/poktroll/service/relay.pulsar.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion api/poktroll/service/tx.pulsar.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion api/poktroll/session/query.pulsar.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion api/poktroll/session/session.pulsar.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading
Loading