agama-project / agama / 21002070733

73%

Build Type

push

github

Committed by web-flow

Commit Message

Adapt the "authentication" section of the web UI to the new API (#3007)

## Problem

#2937 introduced a new API to configure authentication (root and the
first non-root user). But the web interface is still not adapted

## Solution

This adapts the web interface to use the new API.

It also adds a new entry about Authentication for the installation
summary, making it possible to reach the corresponding section of the
UI.

## Hack alert

To make sure the UI is refreshed after discarding the first user, the
page displaying that user gets the information from the proposal,
instead of getting it from the configuration.

Once config is fixed in the backend to also emit events on every change
that could be reverted. It is done in a separate commit so ease the
task.

## Bonus

This pull request also include fixes for RegistrationSummary and
StorageSumary

## Disclaimer

The `hasIssues` flag of `UsersSummary` is based on the existence of
issues in the scope. Currently such issues are only generated when the
service is started. If the configuration is then fixed (let's say a
password is set for root) and broken again (let's say the root password
is removed afterwards), then the issue will not be re-generated and the
interface will not display the corresponding warning icon.

That needs to be fixed in the backend.

## Testing

- Unit tests adapted. Unit tests that were commented out were brought
back to life.
- Tested manually

## Screenshots

<img width="1021" height="556" alt="users1b"
src="https://github.com/user-attachments/assets/a29019ae-5c5d-4504-be0e-c82cb2ac11c0"
/>

<img width="1017" height="575" alt="users2b"
src="https://github.com/user-attachments/assets/fc7d3aed-47ad-4f12-987c-d3ba7a606e28"
/>

<img width="479" height="674" alt="users-error"
src="https://github.com/user-attachments/assets/2b1e4a0a-e512-4ebd-bf5e-590ead4b5bf0"
/>

Run Details

2713 of 5078 branches covered (53.43%)

Branch coverage included in aggregate %.

50 of 79 new or added lines in 9 files covered. (63.29%)

53 existing lines in 4 files now uncovered.

20048 of 26155 relevant lines covered (76.65%)

60.78 hits per line