#1284 Improve the file permission checks in the parser unit tests
Closed: Fixed None Opened 12 years ago by dpal.

Currently the unit test assumes a specific set of permissions on a file in repo. The file it checks is ini/ini.d/smerge.conf Depending upon how the repo is created (root or non root) the permissions on the file might be different. If the default permissions are set in such a way that it would not match the expected values hard coded in the unit test the test will fail.

This task is to review this area and improve the logic around the permission checks.


Fields changed

rhbz: => 0

Fields changed

rhbz: => 0

Fields changed

rhbz: => 0

This issue is addressed in the current patches on the list and in tree.

milestone: Tools Backlog => Tools 1.0

This issue is addressed in the current patches on the list and in tree.

milestone: Tools Backlog => Tools 1.0

This issue is addressed in the current patches on the list and in tree.

milestone: Tools Backlog => Tools 1.0

Fields changed

design: =>
design_review: => 0
fedora_test_page: =>
resolution: => fixed
status: new => closed

Fields changed

design: =>
design_review: => 0
fedora_test_page: =>
resolution: => fixed
status: new => closed

Fields changed

design: =>
design_review: => 0
fedora_test_page: =>
resolution: => fixed
status: new => closed

Metadata Update from @dpal:
- Issue set to the milestone: Tools 1.0

7 years ago

SSSD is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in SSSD's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/SSSD/sssd/issues/2326

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Login to comment on this ticket.

Metadata