Skip to content

fprintd should enroll to a user while not uid.#717

Closed
binli wants to merge 0 commit into
canonical:mainfrom
binli:main
Closed

fprintd should enroll to a user while not uid.#717
binli wants to merge 0 commit into
canonical:mainfrom
binli:main

Conversation

@binli
Copy link
Copy Markdown
Contributor

@binli binli commented Sep 12, 2023

Description

$ sudo env | grep SUDO
SUDO_COMMAND=/usr/bin/env
SUDO_USER=ubuntu
SUDO_UID=1001
SUDO_GID=1001

when running checkbox with sudo, it will 'fprintd-enroll 1001' and the test case will lock screen to let 'ubuntu' user to login, but at that time, it will not work because the enrolled user is not 'ubuntu'.

Resolved issues

Documentation

Tests

@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 12, 2023

Codecov Report

Merging #717 (fa7b5ad) into main (219a9de) will decrease coverage by 0.47%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #717      +/-   ##
==========================================
- Coverage   35.25%   34.78%   -0.47%     
==========================================
  Files         302      301       -1     
  Lines       34173    34141      -32     
  Branches     5907     5915       +8     
==========================================
- Hits        12048    11877     -171     
- Misses      21559    21718     +159     
+ Partials      566      546      -20     
Flag Coverage Δ
provider-base 4.16% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 45 files with indirect coverage changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant