Age | Commit message (Collapse) | Author | Files | Lines |
|
It was documented, but the configuration was never actually read.
https://bugs.freedesktop.org/show_bug.cgi?id=82925
|
|
https://bugs.freedesktop.org/show_bug.cgi?id=82925
|
|
This is better design to catching regressions and problems in a more
pin-pointed fashion. This also avoids having to restart the umockdev testbed
due to changing the environment variables with stop/start_daemon().
|
|
Python 3.4 changed the TestCase._outcome semantics, adjust accordingly.
|
|
This can happen in CI environments without a desktop session.
|
|
Signed-off-by: Richard Hughes <richard@hughsie.com>
|
|
up-self-test.c: In function ‘up_test_history_func’:
up-self-test.c:274:2: warning: implicit declaration of function ‘rmdir’ [-Wimplicit-function-declaration]
rmdir (history_dir);
|
|
Complete the polkit dependency removal by also dropping it from the self tests.
Unbreaks build and up-self-test again.
|
|
With the removal of the suspend/hibernate code, there's no longer
a need for upower to depend on polkit. This patch removes the old
polkit code.
Signed-off-by: Richard Hughes <richard@hughsie.com>
|
|
https://bugs.freedesktop.org/show_bug.cgi?id=80707
|
|
|
|
|
|
|
|
Removes the deprecated (for not very long) property, before we do a 1.0
release.
https://bugs.freedesktop.org/show_bug.cgi?id=78380
|
|
|
|
The IsDocked property has been incorrect for a number of laptops for a
while, as it thought that laptops with hybrid graphics cards were always
docked.
The alternative would have been to use the platform/dock_station
devices, but those are only exported for ACPI docking stations.
Instead, whether an external display is attached (which isn't really
docking) should be checked in the same place where the policy depending
on the value should be applied, such as gnome-settings-daemon.
https://bugs.freedesktop.org/show_bug.cgi?id=36818
|
|
On the Lenovo A600 all-in-one, there's a HCI/HID dual-mode Broadcom
device which will try to get the battery level of the supplied
mouse and keyboard, but will usually fail to do so.
So we have a battery of type "battery" (it's not either a mouse or
a keyboard, as it's supposed to handle both) but our display device
battery code doesn't check whether those batteries are power supplies
or not, using this "empty" (0% after the check timed out) battery in
the shutdown policy.
Check better for power supply devices, not all the "battery" types
are power supply batteries, they might be non-power supply ones
from unknown device types.
|
|
Signed-off-by: Richard Hughes <richard@hughsie.com>
|
|
In addition to build time - this increases compatibilty with OSTree,
which starts out with an empty /var.
Signed-off-by: Richard Hughes <richard@hughsie.com>
|
|
|
|
This has 2 effects:
- we never stop polling (we used to stop polling when the
battery was marked as fully-charged, but that can happen when
the battery is discharging too)
- when in an unknown state, we will poll through the daemon,
and through our unknown poll. This isn't a problem as there'll
another 115 seconds of wait until we get to poll through the
daemon again.
|
|
Next up is doing this for batteries, which need to switch between
a normal/slow poll to a faster one when the battery state is unknown.
|
|
|
|
And remove the gathering of the battery poll from the device
itself, it's of nearly no use.
|
|
|
|
To be used by all polling backends. This changes the defaults
to poll every 120 seconds when a warning level isn't reached, and
switch automatically to 30 seconds poll when the battery level is low.
|
|
Matching the work done in glib, gtk+, etc.
This also differentiates the unknown and normal timeouts in
the Linux power supply driver.
|
|
For system integrators. If your firmware is helpful to user space
and automatically sends out uevent when the battery level changes
(rather than just the battery state) as on most machines,
you can enable "NoPollBatteries" in the configuration option,
and reduce power consumption from UPower and its listeners.
|
|
In up_device_supply_refresh().
|
|
|
|
We cannot ever set the refresh timeout when we have a power line
device, so don't try and remove it there.
|
|
When the AC state changes, let the backends poll for battery changes.
They know better what's _really_ happening (whether the real state
is unknown even if they present a fully-charged battery), etc.
This is only possible because we fixed the Linux backend to poll
as it should always have.
|
|
5 times, with 1 second timeouts, instead of 30 times, with
2 seconds timeouts.
|
|
It was never actually setup, as the fallback state was used
to check whether we should use poll or not.
|
|
Some batteries report energy > energy_full and a percentage ("capacity"
attribute) > 100%. Clamp these within 0 and 100% for both plausibility as well
as to avoid setting an out-of-range property which would then become 0%.
https://launchpad.net/bugs/1240673
|
|
I don't think the kernel exports any numbers with a decimal
portion, but if they did, they would get the wrong values because
some locales use "," as the decimal separator, and not "." as the
kernel/C locale would.
|
|
If one battery is draining and the other one isn't, the time
to empty wouldn't be zero, but it would only match the time
to empty for the single battery.
Instead, ignore the accumulated time to empty/time to full
for multiple batteries and recalculate it.
https://bugzilla.gnome.org/show_bug.cgi?id=710344
|
|
When UPower crashes, we would never notice the battery going flat
because UPower clients aren't polling UPower, so won't autostart it.
Instead we rely on systemd to restart us when we crash. libupower-glib
also supports the client coming and going.
https://bugzilla.gnome.org/show_bug.cgi?id=682912
|
|
Update the expected warning levels to match, and add a big
fat FIXME for the test case itself. That's not how UPSes work,
or how UPower is expected to work.
|
|
If there are no LED class devices on the system, no need
to print a warning when we cannot open the directory.
|
|
Failure to refresh was supposed to be non-fatal, but since
we started putting objects on the bus *after* refresh, we
were skipping the registration if refresh failed, as is
the case in the UPS test case.
|
|
|
|
As was the case in other places, we need to be able to differentiate
warning messages for them to be useful.
|
|
No way to know which one was being called otherwise
|
|
Replace them all with WarningLevel tests.
Note we current crash when adding a UPS, in test_ups_ac.
|
|
|
|
|
|
In the integration tests.
|
|
As can happen with missing metadata, we might not have a time
to empty, so rely on the percentage instead.
|
|
|