Skip to content
This repository was archived by the owner on Aug 1, 2023. It is now read-only.

Commit 5570539

Browse files
jflatowhayesgmCoburn Berrymaxwolffeverestmz
authored
Upgrade everything we can for a chain reset (#353)
* Re-upgrade Substrate WIP: CASH benchmarks aren't working for some reason (commented out) and test is failing due to another account minimal existence issue Also had to delete GrandpaRpcHandler, but hopefully not needed now? * Re-add Gateway addresses, which validators now have Validators are now required to have a minimum CASH balance in order to set session keys, and therefore validate. On committing a pipeline change, any account with a balance above the minimum is marked as existing in the Substrate account store. Any account with a balance below the minumum is purged from the account store. More test coverage needed. * Really use min balance path in tests * Attempt to patch issue blocking ingestion Only filter the blocks that were supported (not dissented), and only allow support or dissent for a given signer. The vote is being cast only on the support side, dissent is only derived if the chain receives a disagreement with the first one it received. * Test m10 patch * Cargo audit fixes Also upgrade substrate again in order to bump deps. This is using another substrate branch so as not to break the one others are using. Couldn't update cranelift-codegen without starting to break stuff in substrate. * Fix first build from integration * Start to build code substitutes * New and improved RPCs (#354) Co-authored-by: Coburn Berry <[email protected]> * Reorg fixes (#349) * Add two basic parser fuzzers and Fuzzbuzz integration (#320) * Add two basic parser fuzzers and Fuzzbuzz integration * Upgrade substrate / fix compilation Co-authored-by: Everest Munro-Zeisberger <[email protected]> * Update integration testing readme (#333) * Fix lock scen * Build Ethereum Contract to test Trx Request from Chain (#282) This patch adds a new contract (and a `TrxRequestBuilder`) which tests executing a trx request from on-chain. The builder is rudimentary, but can be the start of a full-time builder. The contract `Locker` just locks Eth and transfers it to you (the sender)'s account on Gateway. I believe this is currenty failing due to `(Transfer Cash Max 0x...)` failing right now. Note: the one bug discovered so far is that events from a trx are processed in reverse order. We should ensure we fix that in our new event system. (It was) * Bump rust version in CI and simplify scenario workflow * Use saddle 0.1.23 * test dissenting (#356) * Fix some tests * Mark failing tests for follow-up * Fix fetch_eth_blocks and formulate reorg issue (#357) * Fix fetch_eth_blocks and formulate reorg issue We don't need workers to abstain, since that creates different behaviors depending on where the 'invalid blockchain' is found. Also, avoid fetching duplicates in formulating the reorg. Also, really only fetch `slack` blocks, not `slack + 1`. * Fix dissent and concordance scenario * Cleanups (#359) * Fix up scenarios for moon branch * Temp work on scens * Start to diagnose all remaining scen issues * Make the re-org test consistent, if slightly less cool * Fix TOB #2 (#362) Co-authored-by: Geoffrey Hayes <[email protected]> Co-authored-by: Coburn Berry <[email protected]> Co-authored-by: Max Wolff <[email protected]> Co-authored-by: Everest Munro-Zeisberger <[email protected]> Co-authored-by: Wayne Nilsen <[email protected]> Co-authored-by: Coburn Berry <[email protected]>
1 parent 8d0191e commit 5570539

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

95 files changed

+15204
-4161
lines changed

Diff for: .github/workflows/test-workflow.yml

+16-22
Original file line numberDiff line numberDiff line change
@@ -4,23 +4,32 @@ on: push
44
# rustc version set with `rustup default` command in scripts/get_substrate.sh
55

66
jobs:
7+
cargo-audit:
8+
name: Cargo Audit Check
9+
runs-on: ubuntu-latest
10+
steps:
11+
- uses: actions/checkout@v2
12+
- uses: actions-rs/audit-check@v1
13+
with:
14+
token: ${{ secrets.GITHUB_TOKEN }}
15+
716
cargo-format:
817
name: Cargo Format Check
9-
runs-on: ubuntu-20.04
18+
runs-on: ubuntu-latest
1019
steps:
1120
- name: Checkout repository
1221
uses: actions/checkout@v2
1322

1423
- name: Cargo fmt
1524
run: |
16-
NIGHTLY=nightly-2021-03-24
25+
NIGHTLY=nightly-2021-06-01
1726
rustup install $NIGHTLY
1827
rustup component add rustfmt --toolchain $NIGHTLY
1928
cargo +$NIGHTLY fmt -- --check
2029
2130
substrate-tests:
2231
name: Substrate Unit Tests
23-
runs-on: ubuntu-20.04
32+
runs-on: ubuntu-latest
2433
steps:
2534
- uses: actions/checkout@v2
2635
# - uses: actions/cache@v2
@@ -74,7 +83,7 @@ jobs:
7483

7584
substrate-coverage:
7685
name: Substrate Coverage
77-
runs-on: ubuntu-20.04
86+
runs-on: ubuntu-latest
7887
steps:
7988
- uses: actions/checkout@v2
8089
# - uses: actions/cache@v2
@@ -220,32 +229,17 @@ jobs:
220229
run: |
221230
./scripts/get_substrate.sh --fast
222231
223-
- name: Install yarn ethereum dependencies
224-
run: cd ethereum && yarn install
225-
226-
- name: Install integration test ethereum dependencies
227-
run: cd integration && yarn install
228-
229232
- name: Install solc
230233
run: |
231234
sudo wget https://github.com/ethereum/solidity/releases/download/v0.8.1/solc-static-linux -O /usr/local/bin/solc
232235
sudo chmod +x /usr/local/bin/solc
233236
234-
- name: Compile Ethereum contracts
235-
run: |
236-
cd ethereum
237-
yarn compile
238-
239-
- name: Build Gateway
240-
run: |
241-
WASM_BUILD_RUSTFLAGS='--cfg feature="integration"' cargo build --release --features integration
237+
- name: Install integration test dependencies and build
238+
run: cd integration && yarn && yarn build
242239

243240
- name: Run Integration Test
244241
timeout-minutes: 40
245-
run: |
246-
set -eo pipefail
247-
cd integration
248-
sudo PROFILE=release QUIET_SCENARIOS=true npx jest --ci --reporters=default --reporters=jest-junit
242+
run: cd integration && sudo yarn test --ci --reporters=default --reporters=jest-junit
249243

250244
- name: Prepublish Integration results
251245
if: always()

0 commit comments

Comments
 (0)