Add CI test.sh script

We now have two features that require testing in various combinations,
add a `contrib/test.sh` script to do the testing and use it in the
`Test` CI job. Add names to other steps in the job to improve clarity.

Author: tcharding

.github/workflows/rust.yml

@@ -3,7 +3,7 @@ on: [pull_request]
 name: Continuous Integration
 
 jobs:
-  test:
+  Test:
     name: Test Suite
     runs-on: ubuntu-latest
     strategy:
@@ -13,20 +13,17 @@ jobs:
           - stable
           - nightly
     steps:
+      - name: Checkout Crate
       - uses: actions/checkout@v2
+      - name: Checkout Toolchain
       - uses: actions-rs/toolchain@v1
         with:
           profile: minimal
           toolchain: ${{ matrix.rust }}
           override: true
-      - uses: actions-rs/cargo@v1
-        with:
-          command: test
-          args: --verbose --no-default-features --features strict alloc
-      - uses: actions-rs/cargo@v1
-        with:
-          command: test
-          args: --verbose --no-default-features --features strict std
+      - name: Run Test Script
+        env: ${{ matrix.rust }}
+        run: ./contrib/test.sh
 
   fmt:
     name: Rustfmt

contrib/test.sh

@@ -0,0 +1,25 @@
+#!/bin/sh
+#
+# CI test script for rust-bech32.
+#
+# The "strict" feature is used to configure cargo to deny all warnings, always use it in test runs.
+
+set -ex
+
+# Sanity, check tools exist.
+cargo --version
+rustc --version
+
+# Check without features ("strict" is a CI feature only, see above).
+cargo build --no-default-features --features="strict"
+cargo test --no-default-features --features="strict"
+
+# Check "std" feature (implies "alloc").
+cargo build --no-default-features --features="strict std"
+cargo test --no-default-features --features="strict std"
+
+# Check "alloc" feature alone.
+cargo build --no-default-features --features="strict alloc"
+cargo test --no-default-features --features="strict alloc"
+
+exit 0

src/lib.rs

@@ -67,9 +67,9 @@ extern crate core;
 use alloc::borrow::Cow;
 #[cfg(all(feature = "alloc", not(feature = "std")))]
 use alloc::{string::String, vec::Vec};
-use core::convert::TryFrom;
 #[cfg(feature = "alloc")]
 use core::convert::Infallible;
+use core::convert::TryFrom;
 use core::{fmt, mem};
 #[cfg(feature = "std")]
 use std::borrow::Cow;