Change Details

This also: - extracts pytest fixtures needed to cover the GitHubSession code. - avoid mutating the inputted credentials passed in constructors - rework test docstrings to "it should" format for more concise description Related to T4232 TODO in other diffs: - Move pytest fixtures into a runtime pytest module (to be reusable in swh.lister) - Branch the use of GitHubSession in the new function to compute the canonical github urls

This also: - extracts pytest fixtures needed to cover the GitHubSession code. - avoid mutating the inputted credentials passed in constructors - rework test docstrings to "it should" format for more concise description Related to T4232 TODO in other diffs: - D7840: Move pytest fixtures into a runtime pytest module (to be reusable in swh.lister) - D7870: Branch the use of GitHubSession in the new function to compute the canonical github urls

This also: - extracts pytest fixtures needed to cover the GitHubSession code. - avoid mutating the inputted credentials passed in constructors - rework test docstrings to "it should" format for more concise description Related to T4232 TODO in other diffs: - D7840: Move pytest fixtures into a runtime pytest module (to be reusable in swh.lister) - D7870: Branch the use of GitHubSession in the new function to compute the canonical github urls