feat: add signing, tokio-1 to clippy test

feat: add signing to msrv check
fix: test_tcp_connections import order
revert: add default features on accident

.github/workflows/test.yml

@@ -23,7 +23,7 @@ jobs:
  components: clippy
  - uses: actions-rs-plus/clippy-check@v2
  with:
- args: --all --all-targets --features format-generated-code
+ args: --all --all-targets --features format-generated-code --features signing --features tokio-1
 
  internal-tests:
  runs-on: ubuntu-latest
@@ -59,7 +59,7 @@ jobs:
  with:
  use-cross: true
  command: check
- args: --all --all-targets
+ args: --all --all-targets --features signing
 
  build:
  needs: [formatting, linting, internal-tests, mavlink-dump, msrv]

mavlink-core/Cargo.toml

@@ -42,5 +42,4 @@ sha2 = { version = "0.10", optional = true }
 "serde" = ["dep:serde", "dep:serde_arrays"]
 "tokio-1" = ["dep:tokio"]
 "signing" = ["dep:sha2"]
-# DO NOT COMMIT 
-default = ["std", "tcp", "udp", "direct-serial", "serde", "tokio-1", "signing"]
+default = ["std", "tcp", "udp", "direct-serial", "serde"]

mavlink/tests/tcp_loopback_tests.rs

@@ -4,10 +4,10 @@ mod test_shared;
 mod test_tcp_connections {
  use std::thread;
 
- #[cfg(feature = "signing")]
- use mavlink::SigningConfig;
  #[cfg(feature = "signing")]
  use crate::test_shared;
+ #[cfg(feature = "signing")]
+ use mavlink::SigningConfig;
 
  /// Test whether we can send a message via TCP and receive it OK. This also test signing as a property of a MavConnection if the signing feature is enabled.
  #[test]