rhelmer/socorro — comment on commit f831575c70cd6336177ee35f31422f69cf258322
File: socorro/unittest/cron/testFtpScraper.py line 37
By peterbe:

I much prefer this:

```
except:
import sys, traceback
traceback.print_exception(*sys.exc_info())
connection.rollback()
```

However, why would you want to proceed at all with the tests if you're not able to insert the fixtures? Should it let the exception be raised just as is and alerts the developer (or test runner) to deal with it. Something like this:

```
for b in fakeBuildsData:
cursor.execute(...)
connection.commit()
````

Then any error is simply dealt with there and then and you won't get half-committed junk in the test db.