Add network_via_web_auth helper

.flake8

@@ -0,0 +1,2 @@
+[flake8]
+max_line_length = 88

.github/release-drafter.yml

@@ -22,11 +22,6 @@ categories:
 exclude-labels:
   - "changelog: skip"
 
-autolabeler:
-  - label: "changelog: skip"
-    branch:
-      - "/pre-commit-ci-update-config/"
-
 template: |
   $CHANGES
 

.github/renovate.json

@@ -1,13 +1,16 @@
 {
-  "$schema": "https://docs.renovatebot.com/renovate-schema.json",
+  "extends": [
-  "extends": ["config:base"],
+    "config:base"
-  "labels": ["changelog: skip", "dependencies"],
+  ],
+  "labels": [
+    "changelog: skip",
+    "dependencies"
+  ],
   "packageRules": [
     {
       "groupName": "github-actions",
       "matchManagers": ["github-actions"],
       "separateMajorMinor": "false"
     }
-  ],
+  ]
-  "schedule": ["on the first day of the month"]
 }

.github/workflows/deploy.yml

@@ -2,74 +2,50 @@ name: Deploy
 
 on:
   push:
-    branches: [main]
+    branches:
-    tags: ["*"]
+      - main
-  pull_request:
-    branches: [main]
   release:
     types:
       - published
   workflow_dispatch:
 
-permissions:
-  contents: read
-
 jobs:
-  # Always build & lint package.
+  deploy:
-  build-package:
+    if: github.repository_owner == 'pylast'
-    name: Build & verify package
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
         with:
           fetch-depth: 0
 
-      - uses: hynek/build-and-inspect-python-package@v2
+      - name: Set up Python
-
+        uses: actions/setup-python@v4
-  # Upload to Test PyPI on every commit on main.
-  release-test-pypi:
-    name: Publish in-dev package to test.pypi.org
-    if: |
-      github.repository_owner == 'pylast'
-      && github.event_name == 'push'
-      && github.ref == 'refs/heads/main'
-    runs-on: ubuntu-latest
-    needs: build-package
-
-    permissions:
-      id-token: write
-
-    steps:
-      - name: Download packages built by build-and-inspect-python-package
-        uses: actions/download-artifact@v4
         with:
-          name: Packages
+          python-version: "3.x"
-          path: dist
+          cache: pip
+          cache-dependency-path: pyproject.toml
 
-      - name: Upload package to Test PyPI
+      - name: Install dependencies
+        run: |
+          python -m pip install -U pip
+          python -m pip install -U build twine wheel
+
+      - name: Build package
+        run: |
+          python -m build
+          twine check --strict dist/*
+
+      - name: Publish package to PyPI
+        if: github.event.action == 'published'
         uses: pypa/gh-action-pypi-publish@release/v1
         with:
-          repository-url: https://test.pypi.org/legacy/
+          user: __token__
+          password: ${{ secrets.pypi_password }}
 
-  # Upload to real PyPI on GitHub Releases.
+      - name: Publish package to TestPyPI
-  release-pypi:
-    name: Publish released package to pypi.org
-    if: |
-      github.repository_owner == 'pylast'
-      && github.event.action == 'published'
-    runs-on: ubuntu-latest
-    needs: build-package
-
-    permissions:
-      id-token: write
-
-    steps:
-      - name: Download packages built by build-and-inspect-python-package
-        uses: actions/download-artifact@v4
-        with:
-          name: Packages
-          path: dist
-
-      - name: Upload package to PyPI
         uses: pypa/gh-action-pypi-publish@release/v1
+        with:
+          user: __token__
+          password: ${{ secrets.test_pypi_password }}
+          repository_url: https://test.pypi.org/legacy/

.github/workflows/labels.yml

@@ -1,8 +1,5 @@
 name: Sync labels
 
-permissions:
-  pull-requests: write
-
 on:
   push:
     branches:
@@ -15,7 +12,7 @@ jobs:
   sync:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - uses: micnncim/action-label-syncer@v1
         with:
           prune: false

.github/workflows/lint.yml

@@ -2,21 +2,13 @@ name: Lint
 
 on: [push, pull_request, workflow_dispatch]
 
-env:
-  FORCE_COLOR: 1
-  PIP_DISABLE_PIP_VERSION_CHECK: 1
-
-permissions:
-  contents: read
-
 jobs:
   lint:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
-      - uses: actions/setup-python@v5
+      - uses: actions/setup-python@v4
         with:
           python-version: "3.x"
-          cache: pip
+      - uses: pre-commit/action@v3.0.0
-      - uses: pre-commit/action@v3.0.1

.github/workflows/release-drafter.yml

@@ -5,30 +5,14 @@ on:
     # branches to consider in the event; optional, defaults to all
     branches:
       - main
-  # pull_request event is required only for autolabeler
-  pull_request:
-    # Only following types are handled by the action, but one can default to all as well
-    types: [opened, reopened, synchronize]
-  # pull_request_target event is required for autolabeler to support PRs from forks
-  # pull_request_target:
-  #   types: [opened, reopened, synchronize]
   workflow_dispatch:
 
-permissions:
-  contents: read
-
 jobs:
   update_release_draft:
     if: github.repository_owner == 'pylast'
-    permissions:
-      # write permission is required to create a GitHub Release
-      contents: write
-      # write permission is required for autolabeler
-      # otherwise, read permission is required at least
-      pull-requests: write
     runs-on: ubuntu-latest
     steps:
       # Drafts your next release notes as pull requests are merged into "main"
-      - uses: release-drafter/release-drafter@v6
+      - uses: release-drafter/release-drafter@v5
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/require-pr-label.yml

@@ -8,12 +8,8 @@ jobs:
   label:
     runs-on: ubuntu-latest
 
-    permissions:
-      issues: write
-      pull-requests: write
-
     steps:
-      - uses: mheap/github-action-required-labels@v5
+      - uses: mheap/github-action-required-labels@v3
         with:
           mode: minimum
           count: 1

.github/workflows/test.yml

@@ -11,18 +11,18 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        python-version: ["pypy3.10", "3.8", "3.9", "3.10", "3.11", "3.12", "3.13"]
+        python-version: ["pypy-3.8", "3.7", "3.8", "3.9", "3.10", "3.11", "3.12-dev"]
         os: [ubuntu-latest]
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
 
       - name: Set up Python ${{ matrix.python-version }}
-        uses: actions/setup-python@v5
+        uses: actions/setup-python@v4
         with:
           python-version: ${{ matrix.python-version }}
-          allow-prereleases: true
           cache: pip
+          cache-dependency-path: pyproject.toml
 
       - name: Install dependencies
         run: |
@@ -40,7 +40,7 @@ jobs:
           PYLAST_USERNAME: ${{ secrets.PYLAST_USERNAME }}
 
       - name: Upload coverage
-        uses: codecov/codecov-action@v3.1.5
+        uses: codecov/codecov-action@v3
         with:
           flags: ${{ matrix.os }}
           name: ${{ matrix.os }} Python ${{ matrix.python-version }}

.pre-commit-config.yaml

@@ -1,74 +1,63 @@
 repos:
-  - repo: https://github.com/astral-sh/ruff-pre-commit
+  - repo: https://github.com/asottile/pyupgrade
-    rev: v0.5.0
+    rev: v3.1.0
     hooks:
-      - id: ruff
+      - id: pyupgrade
-        args: [--exit-non-zero-on-fix]
+        args: [--py37-plus]
 
-  - repo: https://github.com/psf/black-pre-commit-mirror
+  - repo: https://github.com/psf/black
-    rev: 24.4.2
+    rev: 22.10.0
     hooks:
       - id: black
+        args: [--target-version=py37]
 
   - repo: https://github.com/asottile/blacken-docs
-    rev: 1.18.0
+    rev: v1.12.1
     hooks:
       - id: blacken-docs
-        args: [--target-version=py38]
+        args: [--target-version=py37]
-        additional_dependencies: [black]
+        additional_dependencies: [black==22.10.0]
+
+  - repo: https://github.com/PyCQA/isort
+    rev: 5.10.1
+    hooks:
+      - id: isort
+
+  - repo: https://github.com/PyCQA/flake8
+    rev: 5.0.4
+    hooks:
+      - id: flake8
+        additional_dependencies: [flake8-2020, flake8-implicit-str-concat]
+
+  - repo: https://github.com/pre-commit/pygrep-hooks
+    rev: v1.9.0
+    hooks:
+      - id: python-check-blanket-noqa
 
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.6.0
+    rev: v4.3.0
     hooks:
-      - id: check-added-large-files
-      - id: check-case-conflict
-      - id: check-merge-conflict
       - id: check-json
+      - id: check-merge-conflict
       - id: check-toml
       - id: check-yaml
-      - id: debug-statements
       - id: end-of-file-fixer
-      - id: forbid-submodules
+      - id: requirements-txt-fixer
-      - id: trailing-whitespace
-        exclude: .github/(ISSUE_TEMPLATE|PULL_REQUEST_TEMPLATE).md
-
-  - repo: https://github.com/python-jsonschema/check-jsonschema
-    rev: 0.28.6
-    hooks:
-      - id: check-github-workflows
-      - id: check-renovate
-
-  - repo: https://github.com/rhysd/actionlint
-    rev: v1.7.1
-    hooks:
-      - id: actionlint
 
   - repo: https://github.com/tox-dev/pyproject-fmt
-    rev: 2.1.3
+    rev: 0.3.5
     hooks:
       - id: pyproject-fmt
 
   - repo: https://github.com/abravalheri/validate-pyproject
-    rev: v0.18
+    rev: v0.10.1
     hooks:
       - id: validate-pyproject
 
   - repo: https://github.com/tox-dev/tox-ini-fmt
-    rev: 1.3.1
+    rev: 0.5.2
     hooks:
       - id: tox-ini-fmt
 
-  - repo: https://github.com/pre-commit/mirrors-prettier
-    rev: v4.0.0-alpha.8
-    hooks:
-      - id: prettier
-        args: [--prose-wrap=always, --print-width=88]
-        exclude: .github/(ISSUE_TEMPLATE|PULL_REQUEST_TEMPLATE).md
-
-  - repo: meta
-    hooks:
-      - id: check-hooks-apply
-      - id: check-useless-excludes
-
 ci:
   autoupdate_schedule: quarterly

.scrutinizer.yml

@@ -0,0 +1,9 @@
+checks:
+    python:
+        code_rating: true
+        duplicate_code: true
+filter:
+    excluded_paths:
+        - '*/test/*'
+tools:
+    external_code_coverage: true

CHANGELOG.md

@@ -12,125 +12,117 @@ See GitHub Releases:
 
 ## Changed
 
-- Fix unsafe creation of temp file for caching, and improve exception raising (#356)
+* Fix unsafe creation of temp file for caching, and improve exception raising (#356) @kvanzuijlen
-  @kvanzuijlen
+* [pre-commit.ci] pre-commit autoupdate (#362) @pre-commit-ci
-- [pre-commit.ci] pre-commit autoupdate (#362) @pre-commit-ci
+
 
 ## [4.1.0] - 2021-01-04
-
 ## Added
 
-- Add support for streaming (#336) @kvanzuijlen
+* Add support for streaming (#336) @kvanzuijlen
-- Add Python 3.9 final to Travis CI (#350) @sheetalsingala
+* Add Python 3.9 final to Travis CI (#350) @sheetalsingala
 
 ## Changed
 
-- Update copyright year (#360) @hugovk
+* Update copyright year (#360) @hugovk
-- Replace Travis CI with GitHub Actions (#352) @hugovk
+* Replace Travis CI with GitHub Actions (#352) @hugovk
-- [pre-commit.ci] pre-commit autoupdate (#359) @pre-commit-ci
+* [pre-commit.ci] pre-commit autoupdate (#359) @pre-commit-ci
 
 ## Fixed
 
-- Set limit to 50 by default, not 1 (#355) @hugovk
+* Set limit to 50 by default, not 1 (#355) @hugovk
+
 
 ## [4.0.0] - 2020-10-07
-
 ## Added
 
-- Add support for Python 3.9 (#347) @hugovk
+* Add support for Python 3.9 (#347) @hugovk
 
 ## Removed
 
-- Remove deprecated `Artist.get_cover_image`, `User.get_artist_tracks` and
+* Remove deprecated `Artist.get_cover_image`, `User.get_artist_tracks` and `STATUS_TOKEN_ERROR` (#348) @hugovk
-  `STATUS_TOKEN_ERROR` (#348) @hugovk
+* Drop support for EOL Python 3.5 (#346) @hugovk
-- Drop support for EOL Python 3.5 (#346) @hugovk
+
 
 ## [3.3.0] - 2020-06-25
-
 ### Added
 
-- `User.get_now_playing`: Add album and cover image to info (#330) @hugovk
+* `User.get_now_playing`: Add album and cover image to info (#330) @hugovk
 
 ### Changed
 
-- Improve handling of error responses from the API (#327) @spiritualized
+* Improve handling of error responses from the API (#327) @spiritualized
 
 ### Deprecated
 
-- Deprecate `Artist.get_cover_image`, they're no longer available from Last.fm (#332)
+* Deprecate `Artist.get_cover_image`, they're no longer available from Last.fm (#332) @hugovk
-  @hugovk
 
 ### Fixed
 
-- Fix `artist.get_bio_content()` to return `None` if bio is empty (#326) @hugovk
+* Fix `artist.get_bio_content()` to return `None` if bio is empty (#326) @hugovk
+
 
 ## [3.2.1] - 2020-03-05
-
 ### Fixed
 
-- Only Python 3 is supported: don't create universal wheel (#318) @hugovk
+* Only Python 3 is supported: don't create universal wheel (#318) @hugovk
-- Fix regression calling `get_recent_tracks` with `limit=None` (#320) @hugovk
+* Fix regression calling `get_recent_tracks` with `limit=None` (#320) @hugovk
-- Fix `DeprecationWarning`: Please use `assertRegex` instead (#323) @hugovk
+* Fix `DeprecationWarning`: Please use `assertRegex` instead (#323) @hugovk
 
 ## [3.2.0] - 2020-01-03
-
 ### Added
 
-- Support for Python 3.8
+* Support for Python 3.8
-- Store album art URLs when you call `GetTopAlbums` ([#307])
+* Store album art URLs when you call `GetTopAlbums` ([#307])
-- Retry paging through results on exception ([#297])
+* Retry paging through results on exception ([#297])
-- More error status codes from https://last.fm/api/errorcodes ([#297])
+* More error status codes from https://last.fm/api/errorcodes ([#297])
 
 ### Changed
 
-- Respect `get_recent_tracks`' limit when there's a now playing track ([#310])
+* Respect `get_recent_tracks`' limit when there's a now playing track ([#310])
-- Move installable code to `src/` ([#301])
+* Move installable code to `src/` ([#301])
-- Update `get_weekly_artist_charts` docstring: only for `User` ([#311])
+* Update `get_weekly_artist_charts` docstring: only for `User` ([#311])
-- Remove Python 2 warnings, `python_requires` should be enough ([#312])
+* Remove Python 2 warnings, `python_requires` should be enough ([#312])
-- Use setuptools_scm to simplify versioning during release ([#316])
+* Use setuptools_scm to simplify versioning during release ([#316])
-- Various lint and test updates
+* Various lint and test updates
 
 ### Deprecated
 
-- Last.fm's `user.getArtistTracks` has now been deprecated by Last.fm and is no longer
+* Last.fm's `user.getArtistTracks` has now been deprecated by Last.fm and is no longer
   available. Last.fm returns a "Deprecated - This type of request is no longer
   supported" error when calling it. A future version of pylast will remove its
   `User.get_artist_tracks` altogether. ([#305])
 
-- `STATUS_TOKEN_ERROR` is deprecated and will be removed in a future version. Use
+* `STATUS_TOKEN_ERROR` is deprecated and will be removed in a future version.
-  `STATUS_OPERATION_FAILED` instead.
+  Use `STATUS_OPERATION_FAILED` instead.
 
 ## [3.1.0] - 2019-03-07
-
 ### Added
 
-- Extract username from session via new
+* Extract username from session via new
   `SessionKeyGenerator.get_web_auth_session_key_username` ([#290])
-- `User.get_track_scrobbles` ([#298])
+* `User.get_track_scrobbles` ([#298])
 
 ### Deprecated
 
-- `User.get_artist_tracks`. Use `User.get_track_scrobbles` as a partial replacement.
+*  `User.get_artist_tracks`. Use `User.get_track_scrobbles` as a partial replacement.
-  ([#298])
+   ([#298])
 
 ## [3.0.0] - 2019-01-01
-
 ### Added
-
+* This changelog file ([#273])
-- This changelog file ([#273])
 
 ### Removed
 
-- Support for Python 2.7 ([#265])
+* Support for Python 2.7 ([#265])
 
-- Constants `COVER_SMALL`, `COVER_MEDIUM`, `COVER_LARGE`, `COVER_EXTRA_LARGE` and
+* Constants `COVER_SMALL`, `COVER_MEDIUM`, `COVER_LARGE`, `COVER_EXTRA_LARGE`
-  `COVER_MEGA`. Use `SIZE_SMALL` etc. instead. ([#282])
+  and `COVER_MEGA`. Use `SIZE_SMALL` etc. instead. ([#282])
 
 ## [2.4.0] - 2018-08-08
-
 ### Deprecated
 
-- Support for Python 2.7 ([#265])
+* Support for Python 2.7 ([#265])
 
 [4.2.0]: https://github.com/pylast/pylast/compare/4.1.0...4.2.0
 [4.1.0]: https://github.com/pylast/pylast/compare/4.0.0...4.1.0

COPYING

@@ -32,11 +32,11 @@ TERMS AND CONDITIONS FOR USE, REPRODUCTION, AND DISTRIBUTION
 
 4. Redistribution. You may reproduce and distribute copies of the Work or Derivative Works thereof in any medium, with or without modifications, and in Source or Object form, provided that You meet the following conditions:
 
-You must give any other recipients of the Work or Derivative Works a copy of this License; and
+You must give any other recipients of the Work or Derivative Works a copy of this License; and 
 
-You must cause any modified files to carry prominent notices stating that You changed the files; and
+You must cause any modified files to carry prominent notices stating that You changed the files; and 
 
-You must retain, in the Source form of any Derivative Works that You distribute, all copyright, patent, trademark, and attribution notices from the Source form of the Work, excluding those notices that do not pertain to any part of the Derivative Works; and
+You must retain, in the Source form of any Derivative Works that You distribute, all copyright, patent, trademark, and attribution notices from the Source form of the Work, excluding those notices that do not pertain to any part of the Derivative Works; and 
 
 If the Work includes a "NOTICE" text file as part of its distribution, then any Derivative Works that You distribute must include a readable copy of the attribution notices contained within such NOTICE file, excluding those notices that do not pertain to any part of the Derivative Works, in at least one of the following places: within a NOTICE text file distributed as part of the Derivative Works; within the Source form or documentation, if provided along with the Derivative Works; or, within a display generated by the Derivative Works, if and wherever such third-party notices normally appear. The contents of the NOTICE file are for informational purposes only and do not modify the License. You may add Your own attribution notices within Derivative Works that You distribute, alongside or as an addendum to the NOTICE text from the Work, provided that such additional attribution notices cannot be construed as modifying the License.
 You may add Your own copyright statement to Your modifications and may provide additional or different license terms and conditions for use, reproduction, or distribution of Your modifications, or for any such Derivative Works as a whole, provided Your use, reproduction, and distribution of the Work otherwise complies with the conditions stated in this License.

README.md

@@ -15,44 +15,49 @@ Use the pydoc utility for help on usage or see [tests/](tests/) for examples.
 
 ## Installation
 
+Install via pip:
+
+```sh
+python3 -m pip install pylast
+```
+
 Install latest development version:
 
 ```sh
-python3 -m pip install -U git+https://git.hirad.it/Hirad/pylast
+python3 -m pip install -U git+https://github.com/pylast/pylast
 ```
 
 Or from requirements.txt:
 
 ```txt
--e https://git.hirad.it/Hirad/pylast#egg=pylast
+-e https://github.com/pylast/pylast.git#egg=pylast
 ```
 
 Note:
 
-- pyLast 5.3+ supports Python 3.8-3.13.
+* pyLast 5.1+ supports Python 3.7-3.11.
-- pyLast 5.2+ supports Python 3.8-3.12.
+* pyLast 5.0+ supports Python 3.7-3.10.
-- pyLast 5.1 supports Python 3.7-3.11.
+* pyLast 4.3+ supports Python 3.6-3.10.
-- pyLast 5.0 supports Python 3.7-3.10.
+* pyLast 4.0 - 4.2 supports Python 3.6-3.9.
-- pyLast 4.3 - 4.5 supports Python 3.6-3.10.
+* pyLast 3.2 - 3.3 supports Python 3.5-3.8.
-- pyLast 4.0 - 4.2 supports Python 3.6-3.9.
+* pyLast 3.0 - 3.1 supports Python 3.5-3.7.
-- pyLast 3.2 - 3.3 supports Python 3.5-3.8.
+* pyLast 2.2 - 2.4 supports Python 2.7.10+, 3.4-3.7.
-- pyLast 3.0 - 3.1 supports Python 3.5-3.7.
+* pyLast 2.0 - 2.1 supports Python 2.7.10+, 3.4-3.6.
-- pyLast 2.2 - 2.4 supports Python 2.7.10+, 3.4-3.7.
+* pyLast 1.7 - 1.9 supports Python 2.7, 3.3-3.6.
-- pyLast 2.0 - 2.1 supports Python 2.7.10+, 3.4-3.6.
+* pyLast 1.0 - 1.6 supports Python 2.7, 3.3-3.4.
-- pyLast 1.7 - 1.9 supports Python 2.7, 3.3-3.6.
+* pyLast 0.5 supports Python 2, 3.
-- pyLast 1.0 - 1.6 supports Python 2.7, 3.3-3.4.
+* pyLast < 0.5 supports Python 2.
-- pyLast 0.5 supports Python 2, 3.
-- pyLast < 0.5 supports Python 2.
 
 ## Features
 
-- Simple public interface.
+ * Simple public interface.
-- Access to all the data exposed by the Last.fm web services.
+ * Access to all the data exposed by the Last.fm web services.
-- Scrobbling support.
+ * Scrobbling support.
-- Full object-oriented design.
+ * Full object-oriented design.
-- Proxy support.
+ * Proxy support.
-- Internal caching support for some web services calls (disabled by default).
+ * Internal caching support for some web services calls (disabled by default).
-- Support for other API-compatible networks like Libre.fm.
+ * Support for other API-compatible networks like Libre.fm.
+
 
 ## Getting started
 
@@ -69,53 +74,11 @@ import pylast
 API_KEY = "b25b959554ed76058ac220b7b2e0a026"  # this is a sample key
 API_SECRET = "425b55975eed76058ac220b7b4e8a054"
 
-# In order to perform a write operation you need to authenticate yourself
+network = pylast.network_via_web_auth(
-username = "your_user_name"
-password_hash = pylast.md5("your_password")
-
-network = pylast.LastFMNetwork(
     api_key=API_KEY,
     api_secret=API_SECRET,
-    username=username,
-    password_hash=password_hash,
 )
-```
 
-Alternatively, instead of creating `network` with a username and password, you can
-authenticate with a session key:
-
-```python
-import pylast
-
-SESSION_KEY_FILE = os.path.join(os.path.expanduser("~"), ".session_key")
-network = pylast.LastFMNetwork(API_KEY, API_SECRET)
-if not os.path.exists(SESSION_KEY_FILE):
-    skg = pylast.SessionKeyGenerator(network)
-    url = skg.get_web_auth_url()
-
-    print(f"Please authorize this script to access your account: {url}\n")
-    import time
-    import webbrowser
-
-    webbrowser.open(url)
-
-    while True:
-        try:
-            session_key = skg.get_web_auth_session_key(url)
-            with open(SESSION_KEY_FILE, "w") as f:
-                f.write(session_key)
-            break
-        except pylast.WSError:
-            time.sleep(1)
-else:
-    session_key = open(SESSION_KEY_FILE).read()
-
-network.session_key = session_key
-```
-
-And away we go:
-
-```python
 # Now you can use that object everywhere
 track = network.get_track("Iron Maiden", "The Nomad")
 track.love()
@@ -136,9 +99,8 @@ integration and unit tests with Last.fm, and plenty of code examples.
 
 For integration tests you need a test account at Last.fm that will become cluttered with
 test data, and an API key and secret. Either copy
-[example_test_pylast.yaml](https://github.com/pylast/pylast/blob/main/example_test_pylast.yaml)
+[example_test_pylast.yaml](example_test_pylast.yaml) to test_pylast.yaml and fill out
-to test_pylast.yaml and fill out the credentials, or set them as environment variables
+the credentials, or set them as environment variables like:
-like:
 
 ```sh
 export PYLAST_USERNAME=TODO_ENTER_YOURS_HERE

RELEASING.md

@@ -1,8 +1,8 @@
 # Release Checklist
 
 - [ ] Get `main` to the appropriate code release state.
-      [GitHub Actions](https://github.com/pylast/pylast/actions) should be running
+      [GitHub Actions](https://github.com/pylast/pylast/actions) should be running cleanly for
-      cleanly for all merges to `main`.
+      all merges to `main`.
       [![Test](https://github.com/pylast/pylast/workflows/Test/badge.svg)](https://github.com/pylast/pylast/actions)
 
 - [ ] Edit release draft, adjust text if needed:
@@ -12,8 +12,7 @@
 
 - [ ] Publish release
 
-- [ ] Check the tagged
+- [ ] Check the tagged [GitHub Actions build](https://github.com/pylast/pylast/actions/workflows/deploy.yml)
-      [GitHub Actions build](https://github.com/pylast/pylast/actions/workflows/deploy.yml)
       has deployed to [PyPI](https://pypi.org/project/pylast/#history)
 
 - [ ] Check installation:

example_test_pylast.yaml

@@ -1,4 +1,4 @@
-username: TODO_ENTER_YOURS_HERE
+username: TODO_ENTER_YOURS_HERE
-password_hash: TODO_ENTER_YOURS_HERE
+password_hash: TODO_ENTER_YOURS_HERE
-api_key: TODO_ENTER_YOURS_HERE
+api_key: TODO_ENTER_YOURS_HERE
-api_secret: TODO_ENTER_YOURS_HERE
+api_secret: TODO_ENTER_YOURS_HERE

pyproject.toml

@@ -15,46 +15,47 @@ keywords = [
   "scrobble",
   "scrobbling",
 ]
-license = { text = "Apache-2.0" }
+license = {text = "Apache-2.0"}
-maintainers = [
+maintainers = [{name = "Hugo van Kemenade"}]
-  { name = "Hugo van Kemenade" },
+authors = [{name = "Amr Hassan <amr.hassan@gmail.com> and Contributors", email = "amr.hassan@gmail.com"}]
-]
+requires-python = ">=3.7"
-authors = [
+dependencies = [
-  { name = "Amr Hassan <amr.hassan@gmail.com> and Contributors", email = "amr.hassan@gmail.com" },
+  "httpx",
-]
+  'importlib-metadata; python_version < "3.8"',
-requires-python = ">=3.8"
-classifiers = [
-  "Development Status :: 5 - Production/Stable",
-  "License :: OSI Approved :: Apache Software License",
-  "Programming Language :: Python :: 3 :: Only",
-  "Programming Language :: Python :: 3.8",
-  "Programming Language :: Python :: 3.9",
-  "Programming Language :: Python :: 3.10",
-  "Programming Language :: Python :: 3.11",
-  "Programming Language :: Python :: 3.12",
-  "Programming Language :: Python :: 3.13",
-  "Programming Language :: Python :: Implementation :: CPython",
-  "Programming Language :: Python :: Implementation :: PyPy",
-  "Topic :: Internet",
-  "Topic :: Multimedia :: Sound/Audio",
-  "Topic :: Software Development :: Libraries :: Python Modules",
 ]
 dynamic = [
   "version",
 ]
-dependencies = [
+classifiers = [
-  "httpx",
+    "Development Status :: 5 - Production/Stable",
+    "License :: OSI Approved :: Apache Software License",
+    "Programming Language :: Python :: 3",
+    "Programming Language :: Python :: 3 :: Only",
+    "Programming Language :: Python :: 3.7",
+    "Programming Language :: Python :: 3.8",
+    "Programming Language :: Python :: 3.9",
+    "Programming Language :: Python :: 3.10",
+    "Programming Language :: Python :: 3.11",
+    "Programming Language :: Python :: Implementation :: CPython",
+    "Programming Language :: Python :: Implementation :: PyPy",
+    "Topic :: Internet",
+    "Topic :: Multimedia :: Sound/Audio",
+    "Topic :: Software Development :: Libraries :: Python Modules",
 ]
-optional-dependencies.tests = [
+[project.optional-dependencies]
+tests = [
   "flaky",
   "pytest",
   "pytest-cov",
   "pytest-random-order",
   "pyyaml",
 ]
-urls.Changelog = "https://github.com/pylast/pylast/releases"
+
-urls.Homepage = "https://github.com/pylast/pylast"
+[project.urls]
-urls.Source = "https://github.com/pylast/pylast"
+Changelog = "https://github.com/pylast/pylast/releases"
+Homepage = "https://github.com/pylast/pylast"
+Source = "https://github.com/pylast/pylast"
+
 
 [tool.hatch]
 version.source = "vcs"
@@ -62,36 +63,5 @@ version.source = "vcs"
 [tool.hatch.version.raw-options]
 local_scheme = "no-local-version"
 
-[tool.ruff]
+[tool.isort]
-fix = true
+profile = "black"
-
-lint.select = [
-  "C4",     # flake8-comprehensions
-  "E",      # pycodestyle errors
-  "EM",     # flake8-errmsg
-  "F",      # pyflakes errors
-  "I",      # isort
-  "ISC",    # flake8-implicit-str-concat
-  "LOG",    # flake8-logging
-  "PGH",    # pygrep-hooks
-  "RUF022", # unsorted-dunder-all
-  "RUF100", # unused noqa (yesqa)
-  "UP",     # pyupgrade
-  "W",      # pycodestyle warnings
-  "YTT",    # flake8-2020
-]
-lint.extend-ignore = [
-  "E203", # Whitespace before ':'
-  "E221", # Multiple spaces before operator
-  "E226", # Missing whitespace around arithmetic operator
-  "E241", # Multiple spaces after ','
-]
-lint.isort.known-first-party = [
-  "pylast",
-]
-lint.isort.required-imports = [
-  "from __future__ import annotations",
-]
-
-[tool.pyproject-fmt]
-max_supported_python = "3.13"

src/pylast/__init__.py

@@ -1,9 +1,9 @@
 #
 # pylast -
-#     A Python interface to Last.fm and music.lonestar.it
+#     A Python interface to Last.fm and Libre.fm
 #
 # Copyright 2008-2010 Amr Hassan
-# Copyright 2013-2021 hugovk
+# Copyright 2013-2022 hugovk
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -23,7 +23,6 @@ from __future__ import annotations
 import collections
 import hashlib
 import html.entities
-import importlib.metadata
 import logging
 import os
 import re
@@ -37,11 +36,18 @@ from xml.dom import Node, minidom
 
 import httpx
 
+try:
+    # Python 3.8+
+    import importlib.metadata as importlib_metadata
+except ImportError:
+    # Python 3.7 and lower
+    import importlib_metadata  # type: ignore
+
 __author__ = "Amr Hassan, hugovk, Mice Pápai"
-__copyright__ = "Copyright (C) 2008-2010 Amr Hassan, 2013-2021 hugovk, 2017 Mice Pápai"
+__copyright__ = "Copyright (C) 2008-2010 Amr Hassan, 2013-2022 hugovk, 2017 Mice Pápai"
 __license__ = "apache2"
 __email__ = "amr.hassan@gmail.com"
-__version__ = importlib.metadata.version(__name__)
+__version__ = importlib_metadata.version(__name__)
 
 
 # 1 : This error does not exist
@@ -529,25 +535,26 @@ class _Network:
 
     def scrobble(
         self,
-        artist: str,
+        artist,
-        title: str,
+        title,
-        timestamp: int,
+        timestamp,
-        album: str | None = None,
+        album=None,
-        album_artist: str | None = None,
+        album_artist=None,
-        track_number: int | None = None,
+        track_number=None,
-        duration: int | None = None,
+        duration=None,
-        stream_id: str | None = None,
+        stream_id=None,
-        context: str | None = None,
+        context=None,
-        mbid: str | None = None,
+        mbid=None,
     ):
+
         """Used to add a track-play to a user's profile.
 
         Parameters:
             artist (Required) : The artist name.
             title (Required) : The track name.
-            timestamp (Required) : The time the track started playing, in Unix
+            timestamp (Required) : The time the track started playing, in UNIX
                 timestamp format (integer number of seconds since 00:00:00,
-                January 1st 1970 UTC).
+                January 1st 1970 UTC). This must be in the UTC time zone.
             album (Optional) : The album name.
             album_artist (Optional) : The album artist - if this differs from
                 the track artist.
@@ -593,6 +600,7 @@ class _Network:
 
         params = {}
         for i in range(len(tracks_to_scrobble)):
+
             params[f"artist[{i}]"] = tracks_to_scrobble[i]["artist"]
             params[f"track[{i}]"] = tracks_to_scrobble[i]["title"]
 
@@ -613,6 +621,7 @@ class _Network:
             }
 
             for arg in additional_args:
+
                 if arg in tracks_to_scrobble[i] and tracks_to_scrobble[i][arg]:
                     if arg in args_map_to:
                         maps_to = args_map_to[arg]
@@ -628,6 +637,7 @@ class _Network:
 
 
 class LastFMNetwork(_Network):
+
     """A Last.fm network object
 
     api_key: a provided API_KEY
@@ -705,7 +715,7 @@ class LastFMNetwork(_Network):
 
 class LibreFMNetwork(_Network):
     """
-    A preconfigured _Network object for music.lonestar.it
+    A preconfigured _Network object for Libre.fm
 
     api_key: a provided API_KEY
     api_secret: a provided API_SECRET
@@ -726,28 +736,29 @@ class LibreFMNetwork(_Network):
         username: str = "",
         password_hash: str = "",
     ) -> None:
+
         super().__init__(
-            name="music.lonestar.it",
+            name="Libre.fm",
-            homepage="https://music.lonestar.it",
+            homepage="https://libre.fm",
-            ws_server=("music.lonestar.it", "/2.0/"),
+            ws_server=("libre.fm", "/2.0/"),
             api_key=api_key,
             api_secret=api_secret,
             session_key=session_key,
             username=username,
             password_hash=password_hash,
             domain_names={
-                DOMAIN_ENGLISH: "music.lonestar.it",
+                DOMAIN_ENGLISH: "libre.fm",
-                DOMAIN_GERMAN: "music.lonestar.it",
+                DOMAIN_GERMAN: "libre.fm",
-                DOMAIN_SPANISH: "music.lonestar.it",
+                DOMAIN_SPANISH: "libre.fm",
-                DOMAIN_FRENCH: "music.lonestar.it",
+                DOMAIN_FRENCH: "libre.fm",
-                DOMAIN_ITALIAN: "music.lonestar.it",
+                DOMAIN_ITALIAN: "libre.fm",
-                DOMAIN_POLISH: "music.lonestar.it",
+                DOMAIN_POLISH: "libre.fm",
-                DOMAIN_PORTUGUESE: "music.lonestar.it",
+                DOMAIN_PORTUGUESE: "libre.fm",
-                DOMAIN_SWEDISH: "music.lonestar.it",
+                DOMAIN_SWEDISH: "libre.fm",
-                DOMAIN_TURKISH: "music.lonestar.it",
+                DOMAIN_TURKISH: "libre.fm",
-                DOMAIN_RUSSIAN: "music.lonestar.it",
+                DOMAIN_RUSSIAN: "libre.fm",
-                DOMAIN_JAPANESE: "music.lonestar.it",
+                DOMAIN_JAPANESE: "libre.fm",
-                DOMAIN_CHINESE: "music.lonestar.it",
+                DOMAIN_CHINESE: "libre.fm",
             },
             urls={
                 "album": "artist/%(artist)s/album/%(album)s",
@@ -771,6 +782,37 @@ class LibreFMNetwork(_Network):
         )
 
 
+def network_via_web_auth(
+    api_key: str,
+    api_secret: str,
+    network_class: _Network = LastFMNetwork,
+) -> _Network:
+    session_key_file = os.path.join(os.path.expanduser("~"), ".session_key")
+    network = network_class(api_key, api_secret)
+
+    if os.path.exists(session_key_file):
+        with open(session_key_file) as f:
+            network.session_key = f.read()
+    else:
+        skg = SessionKeyGenerator(network)
+        url = skg.get_web_auth_url()
+
+        print(f"Please authorise to access your account: {url}\n")
+        import webbrowser
+
+        webbrowser.open(url, new=2)
+        while True:
+            try:
+                network.session_key = skg.get_web_auth_session_key(url)
+                with open(session_key_file, "w") as f:
+                    f.write(network.session_key)
+                break
+            except WSError:
+                time.sleep(1)
+
+    return network
+
+
 class _ShelfCacheBackend:
     """Used as a backend for caching cacheable requests."""
 
@@ -893,7 +935,6 @@ class _Request:
         username = "" if username is None else f"?username={username}"
 
         (host_name, host_subdir) = self.network.ws_server
-        timeout = httpx.Timeout(5, read=10)
 
         if self.network.is_proxy_enabled():
             client = httpx.Client(
@@ -901,14 +942,12 @@ class _Request:
                 base_url=f"https://{host_name}",
                 headers=HEADERS,
                 proxies=self.network.proxy,
-                timeout=timeout,
             )
         else:
             client = httpx.Client(
                 verify=SSL_CONTEXT,
                 base_url=f"https://{host_name}",
                 headers=HEADERS,
-                timeout=timeout,
             )
 
         try:
@@ -1021,7 +1060,9 @@ class SessionKeyGenerator:
 
         return url
 
-    def get_web_auth_session_key_username(self, url, token: str = ""):
+    def get_web_auth_session_key_username(
+        self, url: str | None, token: str = ""
+    ) -> tuple[str, str]:
         """
         Retrieves the session key/username of a web authorization process by its URL.
         """
@@ -1041,7 +1082,7 @@ class SessionKeyGenerator:
         username = doc.getElementsByTagName("name")[0].firstChild.data
         return session_key, username
 
-    def get_web_auth_session_key(self, url, token: str = ""):
+    def get_web_auth_session_key(self, url: str, token: str = "") -> str:
         """
         Retrieves the session key of a web authorization process by its URL.
         """
@@ -1158,7 +1199,7 @@ class _BaseObject:
 
     def get_wiki_published_date(self):
         """
-        Returns the date on which the wiki was published.
+        Returns the summary of the wiki.
         Only for Album/Track.
         """
         return self.get_wiki("published")
@@ -1172,7 +1213,7 @@ class _BaseObject:
 
     def get_wiki_content(self):
         """
-        Returns the content of the wiki.
+        Returns the summary of the wiki.
         Only for Album/Track.
         """
         return self.get_wiki("content")
@@ -1242,10 +1283,8 @@ class _Chartable(_BaseObject):
         from_date value to the to_date value.
         chart_kind should be one of "album", "artist" or "track"
         """
-        import sys
-
         method = ".getWeekly" + chart_kind.title() + "Chart"
-        chart_type = getattr(sys.modules[__name__], chart_kind.title())
+        chart_type = eval(chart_kind.title())  # string to type
 
         params = self._get_params()
         if from_date and to_date:
@@ -1357,11 +1396,11 @@ class _Taggable(_BaseObject):
             new_tags.append(tag)
 
         for i in range(0, len(old_tags)):
-            if c_old_tags[i] not in c_new_tags:
+            if not c_old_tags[i] in c_new_tags:
                 to_remove.append(old_tags[i])
 
         for i in range(0, len(new_tags)):
-            if c_new_tags[i] not in c_old_tags:
+            if not c_new_tags[i] in c_old_tags:
                 to_add.append(new_tags[i])
 
         self.remove_tags(to_remove)
@@ -1509,7 +1548,7 @@ class _Opus(_Taggable):
         return f"{self.get_artist().get_name()} - {self.get_title()}"
 
     def __eq__(self, other):
-        if type(self) is not type(other):
+        if type(self) != type(other):
             return False
         a = self.get_title().lower()
         b = other.get_title().lower()
@@ -1547,7 +1586,7 @@ class _Opus(_Taggable):
         return self.info["image"][size]
 
     def get_title(self, properly_capitalized: bool = False):
-        """Returns the album or track title."""
+        """Returns the artist or track title."""
         if properly_capitalized:
             self.title = _extract(
                 self._request(self.ws_prefix + ".getInfo", True), "name"
@@ -2299,8 +2338,8 @@ class User(_Chartable):
         self,
         limit: int = 10,
         cacheable: bool = True,
-        time_from: int | None = None,
+        time_from=None,
-        time_to: int | None = None,
+        time_to=None,
         stream: bool = False,
         now_playing: bool = False,
     ):
@@ -2311,11 +2350,13 @@ class User(_Chartable):
         Parameters:
         limit : If None, it will try to pull all the available data.
         from (Optional) : Beginning timestamp of a range - only display
-        scrobbles after this time, in Unix timestamp format (integer
+        scrobbles after this time, in UNIX timestamp format (integer
-        number of seconds since 00:00:00, January 1st 1970 UTC).
+        number of seconds since 00:00:00, January 1st 1970 UTC). This
+        must be in the UTC time zone.
         to (Optional) : End timestamp of a range - only display scrobbles
-        before this time, in Unix timestamp format (integer number of
+        before this time, in UNIX timestamp format (integer number of
-        seconds since 00:00:00, January 1st 1970 UTC).
+        seconds since 00:00:00, January 1st 1970 UTC). This must be in
+        the UTC time zone.
         stream: If True, it will yield tracks as soon as a page has been retrieved.
 
         This method uses caching. Enable caching only if you're pulling a
@@ -2384,7 +2425,7 @@ class User(_Chartable):
         return _extract(doc, "registered")
 
     def get_unixtime_registered(self):
-        """Returns the user's registration date as a Unix timestamp."""
+        """Returns the user's registration date as a UNIX timestamp."""
 
         doc = self._request(self.ws_prefix + ".getInfo", True)
 
@@ -2779,8 +2820,7 @@ def _collect_nodes(
                     main.getAttribute("totalPages") or main.getAttribute("totalpages")
                 )
             else:
-                msg = "No total pages attribute"
+                raise PyLastError("No total pages attribute")
-                raise PyLastError(msg)
 
             for node in main.childNodes:
                 if not node.nodeType == xml.dom.Node.TEXT_NODE and (

tests/test_album.py

@@ -2,8 +2,6 @@
 """
 Integration (not unit) tests for pylast.py
 """
-from __future__ import annotations
-
 import pylast
 
 from .test_pylast import TestPyLastWithLastFm
@@ -96,8 +94,8 @@ class TestPyLastAlbum(TestPyLastWithLastFm):
         image = album.get_cover_image()
 
         # Assert
-        assert image.startswith("https://")
+        self.assert_startswith(image, "https://")
-        assert image.endswith(".gif") or image.endswith(".png")
+        self.assert_endswith(image, ".gif")
 
     def test_mbid(self) -> None:
         # Arrange

tests/test_artist.py

@@ -2,8 +2,6 @@
 """
 Integration (not unit) tests for pylast.py
 """
-from __future__ import annotations
-
 import pytest
 
 import pylast

tests/test_country.py

@@ -2,8 +2,6 @@
 """
 Integration (not unit) tests for pylast.py
 """
-from __future__ import annotations
-
 import pylast
 
 from .test_pylast import TestPyLastWithLastFm

tests/test_library.py

@@ -2,8 +2,6 @@
 """
 Integration (not unit) tests for pylast.py
 """
-from __future__ import annotations
-
 import pylast
 
 from .test_pylast import TestPyLastWithLastFm
@@ -18,7 +16,7 @@ class TestPyLastLibrary(TestPyLastWithLastFm):
         representation = repr(library)
 
         # Assert
-        assert representation.startswith("pylast.Library(")
+        self.assert_startswith(representation, "pylast.Library(")
 
     def test_str(self) -> None:
         # Arrange
@@ -28,7 +26,7 @@ class TestPyLastLibrary(TestPyLastWithLastFm):
         string = str(library)
 
         # Assert
-        assert string.endswith("'s Library")
+        self.assert_endswith(string, "'s Library")
 
     def test_library_is_hashable(self) -> None:
         # Arrange

tests/test_librefm.py

@@ -2,17 +2,15 @@
 """
 Integration (not unit) tests for pylast.py
 """
-from __future__ import annotations
-
 from flaky import flaky
 
 import pylast
 
-from .test_pylast import load_secrets
+from .test_pylast import PyLastTestCase, load_secrets
 
 
 @flaky(max_runs=3, min_passes=1)
-class TestPyLastWithLibreFm:
+class TestPyLastWithLibreFm(PyLastTestCase):
     """Own class for Libre.fm because we don't need the Last.fm setUp"""
 
     def test_libre_fm(self) -> None:
@@ -40,4 +38,4 @@ class TestPyLastWithLibreFm:
         representation = repr(network)
 
         # Assert
-        assert representation.startswith("pylast.LibreFMNetwork(")
+        self.assert_startswith(representation, "pylast.LibreFMNetwork(")

tests/test_network.py

@@ -1,9 +1,6 @@
 """
 Integration (not unit) tests for pylast.py
 """
-
-from __future__ import annotations
-
 import re
 import time
 
@@ -333,12 +330,12 @@ class TestPyLastNetwork(TestPyLastWithLastFm):
         # Assert
         assert len(images) == 4
 
-        assert images[pylast.SIZE_SMALL].startswith("https://")
+        self.assert_startswith(images[pylast.SIZE_SMALL], "https://")
-        assert images[pylast.SIZE_SMALL].endswith(".png")
+        self.assert_endswith(images[pylast.SIZE_SMALL], ".png")
         assert "/34s/" in images[pylast.SIZE_SMALL]
 
-        assert images[pylast.SIZE_EXTRA_LARGE].startswith("https://")
+        self.assert_startswith(images[pylast.SIZE_EXTRA_LARGE], "https://")
-        assert images[pylast.SIZE_EXTRA_LARGE].endswith(".png")
+        self.assert_endswith(images[pylast.SIZE_EXTRA_LARGE], ".png")
         assert "/300x300/" in images[pylast.SIZE_EXTRA_LARGE]
 
     def test_artist_search(self) -> None:
@@ -365,12 +362,12 @@ class TestPyLastNetwork(TestPyLastWithLastFm):
         # Assert
         assert len(images) == 5
 
-        assert images[pylast.SIZE_SMALL].startswith("https://")
+        self.assert_startswith(images[pylast.SIZE_SMALL], "https://")
-        assert images[pylast.SIZE_SMALL].endswith(".png")
+        self.assert_endswith(images[pylast.SIZE_SMALL], ".png")
         assert "/34s/" in images[pylast.SIZE_SMALL]
 
-        assert images[pylast.SIZE_EXTRA_LARGE].startswith("https://")
+        self.assert_startswith(images[pylast.SIZE_EXTRA_LARGE], "https://")
-        assert images[pylast.SIZE_EXTRA_LARGE].endswith(".png")
+        self.assert_endswith(images[pylast.SIZE_EXTRA_LARGE], ".png")
         assert "/300x300/" in images[pylast.SIZE_EXTRA_LARGE]
 
     def test_track_search(self) -> None:
@@ -399,12 +396,12 @@ class TestPyLastNetwork(TestPyLastWithLastFm):
         # Assert
         assert len(images) == 4
 
-        assert images[pylast.SIZE_SMALL].startswith("https://")
+        self.assert_startswith(images[pylast.SIZE_SMALL], "https://")
-        assert images[pylast.SIZE_SMALL].endswith(".png")
+        self.assert_endswith(images[pylast.SIZE_SMALL], ".png")
         assert "/34s/" in images[pylast.SIZE_SMALL]
 
-        assert images[pylast.SIZE_EXTRA_LARGE].startswith("https://")
+        self.assert_startswith(images[pylast.SIZE_EXTRA_LARGE], "https://")
-        assert images[pylast.SIZE_EXTRA_LARGE].endswith(".png")
+        self.assert_endswith(images[pylast.SIZE_EXTRA_LARGE], ".png")
         assert "/300x300/" in images[pylast.SIZE_EXTRA_LARGE]
 
     def test_search_get_total_result_count(self) -> None:

tests/test_pylast.py

@@ -2,8 +2,6 @@
 """
 Integration (not unit) tests for pylast.py
 """
-from __future__ import annotations
-
 import os
 import time
 
@@ -34,17 +32,25 @@ def load_secrets():  # pragma: no cover
     return doc
 
 
+class PyLastTestCase:
+    def assert_startswith(self, s, prefix, start=None, end=None) -> None:
+        assert s.startswith(prefix, start, end)
+
+    def assert_endswith(self, s, suffix, start=None, end=None) -> None:
+        assert s.endswith(suffix, start, end)
+
+
 def _no_xfail_rerun_filter(err, name, test, plugin) -> bool:
     for _ in test.iter_markers(name="xfail"):
         return False
 
 
 @flaky(max_runs=3, min_passes=1, rerun_filter=_no_xfail_rerun_filter)
-class TestPyLastWithLastFm:
+class TestPyLastWithLastFm(PyLastTestCase):
+
     secrets = None
 
-    @staticmethod
+    def unix_timestamp(self):
-    def unix_timestamp() -> int:
         return int(time.time())
 
     @classmethod
@@ -65,8 +71,7 @@ class TestPyLastWithLastFm:
             password_hash=password_hash,
         )
 
-    @staticmethod
+    def helper_is_thing_hashable(self, thing) -> None:
-    def helper_is_thing_hashable(thing) -> None:
         # Arrange
         things = set()
 
@@ -77,8 +82,7 @@ class TestPyLastWithLastFm:
         assert thing is not None
         assert len(things) == 1
 
-    @staticmethod
+    def helper_validate_results(self, a, b, c) -> None:
-    def helper_validate_results(a, b, c) -> None:
         # Assert
         assert a is not None
         assert b is not None
@@ -102,31 +106,27 @@ class TestPyLastWithLastFm:
         # Assert
         self.helper_validate_results(result1, result2, result3)
 
-    @staticmethod
+    def helper_at_least_one_thing_in_top_list(self, things, expected_type) -> None:
-    def helper_at_least_one_thing_in_top_list(things, expected_type) -> None:
         # Assert
         assert len(things) > 1
         assert isinstance(things, list)
         assert isinstance(things[0], pylast.TopItem)
         assert isinstance(things[0].item, expected_type)
 
-    @staticmethod
+    def helper_only_one_thing_in_top_list(self, things, expected_type) -> None:
-    def helper_only_one_thing_in_top_list(things, expected_type) -> None:
         # Assert
         assert len(things) == 1
         assert isinstance(things, list)
         assert isinstance(things[0], pylast.TopItem)
         assert isinstance(things[0].item, expected_type)
 
-    @staticmethod
+    def helper_only_one_thing_in_list(self, things, expected_type) -> None:
-    def helper_only_one_thing_in_list(things, expected_type) -> None:
         # Assert
         assert len(things) == 1
         assert isinstance(things, list)
         assert isinstance(things[0], expected_type)
 
-    @staticmethod
+    def helper_two_different_things_in_top_list(self, things, expected_type) -> None:
-    def helper_two_different_things_in_top_list(things, expected_type) -> None:
         # Assert
         assert len(things) == 2
         thing1 = things[0]

tests/test_tag.py

@@ -2,8 +2,6 @@
 """
 Integration (not unit) tests for pylast.py
 """
-from __future__ import annotations
-
 import pylast
 
 from .test_pylast import TestPyLastWithLastFm

tests/test_track.py

@@ -1,9 +1,6 @@
 """
 Integration (not unit) tests for pylast.py
 """
-
-from __future__ import annotations
-
 import time
 
 import pytest
@@ -138,7 +135,11 @@ class TestPyLastTrack(TestPyLastWithLastFm):
         similar = track.get_similar()
 
         # Assert
-        found = any(str(track.item) == "Cher - Strong Enough" for track in similar)
+        found = False
+        for track in similar:
+            if str(track.item) == "Madonna - Vogue":
+                found = True
+                break
         assert found
 
     def test_track_get_similar_limits(self) -> None:

tests/test_user.py

@@ -2,8 +2,6 @@
 """
 Integration (not unit) tests for pylast.py
 """
-from __future__ import annotations
-
 import calendar
 import datetime as dt
 import inspect
@@ -26,7 +24,7 @@ class TestPyLastUser(TestPyLastWithLastFm):
         representation = repr(user)
 
         # Assert
-        assert representation.startswith("pylast.User('RJ',")
+        self.assert_startswith(representation, "pylast.User('RJ',")
 
     def test_str(self) -> None:
         # Arrange
@@ -347,7 +345,7 @@ class TestPyLastUser(TestPyLastWithLastFm):
         url = user.get_image()
 
         # Assert
-        assert url.startswith("https://")
+        self.assert_startswith(url, "https://")
 
     def test_user_get_library(self) -> None:
         # Arrange
@@ -430,8 +428,8 @@ class TestPyLastUser(TestPyLastWithLastFm):
         image = user.get_image()
 
         # Assert
-        assert image.startswith("https://")
+        self.assert_startswith(image, "https://")
-        assert image.endswith(".png")
+        self.assert_endswith(image, ".png")
 
     def test_get_url(self) -> None:
         # Arrange

tests/unicode_test.py

@@ -1,5 +1,3 @@
-from __future__ import annotations
-
 from unittest import mock
 
 import pytest
@@ -27,7 +25,7 @@ def test_get_cache_key(artist) -> None:
 
 @pytest.mark.parametrize("obj", [pylast.Artist("B\xe9l", mock_network())])
 def test_cast_and_hash(obj) -> None:
-    assert isinstance(str(obj), str)
+    assert type(str(obj)) is str
     assert isinstance(hash(obj), int)
 
 

tox.ini

@@ -1,35 +1,27 @@
 [tox]
-requires =
+envlist =
-    tox>=4.2
-env_list =
     lint
-    py{py3, 313, 312, 311, 310, 39, 38}
+    py{py3, 311, 310, 39, 38, 37}
+isolated_build = true
 
 [testenv]
-extras =
+passenv =
-    tests
-pass_env =
     FORCE_COLOR
     PYLAST_API_KEY
     PYLAST_API_SECRET
     PYLAST_PASSWORD_HASH
     PYLAST_USERNAME
+extras =
+    tests
 commands =
-    {envpython} -m pytest -v -s -W all \
+    pytest -v -s -W all --cov pylast --cov tests --cov-report term-missing --cov-report xml --random-order {posargs}
-      --cov pylast \
-      --cov tests \
-      --cov-report html \
-      --cov-report term-missing \
-      --cov-report xml \
-      --random-order \
-      {posargs}
 
 [testenv:lint]
+passenv =
+    PRE_COMMIT_COLOR
 skip_install = true
 deps =
     pre-commit
-pass_env =
-    PRE_COMMIT_COLOR
 commands =
     pre-commit run --all-files --show-diff-on-failure