This code path was never actually tested before upgrading to
hypothesis ~6.45.
base64.urlsafe_b64encode returns bytes, which applied str()
to since D7938.
This reverts commit ce5824a68aed3b2717542c4e31d99152af7c2605.
Differential D7955
Fix pagination in api_raw_extrinsic_metadata_swhid vlorentz on Jun 3 2022, 11:43 AM. Authored by
Details This code path was never actually tested before upgrading to base64.urlsafe_b64encode returns bytes, which applied str() This reverts commit ce5824a68aed3b2717542c4e31d99152af7c2605.
Diff Detail
Event TimelineComment Actions Build was aborted Patch application report for D7955 (id=28645)Rebasing onto ce5824a68a... Current branch diff-target is up to date. Changes applied before testcommit 9ad820f6465f3c46bf1476334d1b47bad7a9da20 Author: Valentin Lorentz <vlorentz@softwareheritage.org> Date: Fri Jun 3 11:43:15 2022 +0200 Revert "tests: Fix flaky ones since hypothesis 6.46.11 release" This reverts commit ce5824a68aed3b2717542c4e31d99152af7c2605. commit 429dbf811a37fabd3bcaeb26857a38393a9e3a86 Author: Valentin Lorentz <vlorentz@softwareheritage.org> Date: Fri Jun 3 11:41:40 2022 +0200 Fix pagination in api_raw_extrinsic_metadata_swhid This code path was never actually tested before upgrading to hypothesis ~6.45. base64.urlsafe_b64encode returns bytes, which Django applied str() to, causing the token to be Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1877/ Comment Actions Build was aborted Patch application report for D7955 (id=28648)Rebasing onto ce5824a68a... Current branch diff-target is up to date. Changes applied before testcommit 3a4e607645bd45269ec72cb5e9b4a487c71145ae Author: Valentin Lorentz <vlorentz@softwareheritage.org> Date: Fri Jun 3 11:43:15 2022 +0200 Revert "tests: Fix flaky ones since hypothesis 6.46.11 release" This reverts commit ce5824a68aed3b2717542c4e31d99152af7c2605. commit 1c512a8365afb9485e9ac202046fd1ca4223a3c4 Author: Valentin Lorentz <vlorentz@softwareheritage.org> Date: Fri Jun 3 11:41:40 2022 +0200 Fix pagination in api_raw_extrinsic_metadata_swhid This code path was not properly tested before upgrading to hypothesis ~6.45. base64.urlsafe_b64encode returns bytes, which applied str() to since 6e52f32e3e842dbb68edf1fb220f18c7eef8a28d. Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1878/ Comment Actions Build is green Patch application report for D7955 (id=28648)Rebasing onto ce5824a68a... Current branch diff-target is up to date. Changes applied before testcommit 3a4e607645bd45269ec72cb5e9b4a487c71145ae Author: Valentin Lorentz <vlorentz@softwareheritage.org> Date: Fri Jun 3 11:43:15 2022 +0200 Revert "tests: Fix flaky ones since hypothesis 6.46.11 release" This reverts commit ce5824a68aed3b2717542c4e31d99152af7c2605. commit 1c512a8365afb9485e9ac202046fd1ca4223a3c4 Author: Valentin Lorentz <vlorentz@softwareheritage.org> Date: Fri Jun 3 11:41:40 2022 +0200 Fix pagination in api_raw_extrinsic_metadata_swhid This code path was not properly tested before upgrading to hypothesis ~6.45. base64.urlsafe_b64encode returns bytes, which applied str() to since 6e52f32e3e842dbb68edf1fb220f18c7eef8a28d. See https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1879/ for more details. Comment Actions Looks good to me. I will submit another diff to ensure the query_params parameter of the reverse function is of type Dict[str, Optional[str]] in order to avoid that kind of issues. |