fix: do not run tests that require root

Do not run tests that require root. The tests
requiring superuser privileges are marked as ignored
and will display a message on how to actually run
those tests, this is with:
```
[root@localhost]# cargo test -- --ignored
```

An additional test is added to check the
effective user is actually root before trying
to run the commands.

Signed-off-by: Miguel Martín <[email protected]>

.github/workflows/ci.yml

@@ -193,9 +193,9 @@ jobs:
         run: |
           docker run -d -v `pwd`:/code:z --workdir /code --user root -e SQLITE_MANUFACTURER_DATABASE_URL='../ci-manufacturer-db.sqlite' -e SQLITE_OWNER_DATABASE_URL='../ci-owner-db.sqlite' -e SQLITE_RENDEZVOUS_DATABASE_URL='../ci-rendezvous-db.sqlite' --name tests devcontainer-fdo-rs sleep infinity
           docker exec --user root tests cargo build --lib --bins --workspace --verbose
-          docker exec --user root tests diesel migration run --migration-dir ./migrations/migrations_manufacturing_server_sqlite --database-url ./ci-manufacturer-db.sqlite 
-          docker exec --user root tests diesel migration run --migration-dir ./migrations/migrations_owner_onboarding_server_sqlite --database-url ./ci-owner-db.sqlite 
+          docker exec --user root tests diesel migration run --migration-dir ./migrations/migrations_manufacturing_server_sqlite --database-url ./ci-manufacturer-db.sqlite
+          docker exec --user root tests diesel migration run --migration-dir ./migrations/migrations_owner_onboarding_server_sqlite --database-url ./ci-owner-db.sqlite
           docker exec --user root tests diesel migration run --migration-dir ./migrations/migrations_rendezvous_server_sqlite --database-url ./ci-rendezvous-db.sqlite
-          docker exec --user root tests cargo test
+          docker exec --user root tests cargo test -- --ignored
           docker stop tests
           docker rm tests

client-linuxapp/src/serviceinfo.rs

@@ -838,6 +838,7 @@ mod test {
     use super::BinaryFileInProgress;
 
     use crate::serviceinfo::*;
+    use fdo_util::system_info::get_current_user_name;
 
     #[test]
     fn test_binaryfileinprogress_destination_path() {
@@ -870,7 +871,10 @@ mod test {
     }
 
     #[test]
+    #[ignore = "to run this test you must be root and run `cargo test -- --ignored`"]
     fn test_user_creation_no_pw() {
+        let current_user_name = get_current_user_name();
+        assert_eq!(current_user_name, "root");
         let test_user = "test";
         assert!(create_user(test_user).is_ok());
         let empty_user = "";
@@ -884,7 +888,10 @@ mod test {
     }
 
     #[test]
+    #[ignore = "to run this test you must be root and run `cargo test -- --ignored`"]
     fn test_user_creation_with_pw() {
+        let current_user_name = get_current_user_name();
+        assert_eq!(current_user_name, "root");
         let test_user = "testb";
         let test_password = "password";
         assert!(create_user_with_password(test_user, test_password).is_ok());