Re: Wiki pages: Missing tests in %check and missing upstream test suites

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Feb 27, 2014 at 03:55:02PM +0200, Alexander Todorov wrote:
> Hi folks,
> thanks for your feedback in the last few days. I've created two wiki
> pages about packages which don't execute their tests in %check:
> https://fedoraproject.org/wiki/QA/Testing_in_check

Of the two packages that have 'virt' in their title, it looks like
virt-manager genuinely contains tests upstream but they are not
enabled in Fedora.  Actually some tests fail in Fedora 20.

Attached is a proposed patch to the spec file.  I ran this under
auto-buildrequires to see if it would need any extra BRs, but auto-br
didn't find any.

I didn't want to push it because I have approximately no idea what I'm
doing with python / virt-manager & tests.

> and another one for packages which don't seem to have test suites at all:
> https://fedoraproject.org/wiki/QA/Missing_upstream_test_suites

virt-viewer is mentioned in this list.  Not sure how that could be
meaningfully tested since it requires an X11 display and a network
connection to a VM.

Keep up the good work,

Rich.

-- 
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
virt-p2v converts physical machines to virtual machines.  Boot with a
live CD or over the network (PXE) and turn machines into KVM guests.
http://libguestfs.org/virt-v2v
From 179896a12622b596408405c9b7e6c2ae221636d1 Mon Sep 17 00:00:00 2001
From: "Richard W.M. Jones" <rjones@xxxxxxxxxx>
Date: Thu, 27 Feb 2014 14:11:59 +0000
Subject: [PATCH] Enable tests, but do not fail if they don't pass.

---
 virt-manager.spec | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/virt-manager.spec b/virt-manager.spec
index f4ca564..971839f 100644
--- a/virt-manager.spec
+++ b/virt-manager.spec
@@ -162,6 +162,10 @@ python setup.py install -O1 --root=$RPM_BUILD_ROOT
 %find_lang %{name}
 
 
+%check
+python setup.py test ||:
+
+
 %post
 /bin/touch --no-create %{_datadir}/icons/hicolor &>/dev/null || :
 /usr/bin/update-desktop-database &> /dev/null || :
-- 
1.8.5.3

--
packaging mailing list
packaging@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/packaging

[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite Forum]     [KDE Users]

  Powered by Linux