chore: prune mockito dependencies (#7991)

We don't really need the color feature. Removing it has several
benifits:

- less dependencies
- one less MPL license
- no `atty` (only used by tests but has a security bug)

Cargo.lock

@@ -517,17 +517,6 @@ dependencies = [
  "num-traits",
 ]
 
-[[package]]
-name = "atty"
-version = "0.2.14"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "d9b39be18770d11421cdb1b9947a45dd3f37e93092cbf377614828a319d5fee8"
-dependencies = [
- "hermit-abi 0.1.19",
- "libc",
- "winapi 0.3.9",
-]
-
 [[package]]
 name = "authz"
 version = "0.1.0"
@@ -983,17 +972,6 @@ version = "1.0.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "acbf1af155f9b9ef647e42cdc158db4b64a1b61f743629225fde6f3e0be2a7c7"
 
-[[package]]
-name = "colored"
-version = "2.0.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "b3616f750b84d8f0de8a58bda93e08e2a81ad3f523089b05f1dffecab48c6cbd"
-dependencies = [
- "atty",
- "lazy_static",
- "winapi 0.3.9",
-]
-
 [[package]]
 name = "comfy-table"
 version = "6.2.0"
@@ -2270,15 +2248,6 @@ dependencies = [
  "unicode-segmentation",
 ]
 
-[[package]]
-name = "hermit-abi"
-version = "0.1.19"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "62b467343b94ba476dcb2500d242dadbb39557df889310ac77c5d99100aaac33"
-dependencies = [
- "libc",
-]
-
 [[package]]
 name = "hermit-abi"
 version = "0.2.6"
@@ -3611,7 +3580,6 @@ source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "ea57936ab3bf56156f135f20ee24b840e5a8ad97a8e1710eace33ac078f8f537"
 dependencies = [
  "assert-json-diff",
- "colored",
  "futures",
  "hyper",
  "lazy_static",

client_util/Cargo.toml

@@ -16,4 +16,4 @@ workspace-hack = { version = "0.1", path = "../workspace-hack" }
 
 [dev-dependencies]
 tokio = { version = "1.28", features = ["macros", "parking_lot", "rt-multi-thread"] }
-mockito = "1.0"
+mockito = { version = "1.0", default-features = false }

deny.toml

@@ -7,11 +7,6 @@ yanked = "deny"
 unmaintained = "warn"
 notice = "warn"
 ignore = [
-    # potential unalinged read in atty
-    # https://rustsec.org/advisories/RUSTSEC-2021-0145
-    # Acceptable because only dependencies are dev/test (not used in prod code).
-    "RUSTSEC-2021-0145",
-
     # "It was sometimes possible for SQLite versions >= 1.0.12, < 3.39.2 to allow an array-bounds overflow when large
     # string were input into SQLite's printf function."
     #
@@ -32,8 +27,6 @@ default = "deny"
 exceptions = [
     # We should probably NOT bundle CA certs but use the OS ones.
     { name = "webpki-roots", allow = ["MPL-2.0"] },
-    # Currently needed for testing only via mockito.
-    { name = "colored", allow = ["MPL-2.0"] },
 ]
 
 [[licenses.clarify]]

influxdb2_client/Cargo.toml

@@ -16,7 +16,7 @@ url = "2.4.0"
 uuid = { version = "1", features = ["v4"] }
 
 [dev-dependencies] # In alphabetical order
-mockito = "1.0"
+mockito = { version ="1.0", default-features = false }
 once_cell = { version = "1.18", features = ["parking_lot"] }
 parking_lot = "0.12"
 tokio = { version = "1.28", features = ["macros", "parking_lot", "rt-multi-thread", "sync", "time"] }