Use type safe protobuf client in test framework

[MarkusPettersson98]

Re-write some code in the test framework to prefer the type safe wrapper around the Mullvad app gRPC client instead of its auto-generated dito.

ManagementServiceClient is automatically generated from the protobuf definitions found in management_interface.proto, and contains some very crude types. The MullvadProxyClient is a type-safe wrapper around ManagementServiceClient which performs conversions & validation of the data types from the gRPC server (the daemon) to their respective mappings in the talpid-* and mullvad-* crates. These types are more ergonomic to work with, and since we already have the conversions in place we should prefer those.

This PR also adds a clippy.toml to the test framework workspace. The only configuration for now is the lint disallowed-types which will warn on use of ManagementServiceClient.

---

This change is 

[linear]

DES-522 Use the client from `mullvad_management_interface::client`

[dlon]

Great stuff! 🥳

Reviewed 11 of 13 files at r1, 2 of 2 files at r2, 1 of 1 files at r3, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @MarkusPettersson98)

---

test/test-manager/src/tests/account.rs line 264 at r3 (raw file):

                tokio::time::sleep(THROTTLE_RETRY_DELAY).await;
            }
            Err(err) => return Err(err),

Does not truly matter, but it bothers me that we're using return here but break below for no good reason. 😅

---

test/test-manager/src/tests/mod.rs line 129 at r3 (raw file):

        .await
        .context("Could not clear split tunnel apps in cleanup")?;
    #[cfg(not(target_os = "macos"))]

This should only called on Linux.

[MarkusPettersson98]

Reviewable status: 12 of 14 files reviewed, 1 unresolved discussion (waiting on @dlon)

---

test/test-manager/src/tests/account.rs line 264 at r3 (raw file):

Previously, dlon (David Lönnhager) wrote…

Does not truly matter, but it bothers me that we're using return here but break below for no good reason. 😅

Yeah, changed this return to a break for consistency 😅

---

test/test-manager/src/tests/mod.rs line 129 at r3 (raw file):

Previously, dlon (David Lönnhager) wrote…

This should only called on Linux.

🤦
Fixed!

[dlon]

Reviewed 2 of 2 files at r4, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved

[dlon]

Reviewed 1 of 1 files at r5, 3 of 3 files at r6, 1 of 1 files at r7, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved