Bug 62859

Summary:	test-unit-file: fails due to assertion
Product:	systemd	Reporter:	Michał Górny <mgorny>
Component:	general	Assignee:	systemd-bugs
Status:	RESOLVED WONTFIX	QA Contact:	systemd-bugs
Severity:	normal
Priority:	medium
Version:	unspecified
Hardware:	Other
OS:	All
Whiteboard:
i915 platform:		i915 features:

Description Michał Górny 2013-03-28 10:43:36 UTC

./build-aux/test-driver: line 95: 31794 Aborted                 "$@" > $log_file 2>&1
FAIL: test-unit-file

$ cat test-unit-file.log 
unit_file_get_list: Permission denied
Assertion 'r >= 0' failed at src/test/test-unit-file.c:50, function test_unit_file_get_set(). Aborting.

AFAICS it should be skipped instead due to insufficient permissions.

Comment 1 Zbigniew Jedrzejewski-Szmek 2013-08-04 04:05:00 UTC

Since http://cgit.freedesktop.org/systemd/systemd/commit/?id=751e757 we return error without dumping core. This is really a misconfiguration in the environment, so a legitimate failure.

Comment 2 Michał Górny 2013-08-04 07:48:31 UTC

What's exactly the 'misconfiguration'? Are people expected to run tests as root?

Comment 3 Zbigniew Jedrzejewski-Szmek 2013-08-04 12:36:12 UTC

(In reply to comment #2)
> What's exactly the 'misconfiguration'? Are people expected to run tests as
> root?
No, people are not expected to have unreadable unit files. systemd will serve their contents over dbus anyway, so it's unlikely that chmod'ding files does what people who do it expect.

Comment 4 Michał Górny 2013-08-04 13:00:37 UTC

Well, you could say straight that it fails because file in /etc/systemd is not world-readable. Not that tests which rely on system configuration of *build host* are any good...

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.