Overview

Request 798900 accepted

- Add patch to switch from external mock to unittest.mock
+ o_switch_to_unitest_mock.patch


Tomáš Chvátal's avatar

Why should we switch to git version? Why it is not based on _service? the Version field even with service seems wrong it is not 3.10 Also what I see the git version does not need mock module dependency anymore, so how about you just backport that commit only?


John Paul Adrian Glaubitz's avatar
author source maintainer target maintainer

I switched to a git version because upstream hasn't released new version for a longer time and the updated version fixes the testsuite. I had a brief stab at backporting the mock fix but the patch didn't apply cleanly at all.

I didn't use a service file because I considered this a one-time exception so that we can get the Azure submission fixed.

You're right about the mock dependency in any case, I will remove it.

Suggestions for addressing the testsuite failure? Not sure whether upstream wants to release anytime soon.


Tomáš Chvátal's avatar

The test failure on mock is just because of syntax change. If you just go in the release folder and replace all 'from mock import ..'. or 'import mock' with 'from unittest import mock' you are set without having to generate git version; and if you really think we should have git version you should ask upstream first to release and if that fails use the _service to auto-generate it. The mock module is not ment to be used except as a feature backport to older pythons -> even on your azure stuff you should be able to not depend on the latest mock ; it seems upstream in git just auto bumped the requirement as it was released.


John Paul Adrian Glaubitz's avatar
author source maintainer target maintainer

Resubmitted with a simple switch from mock to unittest.mock which worked.

Thanks!


John Paul Adrian Glaubitz's avatar
author source maintainer target maintainer

I'll try to replace "import mock" with "from unittest import mock" globally and see if that helps.

Request History
John Paul Adrian Glaubitz's avatar

glaubitz created request

- Add patch to switch from external mock to unittest.mock
+ o_switch_to_unitest_mock.patch


Tomáš Chvátal's avatar

scarabeus_iv accepted request

Looks good, fwding

openSUSE Build Service is sponsored by