V2.1.2 more bug fixes

[Deconstrained]

This fixes #45 and #46.

• Update try_decoding to return None for empty input, and add a unit test where previously there was none.
• Update iter_cursor to respect the client default and accept a page_size keyword argument similar to iter_all. Moreover, the similarly-named arguments from iter_all are passed through to it when calling out to it for cursor-based iteration endpoints.

[Copilot]

Pull Request Overview

Updates in this PR address two bugs: ensuring that try_decoding returns None for empty input and enhancing iter_cursor (and indirectly iter_all) to support a page_size keyword argument. Other changes include adding appropriate tests and updating documentation and version numbers.

• Updated tests to pass page_size to iter_all and verify iter_cursor behavior.
• Modified iter_all and iter_cursor in the API client to support page_size and updated try_decoding for empty responses.
• Updated documentation, changelog, and versioning details accordingly.

Reviewed Changes

Copilot reviewed 21 out of 21 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
tests/rest_api_v2_client_test.py	Updated tests to pass and verify page_size parameter and iter_cursor invocation.
tests/mocks.py	Added handling for JSONDecodeError in JSON mocking.
tests/common_test.py	Added unit tests for try_decoding with empty and invalid JSON responses.
pyproject.toml	Bumped version from 2.1.1 to 2.1.2.
pagerduty/rest_api_v2_client.py	Enhanced iter_all and iter_cursor to accept page_size and pass item_hook.
pagerduty/common.py	Updated try_decoding to catch JSONDecodeError and return None when appropriate.
docs/*	Updated version strings and changelog entries to reflect new changes.
README.rst & CHANGELOG.rst	Revised documentation to match the updated CLI and API behaviors.

Comments suppressed due to low confidence (2)

pagerduty/rest_api_v2_client.py:1013

• The variable 'total' is referenced here but does not appear to be defined anywhere. Consider either removing it or adding it as a parameter if its behavior is intended.

data = { 'limit': (self.default_page_size, page_size)[int(bool(page_size))], 'total': int(bool(total)) }

pagerduty/rest_api_v2_client.py:1013

• [nitpick] Using an indexed tuple for assigning 'limit' may be less readable than a clear conditional expression. Consider rewriting it as: 'limit': page_size if page_size is not None else self.default_page_size.

data = { 'limit': (self.default_page_size, page_size)[int(bool(page_size))], ... }