Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve build-test-smart-contracts CI #481

Open
abizjak opened this issue Nov 22, 2023 · 0 comments
Open

Improve build-test-smart-contracts CI #481

abizjak opened this issue Nov 22, 2023 · 0 comments
Labels
[Project] Smart Contract [Type] Task An additional feature or improvement.

Comments

@abizjak
Copy link
Contributor

abizjak commented Nov 22, 2023

Task description

There is the following TODO in the build-test-smart-contracts.yaml file

  "cargo_test_example_contracts":
    name: ${{ matrix.example-contract }} cargo:test
    runs-on: ubuntu-latest
    strategy:
      matrix:
        example-contract:
          - 'counter'
          # removed because test fail with "implement me"
          # - 'escrow'
          
          # the following 2 contracts fail to build due to:
          # error: building tests with panic=abort is not supported without `-Zpanic_abort_tests`
          # - 'fib'
          # - 'lockup'
          
          #Remaining example contracts have not yet been added to ci
    steps:
      - name: Checkout
        uses: actions/checkout@v4
      - name: Test
        working-directory: smart-contracts/rust-contracts/example-contracts/${{ matrix.example-contract }}
        run: |
          git config --global url."https://github.com/".insteadOf "git@github.com:"
          rustup default ${{ env.RUST_CLIPPY }}
          cargo test --target x86_64-unknown-linux-gnu

We should add the remaining example contracts to the list.

@abizjak abizjak added the [Type] Task An additional feature or improvement. label Nov 22, 2023
@abizjak abizjak mentioned this issue Nov 22, 2023
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Project] Smart Contract [Type] Task An additional feature or improvement.
Projects
None yet
Development

No branches or pull requests

2 participants