test_multiple_eth_accounts from golem's concent acceptance tests is failing

[dybi]

to reproduce:
pytest scripts/concent_acceptance_tests/force_payment/test_payment.py::RequestorDoesntPayTestCase::test_multiple_eth_accounts on branch chore-update-concent-acceptance-test-after-sra-update (GM v2.17.0)

[PaweuB]

Pivotal:
https://www.pivotaltracker.com/story/show/162449516

[cameel]

So the problem is that Concent in UC5 allows the provider to submit acceptances for subtasks which should be paid from different Ethereum accounts belonging to the requestor while it should refuse the request? Wasn't it already fixed in #545?

[pawelkisielewicz]

I asked @dybi about it, it was unclear for me as well. In this case there is one provider with two different Ethereum accounts and one requestor. According to expected message in test code Concent should answer ServiceRefused. @cameel I can't see this case in table in #996 and assume that it is expected Concent's behavior

[pawelkisielewicz]

The problem is resolved after add commit 77c0fb41 chore-update-concent-acceptance-test-after-sra-update. The reason of bug was changing provider_ethereum_public_key's in task_to_compute, where this field is made as property instead of change in want_to_compute_task. I added test in Concent for this case as well in #1008