-
Notifications
You must be signed in to change notification settings - Fork 18
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
#697 - Automatically claim payouts for previous epochs #698
#697 - Automatically claim payouts for previous epochs #698
Conversation
…uts-for-previous-epochs
…uts-for-previous-epochs
…vious-epochs' of github.com:oceanprotocol/pdr-backend into issue697-predictoor-automatically-claim-payouts-for-previous-epochs
…vious-epochs' of https://github.com/oceanprotocol/pdr-backend into issue697-predictoor-automatically-claim-payouts-for-previous-epochs
…uts-for-previous-epochs
@@ -63,6 +63,21 @@ def test_predictoor_ss_test_dict(): | |||
assert d["aimodel_ss"]["input_feeds"] == ["binance ETH/USDT c 1h"] | |||
|
|||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You don't need a dedicated test for this. It adds more complexity than needed.
Rather, incorporate this into the existing tests surgically.
pdr_backend/ppss/web3_pp.py
Outdated
@@ -336,6 +336,9 @@ def get_current_epoch(self) -> int: | |||
def set_token(self, web3_pp): | |||
pass | |||
|
|||
def s_start_payouts(self) -> int: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks like it's an addition to a mock predictoor_contract object. Not the correct place. If it were to be added, it should be an addition to a mock predictoor_ss object
Fixes #697
Changes proposed in this PR: