Skip to content

Commit

Permalink
ci: move when we check for genesis warning
Browse files Browse the repository at this point in the history
  • Loading branch information
joshuef authored and RolandSherwin committed May 21, 2024
1 parent 75f0066 commit 721852e
Showing 1 changed file with 19 additions and 17 deletions.
36 changes: 19 additions & 17 deletions .github/workflows/memcheck.yml
Original file line number Diff line number Diff line change
Expand Up @@ -92,23 +92,6 @@ jobs:
platform: ubuntu-latest
set-safe-peers: false

- name: Check we're warned about using default genesis
run: |
git log -1 --oneline
ls -la $RESTART_TEST_NODE_DATA_PATH
cat $RESTART_TEST_NODE_DATA_PATH/safenode.log
- name: Check we're warned about using default genesis
run: |
git log -1 --oneline
ls -la $BOOTSTRAP_NODE_DATA_PATH
cat $BOOTSTRAP_NODE_DATA_PATH/safenode.log
- name: Check we're warned about using default genesis
run: |
git log -1 --oneline
ls -la $NODE_DATA_PATH
cat $NODE_DATA_PATH/safenode.log
shell: bash
# In this case we did *not* want SAFE_PEERS to be set to another value by starting the testnet
- name: Check SAFE_PEERS was not changed
shell: bash
Expand Down Expand Up @@ -156,6 +139,25 @@ jobs:
SN_LOG: "all"
timeout-minutes: 25

# this check needs to be after some transfer activity
- name: Check we're warned about using default genesis
run: |
git log -1 --oneline
ls -la $RESTART_TEST_NODE_DATA_PATH
cat $RESTART_TEST_NODE_DATA_PATH/safenode.log
- name: Check we're warned about using default genesis
run: |
git log -1 --oneline
ls -la $BOOTSTRAP_NODE_DATA_PATH
cat $BOOTSTRAP_NODE_DATA_PATH/safenode.log
- name: Check we're warned about using default genesis
run: |
git log -1 --oneline
ls -la $NODE_DATA_PATH
rg "USING DEFAULT" "$NODE_DATA_PATH" -u
shell: bash

# Uploading same file using different client shall not incur any payment neither uploads
# Note rg will throw an error directly in case of failed to find a matching pattern.
- name: Start a different client to upload the same file
Expand Down

0 comments on commit 721852e

Please sign in to comment.