Florian Festi wrote:
Tim Lauridsen wrote:
Tim Lauridsen wrote:
James Bowes wrote:
While fiddling with some test cases for package removal, I ran into
some
problems with transactioninfo. It looks at the class of a package to
determine if its from the rpmdb, from a repo, or a local rpm. This
makes
testing with fake packages hard.
The attached patch switches from looking at the class type to adding a
package_state variable to packages, which can be easily set for
testing.
I don't think it will break any code using yum out in the wild.
Thoughts?
-James
------------------------------------------------------------------------
_______________________________________________
Yum-devel mailing list
[email protected]
https://lists.dulug.duke.edu/mailman/listinfo/yum-devel
Looks sane to me
Tim
_______________________________________________
Yum-devel mailing list
[email protected]
https://lists.dulug.duke.edu/mailman/listinfo/yum-devel
Since nobody have complained loudly :)
Can you then please commit it, it is very usefull for some of my
skip-broken testcases.
Ehm, sorry. I should have spoken up on the list earlier...
I have an alternative fix for this problem. My solution has the
advantage that we don't need another Package class so we can use POs
more flexible in the test code while James Bowes' solution is a bit less
hacky. I did not yet manage to discuss with James which solution to
choose - so I put that issue on the list now.
Florian
------------------------------------------------------------------------
_______________________________________________
Yum-devel mailing list
[email protected]
https://lists.dulug.duke.edu/mailman/listinfo/yum-devel
I have tested this one, it look like it make erase test cases work right.
Tim
_______________________________________________
Yum-devel mailing list
[email protected]
https://lists.dulug.duke.edu/mailman/listinfo/yum-devel
