[Buildbot] #2872: No error is reported when suppressionFile does not exist

Buildbot trac trac at buildbot.net
Wed Feb 17 08:48:47 UTC 2016

#2872: No error is reported when suppressionFile does not exist
Reporter:  wentasah  |       Owner:  MikeLing
    Type:  defect    |      Status:  assigned
Priority:  minor     |   Milestone:  0.8.x
 Version:            |  Resolution:
Keywords:  simple    |

Comment (by MikeLing):

 Replying to [comment:8 dustin]:
 > Let's try to solve that first, by updating the unit test under
 `slave/buildslave/test` to correctly show the problem, and then applying
 your fix to show that it actually solves the problem.

 Yeah, but after read those tests in there, I found we already have the
 test with doesn't exist

 And after a second thought about it, I suddenly notice the
 itself should return 'No such file or directory:' error with an unexist
 file path, and it should been catch as an 'Exception' I think. Therefore,
 my fix in transfer.py is redundant code for it. :)

 So I'm wondering if we still need work on this issue? Maybe it had been
 fixed accidently? ;)

 Please tell me if I misunderstand something or went into wrong direction,
 thank you!

Ticket URL: <http://trac.buildbot.net/ticket/2872#comment:9>
Buildbot <http://buildbot.net/>
Buildbot: build/test automation

More information about the bugs mailing list