Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make tests pass after year 2039 #7191

Merged
merged 1 commit into from
Feb 11, 2023
Merged

Make tests pass after year 2039 #7191

merged 1 commit into from
Feb 11, 2023

Conversation

bmwiedemann
Copy link
Contributor

@bmwiedemann bmwiedemann commented Feb 4, 2023

What do these changes do?

Make tests pass after year 2039

Are there changes in behavior for the user?

none

Related issue number

N/A

Background:
As part of my work on reproducible builds for openSUSE, I check that software still gives identical build results in the future. The usual offset is +16 years, because that is how long I expect some software will be used in some places. This showed up failing tests in our python-aiohttp package build. See https://reproducible-builds.org/ for why this matters.

Checklist

  • I think the code is well written
  • Unit tests for the changes exist
  • Documentation reflects the changes
  • If you provide code modification, please add yourself to CONTRIBUTORS.txt
    • The format is <Name> <Surname>.
    • Please keep alphabetical order, the file is sorted by names.
  • Add a new news fragment into the CHANGES folder
    • name it <issue_id>.<type> for example (588.bugfix)
    • if you don't have an issue_id change it to the pr id after creating the pr
    • ensure type is one of the following:
      • .feature: Signifying a new feature.
      • .bugfix: Signifying a bug fix.
      • .doc: Signifying a documentation improvement.
      • .removal: Signifying a deprecation or removal of public API.
      • .misc: A ticket has been closed, but it is not of interest to users.
    • Make sure to use full sentences with correct case and punctuation, for example: "Fix issue with non-ascii contents in doctest text files."

@bmwiedemann bmwiedemann requested a review from asvetlov as a code owner February 4, 2023 09:54
@psf-chronographer psf-chronographer bot added the bot:chronographer:provided There is a change note present in this PR label Feb 4, 2023
@codecov
Copy link

codecov bot commented Feb 4, 2023

Codecov Report

Merging #7191 (a5ae686) into master (2a1024f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #7191   +/-   ##
=======================================
  Coverage   97.37%   97.37%           
=======================================
  Files         106      106           
  Lines       31093    31093           
  Branches     3875     3875           
=======================================
  Hits        30278    30278           
  Misses        613      613           
  Partials      202      202           
Flag Coverage Δ
CI-GHA 97.28% <ø> (ø)
OS-Linux 96.94% <ø> (-0.01%) ⬇️
OS-Windows 95.35% <ø> (ø)
OS-macOS 96.50% <ø> (-0.01%) ⬇️
Py-3.10.9 97.03% <ø> (ø)
Py-3.11.0 95.59% <ø> (-0.01%) ⬇️
Py-3.7.15 96.75% <ø> (ø)
Py-3.7.9 95.23% <ø> (ø)
Py-3.8.10 95.13% <ø> (ø)
Py-3.8.16 96.65% <ø> (ø)
Py-3.9.13 95.12% <ø> (-0.01%) ⬇️
Py-3.9.16 96.66% <ø> (-0.01%) ⬇️
Py-pypy7.3.11 94.16% <ø> (-0.01%) ⬇️
VM-macos 96.50% <ø> (-0.01%) ⬇️
VM-ubuntu 96.94% <ø> (-0.01%) ⬇️
VM-windows 95.35% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
tests/test_cookiejar.py 99.09% <ø> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

CHANGES/7191.misc Outdated Show resolved Hide resolved
Background:
As part of my work on reproducible builds for openSUSE, I check that software still gives identical build results in the future.
The usual offset is +16 years, because that is how long I expect some software will be used in some places.
This showed up failing tests in our package build.
See https://reproducible-builds.org/ for why this matters.
@bmwiedemann bmwiedemann requested review from webknjaz and removed request for asvetlov February 6, 2023 07:34
@bmwiedemann bmwiedemann changed the title Make tests pass after 2039 Make tests pass after year 2039 Feb 6, 2023
@Dreamsorcerer Dreamsorcerer merged commit 82c944c into aio-libs:master Feb 11, 2023
@patchback
Copy link
Contributor

patchback bot commented Feb 11, 2023

Backport to 3.9: 💚 backport PR created

✅ Backport PR branch: patchback/backports/3.9/82c944cb6a35e83c2e9d75a9719cb74a478bfd73/pr-7191

Backported as #7203

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Feb 11, 2023
<!-- Thank you for your contribution! -->

## What do these changes do?

Make tests pass after year 2039

## Are there changes in behavior for the user?

none

## Related issue number

N/A

Background:
As part of my work on reproducible builds for openSUSE, I check that
software still gives identical build results in the future. The usual
offset is +16 years, because that is how long I expect some software
will be used in some places. This showed up failing tests in our
`python-aiohttp` package build. See https://reproducible-builds.org/ for
why this matters.

## Checklist

- [X] I think the code is well written
- [X] Unit tests for the changes exist
- [ ] Documentation reflects the changes
- [ ] If you provide code modification, please add yourself to
`CONTRIBUTORS.txt`
  * The format is &lt;Name&gt; &lt;Surname&gt;.
  * Please keep alphabetical order, the file is sorted by names.
- [x] Add a new news fragment into the `CHANGES` folder
  * name it `<issue_id>.<type>` for example (588.bugfix)
* if you don't have an `issue_id` change it to the pr id after creating
the pr
  * ensure type is one of the following:
    * `.feature`: Signifying a new feature.
    * `.bugfix`: Signifying a bug fix.
    * `.doc`: Signifying a documentation improvement.
    * `.removal`: Signifying a deprecation or removal of public API.
* `.misc`: A ticket has been closed, but it is not of interest to users.
* Make sure to use full sentences with correct case and punctuation, for
example: "Fix issue with non-ascii contents in doctest text files."

(cherry picked from commit 82c944c)
Dreamsorcerer pushed a commit that referenced this pull request Feb 11, 2023
)

**This is a backport of PR #7191 as merged into master
(82c944c).**

<!-- Thank you for your contribution! -->

## What do these changes do?

Make tests pass after year 2039

## Are there changes in behavior for the user?

none

## Related issue number

N/A

Background:
As part of my work on reproducible builds for openSUSE, I check that
software still gives identical build results in the future. The usual
offset is +16 years, because that is how long I expect some software
will be used in some places. This showed up failing tests in our
`python-aiohttp` package build. See https://reproducible-builds.org/ for
why this matters.


## Checklist

- [X] I think the code is well written
- [X] Unit tests for the changes exist
- [ ] Documentation reflects the changes
- [ ] If you provide code modification, please add yourself to
`CONTRIBUTORS.txt`
  * The format is &lt;Name&gt; &lt;Surname&gt;.
  * Please keep alphabetical order, the file is sorted by names.
- [x] Add a new news fragment into the `CHANGES` folder
  * name it `<issue_id>.<type>` for example (588.bugfix)
* if you don't have an `issue_id` change it to the pr id after creating
the pr
  * ensure type is one of the following:
    * `.feature`: Signifying a new feature.
    * `.bugfix`: Signifying a bug fix.
    * `.doc`: Signifying a documentation improvement.
    * `.removal`: Signifying a deprecation or removal of public API.
* `.misc`: A ticket has been closed, but it is not of interest to users.
* Make sure to use full sentences with correct case and punctuation, for
example: "Fix issue with non-ascii contents in doctest text files."

Co-authored-by: Bernhard M. Wiedemann <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot:chronographer:provided There is a change note present in this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants