Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

Task API RequestorMarketStrategy fetch in VerificationMixin #4831

Merged
merged 4 commits into from
Oct 29, 2019

Conversation

mfranciszkiewicz
Copy link
Contributor

  • fetches a RequestorMarketStrategy class from AppDefinition for Task API tasks
  • RequestorMarketStrategy.get_payment_computer no longer requires a Task object argument

@codecov
Copy link

codecov bot commented Oct 25, 2019

Codecov Report

❗ No coverage uploaded for pull request base (mwu/e2e-rtm@8878f27). Click here to learn what that means.
The diff coverage is 90.47%.

@@              Coverage Diff               @@
##             mwu/e2e-rtm    #4831   +/-   ##
==============================================
  Coverage               ?   88.93%           
==============================================
  Files                  ?      229           
  Lines                  ?    21026           
  Branches               ?        0           
==============================================
  Hits                   ?    18699           
  Misses                 ?     2327           
  Partials               ?        0

tests/golem/task/server/test_verification.py Outdated Show resolved Hide resolved
tests/golem/task/server/test_verification.py Outdated Show resolved Hide resolved
tests/golem/task/server/test_verification.py Outdated Show resolved Hide resolved
@mfranciszkiewicz mfranciszkiewicz merged commit 9f13aaf into mwu/e2e-rtm Oct 29, 2019
@mfranciszkiewicz mfranciszkiewicz deleted the task_api_market branch October 29, 2019 15:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants