Skip to content

Commit

Permalink
chore(deps): update dependency ops to v2.17.0 (#71)
Browse files Browse the repository at this point in the history
* chore(deps): update dependency ops to v2.17.0

* fix linting

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Arturo Seijas <arturo.seijas@canonical.com>
  • Loading branch information
renovate[bot] and arturo-seijas authored Oct 1, 2024
1 parent 15d5cfa commit 6083f43
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion requirements.txt
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
cryptography==43.0.1
jsonschema==4.23.0
lightkube==0.15.3
ops==2.16.1
ops==2.17.0
pydantic==2.9.2
rpds-py==0.18.1
2 changes: 1 addition & 1 deletion tests/integration/helper.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ def __init__(
# Ignore pylint rule as this is the parent method signature
def send(
self, request, stream=False, timeout=None, verify=True, cert=None, proxies=None
): # pylint: disable=too-many-arguments
): # pylint: disable=too-many-arguments, too-many-positional-arguments
"""Wrap HTTPAdapter send to modify the outbound request.
Args:
Expand Down
8 changes: 4 additions & 4 deletions tests/unit/test_charm.py
Original file line number Diff line number Diff line change
Expand Up @@ -82,14 +82,14 @@ def test_deploy_lightkube_error(
pytest.param(404, id="not found."),
],
)
def test_reconcile_api_error_4xx( # pylint: disable=too-many-arguments
def test_reconcile_api_error_4xx(
harness: Harness,
client_with_mock_external: MagicMock,
certificates_relation_data: dict[str, str],
monkeypatch: pytest.MonkeyPatch,
error_code: int,
config: dict[str, str],
):
): # pylint: disable=too-many-arguments, too-many-positional-arguments
"""
arrange: Given a charm with valid tls/gateway integration and mocked client returning 4xx.
act: Update the charm with valid config.
Expand Down Expand Up @@ -147,14 +147,14 @@ def test_reconcile_api_error_forbidden(


@pytest.mark.usefixtures("client_with_mock_external")
def test_create_http_route_insufficient_permission( # pylint: disable=too-many-arguments
def test_create_http_route_insufficient_permission(
harness: Harness,
certificates_relation_data: dict[str, str],
gateway_relation_application_data: dict[str, str],
gateway_relation_unit_data: dict[str, str],
monkeypatch: pytest.MonkeyPatch,
config: dict[str, str],
):
): # pylint: disable=too-many-arguments, too-many-positional-arguments
"""
arrange: Given a charm with valid tls/gateway integration and mocked
http_route resource manager returning 403 error.
Expand Down
2 changes: 1 addition & 1 deletion tests/unit/test_gateway.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@


@pytest.mark.usefixtures("client_with_mock_external")
def test_create_gateway( # pylint: disable=too-many-arguments
def test_create_gateway( # pylint: disable=too-many-arguments, too-many-positional-arguments
harness: Harness,
certificates_relation_data: dict[str, str],
gateway_relation_application_data: dict[str, str],
Expand Down

0 comments on commit 6083f43

Please sign in to comment.