Merge pull request #26 from goto-opensource/feature/hendriks/packaging-check

ci: run cargo-package to be safe

Author: Tobias Waurick

.github/workflows/build_test.yml

@@ -35,6 +35,12 @@ jobs:
           command: check
           args: --benches
 
+      - name: check (tests)
+        uses: actions-rs/cargo@v1
+        with:
+          command: check
+          args: --tests
+
       - name: test
         uses: actions-rs/cargo@v1
         with:
@@ -45,6 +51,13 @@ jobs:
         with:
           command: clippy
           args: --all-targets --all-features -- -Dwarnings
+
+      # test packaging to avoid surprisis at release time
+      - name: package
+        uses: actions-rs/cargo@v1
+        with:
+          command: package
+
 
   build-wasm:
     name: build wasm32

Cargo.toml

@@ -29,7 +29,10 @@ criterion = { version= "0.4", features=["html_reports"] }
 hex = "0.4"
 pretty_assertions = "1.3"
 rand = "0.8"
-test_vectors = { version = "0.1.0", path = "utils/test_vectors" }
+serde_json = "1.0"
+serde = { version = "1.0", features = ["derive"] }
+lazy_static = "1.4.0"
+
 
 [features]
 default = []

src/crypto/aead.rs

@@ -169,7 +169,7 @@ mod test {
         }
 
         mod test_vectors {
-            use test_vectors::get_test_vector;
+            use crate::test_vectors::get_test_vector;
 
             use crate::{
                 crypto::{

src/crypto/key_expansion.rs

@@ -68,7 +68,7 @@ mod ring {
 
 #[cfg(test)]
 mod test {
-    use test_vectors::get_test_vector;
+    use crate::test_vectors::get_test_vector;
 
     use crate::{
         crypto::{

src/crypto/secret.rs

@@ -27,7 +27,7 @@ impl Secret {
 
 #[cfg(test)]
 mod test {
-    use test_vectors::get_test_vector;
+    use crate::test_vectors::get_test_vector;
 
     use crate::{
         crypto::{cipher_suite::CipherSuiteVariant, key_expansion::KeyMaterial},

src/header/mod.rs

@@ -253,7 +253,7 @@ mod test {
 
     #[test]
     fn serialize_test_vectors() {
-        test_vectors::get_test_vector(crate::CipherSuiteVariant::AesGcm128Sha256 as u8)
+        crate::test_vectors::get_test_vector(crate::CipherSuiteVariant::AesGcm128Sha256 as u8)
             .encryptions
             .iter()
             .for_each(|test_vector| {
@@ -267,7 +267,7 @@ mod test {
 
     #[test]
     fn deserialize_test_vectors() {
-        test_vectors::get_test_vector(crate::CipherSuiteVariant::AesGcm256Sha512 as u8)
+        crate::test_vectors::get_test_vector(crate::CipherSuiteVariant::AesGcm256Sha512 as u8)
             .encryptions
             .iter()
             .for_each(|test_vector| {

src/lib.rs

@@ -33,3 +33,6 @@ pub mod header;
 pub mod receiver;
 pub mod sender;
 pub use crypto::cipher_suite::CipherSuiteVariant;
+
+#[cfg(test)]
+pub mod test_vectors;

utils/test_vectors/src/lib.rs renamed to src/test_vectors/mod.rs

@@ -1,5 +1,6 @@
 // Copyright (c) 2023 GoTo Group, Inc
 // SPDX-License-Identifier: Apache-2.0 AND MIT
+#![allow(clippy::missing_panics_doc)]
 
 extern crate serde;
 
@@ -61,7 +62,7 @@ fn parse_test_vectors() -> Vec<TestVector> {
 
 #[cfg(test)]
 mod test {
-    use crate::get_test_vector;
+    use super::get_test_vector;
 
     #[test]
     fn should_parse_test_vectors() {