Merge #523: Fixing the Inclusivity of Z in StrKeyTranslator

76fb99c Fix the CI failing due to `which` crate (Harshil Jani)
ebe3245 Fixing the Inclusivity of Z in StrKeyTranslator (Harshil Jani)

Pull request description:

  Fixing `StrKeyTranslator` in `test_utils.rs` where Z was not included.
  Closes #496
  Signed-off-by: Harshil Jani <[email protected]>

ACKs for top commit:
  apoelstra:
    ACK 76fb99c

Tree-SHA512: 91f73014338c8943ab59bfc4c9e16de44ac6d752f2ce7cb3a84413fbf17bf47d17351ed20707b45ce9c554341a43d0b2faa4ab856d3e734275f5dfd626f4f515

Author: apoelstra

contrib/test.sh

@@ -16,6 +16,7 @@ if cargo --version | grep "1\.41\.0"; then
     cargo update -p form_urlencoded --precise 1.0.1
     cargo update -p once_cell --precise 1.13.1
     cargo update -p bzip2 --precise 0.4.2
+    cargo update -p which --precise 4.3.0
 fi
 
 # Pin dependencies required to build with Rust 1.47.0

src/test_utils.rs

@@ -136,7 +136,7 @@ impl StrKeyTranslator {
             .collect();
         let mut pk_map = HashMap::new();
         let mut pkh_map = HashMap::new();
-        for (i, c) in (b'A'..b'Z').enumerate() {
+        for (i, c) in (b'A'..=b'Z').enumerate() {
             let key = String::from_utf8(vec![c]).unwrap();
             pk_map.insert(key.clone(), pks[i]);
             pkh_map.insert(key, pks[i].to_pubkeyhash(SigType::Ecdsa));