club_profile_url and club_manager_url_via_login()

CHANGELOG.md

@@ -8,12 +8,19 @@ and this project tries to adhere to [Semantic Versioning](https://semver.org/spe
 Historic and pre-release versions aren't necessarily included.
 
 
+## [UNRELEASED] - TBC
+
+### Added
+
+- `club_profile_url` and `club_manager_url_via_login()` 
+
+
 ## [0.8.1] - 2023-12-19
 
 ### Fixed
 
 - `get_manager_member_counts()` still returned `["pending"]` instead of
-  `["new"]`.
+  `["new"]`
 
 
 ## [0.8.0] - 2023-12-19
@@ -22,12 +29,11 @@ Historic and pre-release versions aren't necessarily included.
 
 - **BREAKING CHANGES**: Functions renamed to `get_profile_info()` and 
   `get_manager_member_counts()`. `get_manager_member_counts()` returns `["new"]`
-  instead of `["pending"]`.
+  instead of `["pending"]`
 
 - Update dev/test dependencies: isort, mypy; CI dependencies actions/checkout, 
   actions/setup-python
 
-
 ### Added
 
 - `get_profile_info()`: raise exception on redirect; better error messages; basic 
@@ -57,7 +63,8 @@ Historic and pre-release versions aren't necessarily included.
 
 ### Changed
 
-- `get_private_member_counts()`: raise exception if zero 'active members' would be returned.
+- `get_private_member_counts()`: raise exception if zero 'active members' would be 
+  returned
 
 - Update dev/test dependencies: mypy, ruff
 

README.md

@@ -39,8 +39,15 @@ See also https://playwright.dev/python/docs/browsers#install-system-dependencies
 
 ## Usage
 
+### Club profile URL
 
-### Get info from a club's profile page
+```
+britishcycling_clubs.club_profile_url(
+    club_id: str
+) -> str
+```
+
+### Get info from a club's profile
 
 ```
 britishcycling_clubs.get_profile_info(
@@ -49,17 +56,27 @@ britishcycling_clubs.get_profile_info(
 ```
 Return information from the club's public profile page; doesn't require login.
 
-Specifically, returns these values:
+Specifically, return a dict with these keys and corresponding values:
 
-- Club name
-- Total club members
+- `"club_name"`: Club name
+- `"total_members"`: Total club members
 
 Example script `example_profile_info.py` loads club ID from `config.ini` (you'll
-need to copy `config_dist.ini`, populate club ID only and rename).  It 
-then retrieves and prints the club name and total member count.
+need to copy `config_dist.ini`, populate club ID only and rename).
+It then retrieves and prints the club name and total member count.
+
+
+### Club manager URL (via login)
 
+```
+britishcycling_clubs.club_manager_url_via_login(
+    club_id: str
+) -> str
+```
+URL which redirects to Club Manager URL, via login if needed.
 
-### Get member counts from a club's Club Manager pages
+
+### Get member counts from Club Manager
 
 ```
 britishcycling_clubs.get_manager_member_counts(
@@ -71,17 +88,17 @@ britishcycling_clubs.get_manager_member_counts(
 ```
 Get numbers of active, new, expired members from the club manager page.
 
-Specifically, returns the counts from these tabs:
+Specifically, return a dict with these keys, and values from badges on corresponding
+tabs:
 
-- Active Club Members
-- New Club Subscriptions
-- Expired Club Members
+- `"active"`: Active Club Members
+- `"expired"`: Expired Club Members
+- `"new"`: New Club Subscriptions
 
 This takes about 10s.
 
 Example script `example_manager_member_counts.py` loads club ID and credentials from
 `config.ini` (you'll need to copy `config_dist.ini`, populate and rename to 
-`config.ini`). It then retrieves and prints the number of active, expired and new 
+`config.ini`).
+It then retrieves and prints the number of active, expired and new 
 club member counts from the club's Club Manager pages. 
-
-

britishcycling_clubs/__init__.py

@@ -1,6 +1,10 @@
 """Module with functions to retrieve information about a club."""
 
+from britishcycling_clubs.manager import (
+    club_manager_url_via_login as club_manager_url_via_login,
+)
 from britishcycling_clubs.manager import (
     get_manager_member_counts as get_manager_member_counts,
 )
+from britishcycling_clubs.profile import club_profile_url as club_profile_url
 from britishcycling_clubs.profile import get_profile_info as get_profile_info

britishcycling_clubs/profile.py

@@ -26,9 +26,11 @@ def get_profile_info(club_id: str) -> ProfileInfo:
 
     Returns
     -------
-    _ProfileInfo
+    dict[str, str | int]
+        keys: 'club_name', 'total_members'
+        values: corresponding str or int
     """
-    profile_url = f"{_PROFILE_BASE_URL}{club_id}/"
+    profile_url = club_profile_url(club_id)
     r = requests.get(profile_url, timeout=_REQUESTS_TIMEOUT)
     r.raise_for_status()
     if r.url != profile_url:
@@ -41,6 +43,17 @@ def get_profile_info(club_id: str) -> ProfileInfo:
     }
 
 
+def club_profile_url(club_id: str) -> str:
+    """Return URL of club's profile page.
+
+    Parameters
+    ----------
+    club_id
+        From the URL used to access club pages.
+    """
+    return f"{_PROFILE_BASE_URL}{club_id}/"
+
+
 def _club_name_from_profile(soup: BeautifulSoup) -> str:
     """Return the club's name from profile page soup."""
     club_name_h1 = soup.find("h1", class_="article__header__title-body__text")

tests/test_manager.py

@@ -1,7 +1,18 @@
 """Tests for 'manager' functions."""
 import pytest
 
-from britishcycling_clubs.manager import _process_manager_member_counts
+from britishcycling_clubs.manager import (
+    _process_manager_member_counts,
+    club_manager_url_via_login,
+)
+
+
+def test_club_manager_url_via_login__happy_path() -> None:
+    """Test that correct URL is returned."""
+    assert (
+        club_manager_url_via_login("000")
+        == "https://www.britishcycling.org.uk/uac/connect?success_url=/dashboard/club/membership?club_id=000/"
+    )
 
 
 def test__process_manager_member_counts__happy_path() -> None:

tests/test_profile.py

@@ -4,8 +4,17 @@
 from britishcycling_clubs.profile import (
     _club_name_from_profile,
     _total_members_from_profile,
+    club_profile_url,
 )
 
+
+def test_club_profile_url__happy_path() -> None:
+    """Test that correct URL is returned."""
+    assert (
+        club_profile_url("000") == "https://www.britishcycling.org.uk/club/profile/000/"
+    )
+
+
 # Partial extract from actual page
 PROFILE_PAGE_EXTRACT = """
 <html>