Remove local-encoding as a test dependency

Summary:
Implementation usage of local-encoding was removed in D9447823, and we discussed removing the crate then. This is a very old, unmaintained crate and sapling is the only remaining user of it.

Remove local-encoding as a test dependency.

Reviewed By: zzl0

Differential Revision: D67541691

fbshipit-source-id: 9df3ac57947ec405a6998f55a16a35b52015751d

eden/scm/lib/encoding/Cargo.toml

@@ -18,9 +18,6 @@ sapling-types = { version = "0.1.0", path = "../types" }
 [target.'cfg(target_os = "windows")'.dependencies]
 winapi = { version = "0.3", features = ["everything", "std"] }
 
-[target.'cfg(target_os = "windows")'.dev-dependencies]
-local-encoding = "0.2"
-
 [target.'cfg(windows)'.dependencies]
 local-encoding = "*"
 winapi = "0.3"

eden/scm/lib/encoding/src/lib.rs

@@ -93,16 +93,6 @@ mod tests {
     fn test_utf8_roundtrip() {
         let bytes = b"\xE7\xAE\xA1\xE7\x90\x86\xE5\x91\x98\x2F\xE6\xA1\x8C\xE9\x9D\xA2";
 
-        #[cfg(windows)]
-        let bytes = {
-            use local_encoding::Encoder;
-            use local_encoding::Encoding;
-            match Encoding::ANSI.to_bytes(::std::str::from_utf8(bytes).expect("from_utf8")) {
-                Ok(s) => s,
-                _ => return, // Cannot be encoded using local encoding. Skip the test.
-            }
-        };
-
         check_roundtrip(&bytes[..]).expect("roundtrip");
     }