Merge pull request #7 from joehoyle/test

Run tests as part of build
joehoyle · Jan 17, 2023 · def114b · def114b
2 parents 1b8dc40 + 3342673
commit def114b
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -64,7 +64,7 @@ jobs:
       - run: mkdir ~/.cargo-registry
 
       - name: Build
-        run: docker run --rm -v ~/.cargo-registry:/usr/local/cargo/registry -v $PWD:/code localhost:5000/phpv8js-build:latest bash -c 'rustup target add ${{ matrix.target }} ; cargo build --release --target ${{ matrix.target }}'
+        run: docker run --rm -v ~/.cargo-registry:/usr/local/cargo/registry -v $PWD:/code localhost:5000/phpv8js-build:latest bash -c 'rustup target add ${{ matrix.target }} ; cargo test --target ${{ matrix.target }} && cargo build --release --target ${{ matrix.target }}'
 
       - name: Rename file
         run: cp target/${{ matrix.target }}/release/libv8js.so php${{ matrix.php-version }}-${{ matrix.target }}-libv8js.so
@@ -118,10 +118,10 @@ jobs:
           override: true
       - name: Setup PHP version
         run: brew install php@${{ matrix.php-version }} && brew unlink php && brew link --force php@${{ matrix.php-version }}
+
       - name: Build
         uses: actions-rs/cargo@v1
         with:
-          use-cross: ${{ matrix.cross }}
           command: build
           args: --release --target ${{ matrix.target }} --lib