dont start tasks twice
This commit is contained in:
@@ -59,10 +59,7 @@ async def install_extension(
|
|||||||
if installed_ext:
|
if installed_ext:
|
||||||
await stop_extension_background_work(ext_info.id)
|
await stop_extension_background_work(ext_info.id)
|
||||||
|
|
||||||
extension = Extension.from_installable_ext(ext_info)
|
return Extension.from_installable_ext(ext_info)
|
||||||
await start_extension_background_work(ext_info.id)
|
|
||||||
|
|
||||||
return extension
|
|
||||||
|
|
||||||
|
|
||||||
async def check_extensions_limit(installed_ext: InstallableExtension | None = None):
|
async def check_extensions_limit(installed_ext: InstallableExtension | None = None):
|
||||||
|
|||||||
@@ -86,7 +86,7 @@ async def test_install_extension_creates_new_extension_and_starts_background_wor
|
|||||||
assert stored is not None
|
assert stored is not None
|
||||||
download_mock.assert_awaited_once()
|
download_mock.assert_awaited_once()
|
||||||
extract_mock.assert_called_once()
|
extract_mock.assert_called_once()
|
||||||
start_mock.assert_awaited_once_with(ext_id)
|
start_mock.assert_not_awaited()
|
||||||
|
|
||||||
|
|
||||||
@pytest.mark.anyio
|
@pytest.mark.anyio
|
||||||
@@ -138,7 +138,7 @@ async def test_install_extension_updates_existing_upgrade_and_preserves_payments
|
|||||||
assert stored.meta.payments == [existing_payment]
|
assert stored.meta.payments == [existing_payment]
|
||||||
extract_mock.assert_called_once()
|
extract_mock.assert_called_once()
|
||||||
stop_mock.assert_awaited_once_with(ext_id)
|
stop_mock.assert_awaited_once_with(ext_id)
|
||||||
start_mock.assert_awaited_once_with(ext_id)
|
start_mock.assert_not_awaited()
|
||||||
|
|
||||||
|
|
||||||
@pytest.mark.anyio
|
@pytest.mark.anyio
|
||||||
|
|||||||
Reference in New Issue
Block a user