PR#4644: tests: a try except block on tearDown to prevent failing tests due to directory removing race conditions - - Pagure.io

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

pagure

#4644 tests: a try except block on tearDown to prevent failing tests due to directory removing race conditions

Merged Oct 29, 2019 by pingou. Opened Oct 24, 2019 by jlanda.

jlanda/pagure fix-test-race-condition into master

Download 4644.patch

jlanda commented Oct 24, 2019

Seems like fixes #4641 :)

ngompa commented Oct 24, 2019

:thumbsup:

jlanda commented Oct 24, 2019

let's try removing conditional bypass...

rebased onto 902dcfcb60588decfaa34d5f88d87256ec0c109e

Oct 24, 2019

pretty please pagure-ci rebuild

Oct 24, 2019

2 new commits added

test_pagure_admin: remove conditional test skipping
tests: a try except block on tearDown to prevent failing tests due to directory removing race conditions

Oct 24, 2019

pretty please pagure-ci rebuild

Oct 24, 2019

pretty please pagure-ci rebuild

Oct 24, 2019

jlanda commented Oct 24, 2019

jenkin's 2909 fails are not related to this PR but to the pipeline

Edited Oct 24, 2019 by jlanda

pretty please pagure-ci rebuild

Oct 24, 2019

jlanda commented Oct 24, 2019

5/5 success jobs with this. I feel quite confident with it :)

pretty please pagure-ci rebuild

Oct 25, 2019

pretty please pagure-ci rebuild

Oct 25, 2019

pretty please pagure-ci rebuild

Oct 25, 2019

pretty please pagure-ci rebuild

Oct 25, 2019

pretty please pagure-ci rebuild

Oct 28, 2019

Pull-Request has been merged by pingou

Oct 29, 2019

Metadata

Assignee

None

Tags

No Tags

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.