search

abrt / retrace-server

Application for remote coredump analysis
GNU General Public License v2.0
40 stars 30 forks source link

Remove old test, create new one #429

Closed mgrabovsky closed 3 years ago

mgrabovsky commented 3 years ago

Remove an old attempt at an integration test, which didn't work reliably anyway. Create a few new unit tests for the Podman backend class.

Moreover, fix a bug found by the test. An off-by-one error made the short container ID one character too short.

mgrabovsky commented 3 years ago

There's something wrong with the tests in the EPEL 8 build, so let's keep this open for now. I'll get back to it later.

msrb commented 3 years ago

I was able to run the tests successfully on my Fedora 34 workstation (outside of RPM). I can try on RHEL 8 (+EPEL) as well.

msrb commented 3 years ago

So far, it's +1 from me. Let's test that EPEL further.

mgrabovsky commented 3 years ago

@msrb I've fixed the tests on EPEL (more precisely, for Python 3.8). Could you please provide your green check mark? :)