Bug 1076119 - PKG: Enable tests in %check
Summary: PKG: Enable tests in %check
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: systemd
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: systemd-maint
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-03-13 15:16 UTC by Alexander Todorov
Modified: 2015-02-18 21:33 UTC (History)
8 users (show)

Fixed In Version: systemd-219-3.fc22
Clone Of:
Environment:
Last Closed: 2015-02-18 21:33:10 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
PATCH (414 bytes, patch)
2014-03-13 15:16 UTC, Alexander Todorov
no flags Details | Diff
Patch with VERBOSE=1 (434 bytes, patch)
2014-03-17 09:20 UTC, Alexander Todorov
no flags Details | Diff

Description Alexander Todorov 2014-03-13 15:16:51 UTC
Created attachment 874027 [details]
PATCH

Description of problem:
Just make check in %check

Version-Release number of selected component (if applicable):
systemd-211-1.fc21.src.rpm

Comment 1 Zbigniew Jędrzejewski-Szmek 2014-03-15 04:42:49 UTC
Unfortunately 'make check' is rather fragile and doesn't pass in mock.

http://kojipkgs.fedoraproject.org//work/tasks/5499/6635499/build.log

I think it'd be great to enable tests, but they should be made more robust (by applying patches upstream) first.

Comment 2 Alexander Todorov 2014-03-17 08:34:44 UTC
I think it has something to do with D-Bus but I need to check. Is there any way we can get more logs from this build job? (e.g. test-suite.log if possible). 

Some test suites make it possible to print to stdout/stderr instead of a log file but I'm not sure about systemd's one.

Comment 3 Alexander Todorov 2014-03-17 09:20:16 UTC
Created attachment 875395 [details]
Patch with VERBOSE=1

This should cause all errors to be printed to stderr/stdout.

Comment 4 Alexander Todorov 2014-03-17 09:30:45 UTC
From ./src/libsystemd/sd-bus/test-bus-chat.c

    547         r = server_init(&bus);
    548         if (r < 0) {
    549                 log_info("Failed to connect to bus, skipping tests.");
    550                 return EXIT_TEST_SKIP;
    551         }


For some reason this succeeds on the build system but then the rest fails. On my test system I stopped DBus and this test was skipped instead of failing. 

Can you apply updated patch so we can see what breaks and where? Thanks.

Comment 5 Zbigniew Jędrzejewski-Szmek 2014-03-17 16:17:10 UTC
Here you go: http://koji.fedoraproject.org/koji/taskinfo?taskID=6640577.

http://kojipkgs.fedoraproject.org//work/tasks/579/6640579/build.log has the details. It looks like something that should be fixed...

BTW, you can build arbitrary scratch packages in koji without any privileges.

Comment 6 Alexander Todorov 2014-03-18 09:54:43 UTC
(In reply to Zbigniew Jędrzejewski-Szmek from comment #5)
>
> http://kojipkgs.fedoraproject.org//work/tasks/579/6640579/build.log has the
> details. It looks like something that should be fixed...
> 

Yeah, looks like these two test cases are not properly handling the fact that they can't open the bus. They should be skipped not fail. However this goes beyond my powers. 

Do you want a separate bug reported for it, or you will fix it here ?

Comment 7 Zbigniew Jędrzejewski-Szmek 2014-03-18 12:39:28 UTC
Here's enough.

Comment 8 Zbigniew Jędrzejewski-Szmek 2014-10-12 22:31:59 UTC
Down to one failure: http://koji.fedoraproject.org/koji/taskinfo?taskID=7841513.

Comment 10 Zbigniew Jędrzejewski-Szmek 2015-01-27 22:00:09 UTC
Tests still fail with 218. We should check again with 219.


Note You need to log in before you can comment on or make changes to this bug.