gh-113086: Add tests for os.chmod() and os.lchmod() by serhiy-storchaka · Pull Request #113087 · python/cpython · GitHub
Skip to content

gh-113086: Add tests for os.chmod() and os.lchmod() - #113087

Merged
serhiy-storchaka merged 1 commit into
python:mainfrom
serhiy-storchaka:test-chmod
Dec 14, 2023
Merged

gh-113086: Add tests for os.chmod() and os.lchmod()#113087
serhiy-storchaka merged 1 commit into
python:mainfrom
serhiy-storchaka:test-chmod

Conversation

@serhiy-storchaka

@serhiy-storchaka serhiy-storchaka commented Dec 14, 2023 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

Also make test_copymode_symlink_to_symlink in test_shutil more strict.

@serhiy-storchaka
pythongh-113086: Add tests for os.chmod() and os.lchmod()
9ee5c7b 
Also make test_copymode_symlink_to_symlink in test_shutil more strict.
@serhiy-storchaka serhiy-storchaka added tests Tests in the Lib/test dir skip news needs backport to 3.11 only security fixes needs backport to 3.12 only security fixes labels Dec 14, 2023
@bedevere-app bedevere-app Bot mentioned this pull request Dec 14, 2023
Closed
@serhiy-storchaka
serhiy-storchaka merged commit b4f2c89 into python:main Dec 14, 2023
@miss-islington-app

miss-islington-app Bot commented Dec 14, 2023

Copy link
Copy Markdown

@serhiy-storchaka
serhiy-storchaka deleted the test-chmod branch December 14, 2023 07:16
@bedevere-app

bedevere-app Bot commented Dec 14, 2023

Copy link
Copy Markdown

GH-113088 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.12 only security fixes label Dec 14, 2023
@bedevere-app

bedevere-app Bot commented Dec 14, 2023

Copy link
Copy Markdown

GH-113089 is a backport of this pull request to the 3.11 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.11 only security fixes label Dec 14, 2023
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 14, 2023
@serhiy-storchaka @miss-islington
pythongh-113086: Add tests for os.chmod() and os.lchmod() (pythonGH-1…
ffb8412 
…13087)

Also make test_copymode_symlink_to_symlink in test_shutil more strict.
(cherry picked from commit b4f2c89)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 14, 2023
@serhiy-storchaka @miss-islington
pythongh-113086: Add tests for os.chmod() and os.lchmod() (pythonGH-1…
f010576 
…13087)

Also make test_copymode_symlink_to_symlink in test_shutil more strict.
(cherry picked from commit b4f2c89)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
serhiy-storchaka added a commit that referenced this pull request Dec 14, 2023
@miss-islington @serhiy-storchaka
[3.12] gh-113086: Add tests for os.chmod() and os.lchmod() (GH-113087) (
0bd1899 
GH-113088)

Also make test_copymode_symlink_to_symlink in test_shutil more strict.
(cherry picked from commit b4f2c89)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@bedevere-bot

bedevere-bot commented Dec 14, 2023

Copy link
Copy Markdown

⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️

Hi! The buildbot wasm32-wasi 3.x has failed when building commit b4f2c89.

What do you need to do:

  1. Don't panic.
  2. Check the buildbot page in the devguide if you don't know what the buildbots are or how they work.
  3. Go to the page of the buildbot that failed (https://buildbot.python.org/all/#builders/1046/builds/3741) and take a look at the build logs.
  4. Check if the failure is related to this commit (b4f2c89) or if it is a false positive.
  5. If the failure is related to this commit, please, reflect that on the issue and make a new Pull Request with a fix.

You can take a look at the buildbot page here:

https://buildbot.python.org/all/#builders/1046/builds/3741

Failed tests:

  • test_posix

Failed subtests:

  • test_chmod_file - test.test_posix.PosixTester.test_chmod_file
  • test_chmod_dir - test.test_posix.PosixTester.test_chmod_dir

Summary of the results of the build (if available):

==

Click to see traceback logs 
Traceback (most recent call last):
  File "/Lib/test/test_posix.py", line 970, in test_chmod_dir
    self.check_chmod(posix.chmod, target)
    ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^
  File "/Lib/test/test_posix.py", line 952, in check_chmod
    self.assertEqual(os.stat(target).st_mode, new_mode)
    ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: 16384 != 16530


Traceback (most recent call last):
  File "/Lib/test/test_posix.py", line 960, in test_chmod_file
    self.check_chmod(posix.chmod, os_helper.TESTFN)
    ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Lib/test/test_posix.py", line 952, in check_chmod
    self.assertEqual(os.stat(target).st_mode, new_mode)
    ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: 32768 != 32914

serhiy-storchaka added a commit that referenced this pull request Dec 14, 2023
@miss-islington @serhiy-storchaka
[3.11] gh-113086: Add tests for os.chmod() and os.lchmod() (GH-113087) (
e4cdb74 
GH-113089)

Also make test_copymode_symlink_to_symlink in test_shutil more strict.
(cherry picked from commit b4f2c89)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@bedevere-bot

bedevere-bot commented Dec 14, 2023

Copy link
Copy Markdown

⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️

Hi! The buildbot wasm32-wasi 3.11 has failed when building commit e4cdb74.

What do you need to do:

  1. Don't panic.
  2. Check the buildbot page in the devguide if you don't know what the buildbots are or how they work.
  3. Go to the page of the buildbot that failed (https://buildbot.python.org/all/#builders/1047/builds/1156) and take a look at the build logs.
  4. Check if the failure is related to this commit (e4cdb74) or if it is a false positive.
  5. If the failure is related to this commit, please, reflect that on the issue and make a new Pull Request with a fix.

You can take a look at the buildbot page here:

https://buildbot.python.org/all/#builders/1047/builds/1156

Failed tests:

  • test_posix

Failed subtests:

  • test_chmod_file - test.test_posix.PosixTester.test_chmod_file
  • test_chmod_dir - test.test_posix.PosixTester.test_chmod_dir

Summary of the results of the build (if available):

==

Click to see traceback logs 
Traceback (most recent call last):
  File "/Lib/test/test_posix.py", line 960, in test_chmod_file
    self.check_chmod(posix.chmod, os_helper.TESTFN)
  File "/Lib/test/test_posix.py", line 952, in check_chmod
    self.assertEqual(os.stat(target).st_mode, new_mode)
AssertionError: 32768 != 32914


Traceback (most recent call last):
  File "/Lib/test/test_posix.py", line 970, in test_chmod_dir
    self.check_chmod(posix.chmod, target)
  File "/Lib/test/test_posix.py", line 952, in check_chmod
    self.assertEqual(os.stat(target).st_mode, new_mode)
AssertionError: 16384 != 16530

@bedevere-bot

bedevere-bot commented Dec 14, 2023

Copy link
Copy Markdown

corona10 pushed a commit to corona10/cpython that referenced this pull request Dec 15, 2023
@serhiy-storchaka @corona10
pythongh-113086: Add tests for os.chmod() and os.lchmod() (pythonGH-1…
59bcb6c 
…13087)

Also make test_copymode_symlink_to_symlink in test_shutil more strict.
aisk pushed a commit to aisk/cpython that referenced this pull request Feb 11, 2024
@serhiy-storchaka @aisk
pythongh-113086: Add tests for os.chmod() and os.lchmod() (pythonGH-1…
2224761 
…13087)

Also make test_copymode_symlink_to_symlink in test_shutil more strict.
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
@serhiy-storchaka @Glyphack
pythongh-113086: Add tests for os.chmod() and os.lchmod() (pythonGH-1…
822db63 
…13087)

Also make test_copymode_symlink_to_symlink in test_shutil more strict.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@serhiy-storchaka @bedevere-bot