Skip to content

chore(deps): lock file maintenance #459

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

Merged
merged 2 commits into from
Apr 23, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions tests/unit/actor/test_actor_helpers.py
Original file line number Diff line number Diff line change
Expand Up @@ -122,6 +122,7 @@ async def test_abort_actor_run(apify_client_async_patcher: ApifyClientAsyncPatch
# NOTE: The following methods are properly tested using integrations tests.


@pytest.mark.skip(reason='There are issues with log propagation to caplog, see issue #462.')
async def test_metamorph_fails_locally(caplog: pytest.LogCaptureFixture) -> None:
caplog.set_level('WARNING')
async with Actor:
Expand All @@ -132,6 +133,7 @@ async def test_metamorph_fails_locally(caplog: pytest.LogCaptureFixture) -> None
assert 'Actor.metamorph() is only supported when running on the Apify platform.' in caplog.records[0].message


@pytest.mark.skip(reason='There are issues with log propagation to caplog, see issue #462.')
async def test_reboot_fails_locally(caplog: pytest.LogCaptureFixture) -> None:
caplog.set_level('WARNING')
async with Actor:
Expand All @@ -142,6 +144,7 @@ async def test_reboot_fails_locally(caplog: pytest.LogCaptureFixture) -> None:
assert 'Actor.reboot() is only supported when running on the Apify platform.' in caplog.records[0].message


@pytest.mark.skip(reason='There are issues with log propagation to caplog, see issue #462.')
async def test_add_webhook_fails_locally(caplog: pytest.LogCaptureFixture) -> None:
caplog.set_level('WARNING')
async with Actor:
Expand All @@ -154,6 +157,7 @@ async def test_add_webhook_fails_locally(caplog: pytest.LogCaptureFixture) -> No
assert 'Actor.add_webhook() is only supported when running on the Apify platform.' in caplog.records[0].message


@pytest.mark.skip(reason='There are issues with log propagation to caplog, see issue #462.')
async def test_set_status_message_locally(caplog: pytest.LogCaptureFixture) -> None:
caplog.set_level('INFO')
async with Actor:
Expand All @@ -165,6 +169,7 @@ async def test_set_status_message_locally(caplog: pytest.LogCaptureFixture) -> N
assert '[Status message]: test-status-message' in matching_records[0].message


@pytest.mark.skip(reason='There are issues with log propagation to caplog, see issue #462.')
async def test_set_terminal_status_message_locally(caplog: pytest.LogCaptureFixture) -> None:
caplog.set_level('INFO')
async with Actor:
Expand Down
7 changes: 3 additions & 4 deletions tests/unit/actor/test_actor_log.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,15 +2,14 @@

import contextlib
import logging
from typing import TYPE_CHECKING

import pytest

from apify import Actor
from apify.log import logger

if TYPE_CHECKING:
import pytest


@pytest.mark.skip(reason='There are issues with log propagation to caplog, see issue #462.')
async def test_actor_logs_messages_correctly(caplog: pytest.LogCaptureFixture) -> None:
caplog.set_level(logging.DEBUG, logger='apify')

Expand Down
1 change: 1 addition & 0 deletions tests/unit/test_platform_event_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@
from apify._platform_event_manager import PlatformEventManager, SystemInfoEventData


@pytest.mark.skip(reason='There are issues with log propagation to caplog, see issue #462.')
async def test_lifecycle_local(caplog: pytest.LogCaptureFixture) -> None:
caplog.set_level(logging.DEBUG, logger='apify')
config = Configuration.get_global_configuration()
Expand Down
2 changes: 2 additions & 0 deletions tests/unit/test_proxy_configuration.py
Original file line number Diff line number Diff line change
Expand Up @@ -432,6 +432,7 @@ async def test_initialize_with_manual_password(monkeypatch: pytest.MonkeyPatch,
assert proxy_configuration.is_man_in_the_middle is False


@pytest.mark.skip(reason='There are issues with log propagation to caplog, see issue #462.')
async def test_initialize_with_manual_password_different_than_user_one(
monkeypatch: pytest.MonkeyPatch,
caplog: pytest.LogCaptureFixture,
Expand Down Expand Up @@ -488,6 +489,7 @@ async def test_initialize_when_not_connected(monkeypatch: pytest.MonkeyPatch, re
await proxy_configuration.initialize()


@pytest.mark.skip(reason='There are issues with log propagation to caplog, see issue #462.')
async def test_initialize_when_status_page_unavailable(
monkeypatch: pytest.MonkeyPatch,
caplog: pytest.LogCaptureFixture,
Expand Down
6 changes: 3 additions & 3 deletions uv.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading
Loading