add some unit tests and automation

Author: gpeacock

.github/workflows/build.yml

@@ -0,0 +1,54 @@
+name: Build
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+
+jobs:
+  build-nightly-only:
+    name: Build nightly only
+    runs-on: ubuntu-latest
+    container: quay.io/pypa/manylinux2014_x86_64
+    steps:
+      - uses: actions/checkout@v1
+      - name: Install nightly rust
+        run: curl --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y --default-toolchain nightly
+      # Insert steps here to install you other dependencies. Note that you're running inside of cent os 7, not ubuntu
+      - name: Installing custom dependency
+        run: echo "Installing custom dependency"
+      - name: Build
+        run: |
+          source $HOME/.cargo/env
+          for PYBIN in /opt/python/cp3[6789]*/bin; do
+            "${PYBIN}/pip" install maturin
+            "${PYBIN}/maturin" build -m nightly-only/Cargo.toml -i "${PYBIN}/python" --release --manylinux 2014
+          done
+      # Auditwheel isn't generally necessary. This is only shown as refernce for cases where you link
+      # external libraries, in which case you have to set manylinux to off and then use auditwheel repair
+      - name: Auditwheel repair
+        run: |
+          for wheel in nightly-only/target/wheels/*.whl; do
+              auditwheel repair "${wheel}"
+          done
+
+
+  # This is similar to the above, except we're only using the lowest supported python version to build a single
+  # abi3 wheel
+  build-nightly-only-abi3:
+    name: Build nightly only abi3
+    runs-on: ubuntu-latest
+    container: quay.io/pypa/manylinux2014_x86_64
+    steps:
+      - uses: actions/checkout@v1
+      - name: Install nightly rust
+        run: curl --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y --default-toolchain nightly
+      - name: Install maturin
+        run: /opt/python/cp36-cp36m/bin/pip install maturin
+      - name: Build with maturin
+        run: |
+          source $HOME/.cargo/env
+          /opt/python/cp36-cp36m/bin/maturin build -m nightly-only-abi3/Cargo.toml -i /opt/python/cp36-cp36m/bin/python --release --manylinux 2014
+      - name: Auditwheel repair
+        run: auditwheel repair nightly-only-abi3/target/wheels/*.whl

.gitignore

@@ -3,6 +3,8 @@
 debug/
 target/
 
+venv/
+
 # Remove Cargo.lock from gitignore if creating an executable, leave it for libraries
 # More information here https://doc.rust-lang.org/cargo/guide/cargo-toml-vs-cargo-lock.html
 Cargo.lock
@@ -15,6 +17,7 @@ Cargo.lock
 
 # Python caches
 __pycache__/
+.pytest_cache/
 
 # Mac OS X files
 .DS_Store

src/json_api.rs

@@ -84,3 +84,25 @@ pub fn add_manifest_to_file_json(
     let signer = signer_info.signer()?;
     manifest.embed(&source, &dest, &*signer).map_err(Error::Sdk)
 }
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    /// returns a path to a file in the fixtures folder
+    pub fn test_path(path: &str) -> String {
+        let base = env!("CARGO_MANIFEST_DIR");
+        format!("{}/{}", base, path)
+    }
+
+    #[test]
+    fn test_verify_from_file() {
+        let path = test_path("tests/fixtures/C.jpg");
+        let result = verify_from_file_json(&path);
+        assert!(result.is_ok());
+        let json_report = result.unwrap();
+        println!("{}", json_report);
+        assert!(json_report.contains("C.jpg"));
+        //assert!(!json_report.contains("validation_status"));
+    }
+}

tests/fixtures/C.jpg

@@ -12,10 +12,21 @@
 # each license.
 
 import c2pa_python as c2pa
+import pytest
 
 def test_version():
     assert c2pa.version() == "0.1.0"
 
 def test_sdk_version():
     assert c2pa.sdk_version() == "0.25.2"
 
+
+#def test_verify_from_file():
+#    json_store = c2pa.verify_from_file_json("tests/fixtures/A.jpg") 
+#    assert not "validation_status" in json_store
+
+def test_verify_from_file_no_store():
+    with pytest.raises(c2pa.Error.Sdk) as err:  
+        json_store = c2pa.verify_from_file_json("tests/fixtures/A.jpg") 
+    assert str(err.value) == "no JUMBF data found"  
+ 

tests/test_c2pa.py

@@ -69,7 +69,8 @@ def getitem(d, key):
 
 allowed = True # opt out model, assume training is ok if the assertion doesn't exist
 try:
-    manifest_store = json.loads(c2pa.verify_from_file_json(testOutputFile))
+    json_store = c2pa.verify_from_file_json(testOutputFile)
+    manifest_store = json.loads(json_store)
 
     manifest = manifest_store["manifests"][manifest_store["active_manifest"]]
     for assertion in manifest["assertions"]: