Commit graph

1395 commits

Author SHA1 Message Date
Bastien Nocera
b68fe26fc9 build: Require GIR to be created to run tests 2021-09-07 17:15:52 +02:00
Bastien Nocera
9035db94fc build: Add missing GLib log domains
The daemon sources and libupower-glib were built without their historic
log domains which meant some debug messages did not appear when running
upowerd in verbose mode.

This fixes the test_no_poll_batteries test.
2021-09-07 17:15:52 +02:00
Bastien Nocera
69a7cc39db ci: Fail CI on skipped tests
Parse the junit results and check whether there are any skipped tests.
2021-09-07 14:49:25 +02:00
Bastien Nocera
9e9e7dc857 ci: Always export the test logs
In case we want to read them even if successful (or skipped).
2021-09-07 14:24:12 +02:00
Bastien Nocera
02da4a2053 ci: Fix path to test log
It changed when we switched to meson.
2021-09-07 14:24:12 +02:00
Bastien Nocera
68584d33f4 linux: Split the integration test into individual tests
unittest_inspector.py lists the tests in the integration-test.py script,
which are then added as individual tests.
2021-09-07 11:42:24 +02:00
Bastien Nocera
20cb2e9610 linux: Rename integration test script
Add the python suffix.
2021-09-07 11:42:17 +02:00
pacien
c4cf2e579c etc: Document Time* unit 2021-09-07 09:27:20 +00:00
Bastien Nocera
9b93b6c402 build: Remove autotools 2021-09-06 14:06:40 +02:00
Bastien Nocera
6b9134c5d4 build: Clean up include_directories usage 2021-09-06 14:06:40 +02:00
Benjamin Berg
738d1002b0 meson: Depend on upowerd_dbus for the entire daemon 2021-09-06 14:06:40 +02:00
Benjamin Berg
02ab49963b meson: Fix header/source extraction from gdbus_codegen
Otherwise we depend on the C sources being ready rather than the headers
for building the rest of upowerd.
2021-09-06 14:06:40 +02:00
Bastien Nocera
55e4557425 ci: Ignore ABI changes for keyboard backlight symbols
We don't care that the up_exported_kbd_backlight_* symbols and functions
were removed from the libupower-glib ABI, they should never have been
exported in the first place as this API is only used server-side, in the
daemon.
2021-09-06 14:06:40 +02:00
Bastien Nocera
c0402747fd tools: Include top-level library include
So we don't need to define UP_COMPILATION
2021-09-02 17:47:30 +02:00
Benjamin Berg
6c28e2c52d ci: Build using meson 2021-09-02 17:47:30 +02:00
Benjamin Berg
f2e702d4dc build: Support building upower with meson 2021-09-02 17:47:30 +02:00
Benjamin Berg
46ed299451 tests: Return 77 when skipping tests
77 is the special value meaning that the test was skipped. Both meson
make check will display the information correctly.

Note that the test is currently executed directly in check-local. So add
a workaround to ignore the 77 error code and exit 0 instead in that
case.
2021-09-02 17:40:56 +02:00
Benjamin Berg
fbbcb0c03f build: Remove "deprecated" option
The functionality that this option was disabling/enabling has been
removed.
2021-09-02 17:40:43 +02:00
Benjamin Berg
95ee81fc6e build: Remove systemdutildir option
It appears that this option is not used (anymore).
2021-09-02 17:40:30 +02:00
Bastien Nocera
820031cd26 README: Update dependencies
libgudev got a version bump and libusb isn't used anymore.
2021-09-02 17:39:30 +02:00
Benjamin Berg
4b481113c5 ci: Remove libusb dependency
This was missed in commit 64582256a8 ("build: Remove libusb
dependency in Linux").
2021-09-02 17:39:07 +02:00
Bastien Nocera
bd59723fb8 lib: Mark device refresh function as deprecated
As the daemon API will soon be removed.

See https://gitlab.freedesktop.org/upower/upower/-/merge_requests/66
2021-08-19 18:57:35 +02:00
Bastien Nocera
4e962fdc8d lib: Mark lid related functions and properties as deprecated
As we want to eventually remove them.

See https://gitlab.freedesktop.org/upower/upower/-/merge_requests/5
2021-08-19 18:57:30 +02:00
Bastien Nocera
0f6cc0a10b 0.99.13 2021-08-17 15:55:59 +02:00
Bastien Nocera
b5a6e2491a tests: Run test suite in verbose mode by default
So we can see progress better.
2021-08-17 15:50:58 +02:00
Bastien Nocera
c5e73680dd tests: Export test log on failure 2021-08-17 14:47:25 +02:00
Dylan Van Assche
3acbedca26 tests: Test inhibitor lock for critical action
Inhibitor lock should be taken between the critical
action notification and the execution of the critical action.

Requires python-dbusmock > 0.23.1, test is skipped on lower versions.

python-dbusmock in the CI is installed from git and bumped version
to 0.23.2 until a new release is available.
2021-08-17 14:31:25 +02:00
Dylan Van Assche
0dc2a521be up-daemon: prevent suspending for critical action
Take inhibitor lock when notifying the user
about a critical battery level and release lock
when executing critical battery action.
2021-08-17 14:12:19 +02:00
Dylan Van Assche
40e5d237cf up-backend: add inhibitor lock interface
Phones are suspended most of the time so they are not awake for > 20s
to allow UPower to take action when battery is critical.
Add an interface to take and release inhibitor locks which
prevent the device from suspending to allow UPower to execute
the critical power action.
2021-08-17 14:12:19 +02:00
Bastien Nocera
e66c7da2d2 build: Require new gudev to fix battery detection
gudev 234 had bugs converting cached sysfs properties to boolean which
caused upower to think that batteries were not there, as the "present"
sysfs attribute was misread.

Require at least gudev 235 to avoid battery detection being broken.

Closes: #149
2021-08-10 12:59:16 +02:00
Bastien Nocera
26c90728c3 linux: Add test for touchpads being tagged as mice
udev adds both tags to touchpads, so replicate that behaviour.
2021-08-10 10:47:37 +00:00
Bastien Nocera
781e856974 linux: Fix touchpad not being the right type
Touchpads are also tagged as mice, so make sure that we check for
the touchpad property before checking for mouse one.
2021-08-10 10:47:37 +00:00
Bastien Nocera
3b91711e82 linux: Don't throw away large but possible energy rates
USB PD 3.1 allows up to 240W (48V, 5A) and some proprietary supplies
already delivered more than 100W over USB-C (USB PD 3.0 limit).

Closes: #147

Reported by StefanBruens
2021-08-10 12:32:24 +02:00
Bastien Nocera
a14de39241 etc: Tweak default percentage levels
If we want the computer to be able to take useful action about the low
battery, we should have a slightly higher "low" percentage level so that
power saving made really makes a difference in runtime.

Also bump "critical" slightly so that doom isn't quite as near but in the
distance nonetheless.

The "action" level stays the same, as 1% is too close to some batteries'
actual switch off point, eg. the computer might brown out before we see
1%.
2021-07-22 12:36:55 +02:00
Pablo Correa Gómez
988e79ae24 device: Don't update properties when device isn't exported
The code in up_device_notify() will still eventually be reached when the
up_device_coldplug() implementations are called, and properties are set
for the device type for the first time (rather than during instance
construction).
2021-06-30 10:56:31 +02:00
Pablo Correa Gómez
d8482bc767 tests: Add test for non-default low level 2021-06-30 10:56:25 +02:00
Pablo Correa Gómez
665a3b2757 daemon: Sync icon and warning for non-default low level
Before, the low level for changing the battery icon was hardcoded.
However, as the `low_percentage` property is settable by the user using
`PercentageLow` option. That can lead to inconsistencies when PercentageLow
is not the default. For example, if PercentageLow is set higher than 10,
the Low Battery level warning will be sent at the user-set level, but the
battery icon would not be updated to "caution" until the percentage
drops below 10%.

This issue is solved in this commit by using the `low_percentage` property
for the comparison instead of hardcoding the default.
2021-06-30 10:56:25 +02:00
Pablo Correa Gómez
9b2ab574e1
daemon: Make get_device_charge_icon() public 2021-06-18 13:08:59 +02:00
Philipp Zabel
b64902eed2 tests: Add a test case for batteries with zero power draw
Make sure that the issue reported in #7 and #44 is fixed.
The mocked battery has a zero power_now attribute and a non-zero legacy
current_now attribute on purpose, to detect if upowerd tries to read
current_now if the power_now value is small.
2021-06-17 18:15:01 +02:00
Philipp Zabel
2949136b15 linux: Fix < 0.01 W energy-rate readings from power_now sysfs property
Currently, if a power supplies' power_now sysfs file reports discharge
rates < 0.01 W, the code will try to calculate the discharge rate from
the legacy sysfs files. On new kernels where those don't exist, this
produces wrong results.

For example, on a dual-battery Thinkpad T450s, while the external
battery is discharging, the internal battery reports power_now = 0,
but the corresponding upower energy-rate field incorrectly reads
about 2.3 W.

This patch fixes the issue by falling back to the legacy code only if
the legacy current_now sysfs file exists.

Closes: #7, #44
2021-06-17 18:14:53 +02:00
Bastien Nocera
0f2837acde 0.99.12 2021-06-17 15:45:02 +02:00
Bastien Nocera
95763ef43e build: Force enable introspection for "make distcheck" to work
Introspection support is needed to be able to instatiate a UPClient
object in the test suite.
2021-06-17 15:45:02 +02:00
Bastien Nocera
0e9c6192ab ci: Export the tarball in the artifacts
Saves us from generating it locally.
2021-06-17 15:45:02 +02:00
Bastien Nocera
daf15b0f91 data: Also remove HID++ udev rules
Now that HID++ user-space support has been removed.
2021-06-17 14:37:25 +02:00
Bastien Nocera
fb431ad9ee ci: Ignore changes to "kind" enums 2021-06-17 13:47:33 +02:00
Bastien Nocera
96ea5b3521 linux: Make unknown Bluetooth devices appear as generic
Make Bluetooth devices for which we can't guess the "kind" or form
factor be "generic Bluetooth" devices, rather than "unknown" devices.

That should make it clearer in UIs that the devices are Bluetooth rather
than connected to the computer somehow.

Closes: #137
2021-06-17 13:47:33 +02:00
Bastien Nocera
2bddb64f68 lib: Add UP_DEVICE_KIND_BLUETOOTH_GENERIC type
This will be useful to show information about Bluetooth devices which
don't fit in with the existing types, and for which we don't want to
show an unknown kind.
2021-06-17 13:47:33 +02:00
Bastien Nocera
d2053eff93 linux: Remove user-space support for Logitech Unifying devices
All those devices have been supported in the Linux kernel for a number
of years already, so the user-space support has just not been exercised.
2021-06-17 12:49:59 +02:00
Bastien Nocera
b09846dbed linux: Remove sysfs-utils helpers
They've been replaced by gudev functions.
2021-06-17 12:02:42 +02:00
Bastien Nocera
0e3c511d2a linux: Use new uncached sysfs attr gudev API 2021-06-17 12:01:38 +02:00