[Rpm-maint] [rpm-software-management/rpm] Eliminate the need for run() wrapper in the test-suite (PR #3447)

[Panu Matilainen]

Okay... So that "rpmdb --export and --import" test fails due to the --dbpath definition being removed. What's peculiar is that said test explicitly sets dbpath by itself: `--dbpath ${RPMTEST}/data/misc/`. But, due to the way the wrapper functions work, the dbpath set by the test doesn't get used, it gets overridden by the --dbpath in the run() wrapper. And when this PR removes the redundant path override, the one specified in the test gets used. So basically the [bug is in the test](https://programmerhumor.io/wp-content/uploads/2023/07/programmerhumor-io-testing-memes-debugging-memes-c242cb2722b41c1.jpg) (or the feature it tests), just masked by this other fun.

What makes it even more entertaining is that this PR passes "make check" and "make ci" for me locally, so it seems like one of those "only on Ubuntu" gotchas. I won't be able to finish this today, but I pushed the full PR back as originally submitted and converted to draft.


-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3447#issuecomment-2479016623
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/3447/c2479016623 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20241115/9f6bb0f0/attachment.html>