Page MenuHomeSoftware Heritage
Differential D7908 Diff 28548 swh/graphql/tests/functional/test_origin_connection.py

Changeset View

Standalone View

View Options

swh/graphql/tests/functional/test_origin_connection.py

# Copyright (C) 2022 The Software Heritage developers # Copyright (C) 2022 The Software Heritage developers
# See the AUTHORS file at the top-level directory of this distribution # See the AUTHORS file at the top-level directory of this distribution
# License: GNU General Public License version 3, or any later version # License: GNU General Public License version 3, or any later version
# See top-level LICENSE file for more information # See top-level LICENSE file for more information
from ..data import get_origins
from .utils import get_query_response from .utils import get_query_response
def test_get(client): def test_get(client, storage):
query_str = """ query_str = """
{ {
origins(first: 10) { origins(first: 10) {
nodes { nodes {
url url
} }
} }
} }
""" """
data, _ = get_query_response(client, query_str) data, _ = get_query_response(client, query_str)
assert len(data["origins"]["nodes"]) == 2 assert len(data["origins"]["nodes"]) == len(get_origins())
def test_get_filter_by_pattern(client): def test_get_filter_by_pattern(client):
query_str = """ query_str = """
{ {
origins(first: 10, urlPattern: "forge1") { origins(first: 10, urlPattern: "somewhere.org/den") {
nodes { nodes {
url url
} }
} }
} }
""" """
data, _ = get_query_response(client, query_str) data, _ = get_query_response(client, query_str)
assert len(data["origins"]["nodes"]) == 1 assert len(data["origins"]["nodes"]) == 1
def test_basic_pagination(client): def test_get_filter_by_non_existing_pattern(client):
query_str = """ query_str = """
{ {
origins(first: 2) { origins(first: 10, urlPattern: "somewhere.org/den/test/") {
nodes { nodes {
id url
}
pageInfo {
hasNextPage
endCursor
} }
} }
} }
""" """
data, _ = get_query_response(client, query_str)
assert len(data["origins"]["nodes"]) == 0
def test_basic_pagination(client):
query_str = f"""
{{
origins(first: {len(get_origins())}) {{
nodes {{
id
}}
pageInfo {{
hasNextPage
endCursor
}}
}}
}}
"""
data, _ = get_query_response(client, query_str) data, _ = get_query_response(client, query_str)
assert len(data["origins"]["nodes"]) == 2 assert len(data["origins"]["nodes"]) == len(get_origins())
ardumontUnsubmitted
Done
Inline Actions
def test_basic_pagination(client):
query_str = """
{
- origins(first: %s) {
+ origins(first: {first_origin}) {
nodes {
id
}
pageInfo {
hasNextPage
endCursor
}
}
}
"""
- data, _ = get_query_response(client, query_str % len(get_origins()))
+ data, _ = get_query_response(client, query_str.format(first_origin=len(get_origins())))
assert len(data["origins"]["nodes"]) == len(get_origins())

I usually prefer using format for pattern like this, that allows to give some more meaning to the value we are replacing in the string.

ardumont: I usually prefer using format for pattern like this, that allows to give some more meaning to…
assert data["origins"]["pageInfo"] == {"hasNextPage": False, "endCursor": None} assert data["origins"]["pageInfo"] == {"hasNextPage": False, "endCursor": None}
About · Developer information · Legal