Skip to content

Commit 5f21a30

Browse files
committed
Fix CI paths
1 parent abe6a65 commit 5f21a30

2 files changed

Lines changed: 23 additions & 31 deletions

File tree

.github/workflows/tests.yml

Lines changed: 22 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -707,28 +707,28 @@ jobs:
707707
cache-dependency-path: complement/go.sum
708708
go-version-file: complement/go.mod
709709

710-
# - name: Run Complement Tests
711-
# id: run_complement_tests
712-
# # -p=1: We're using `-p 1` to force the test packages to run serially as GHA boxes
713-
# # are underpowered and don't like running tons of Synapse instances at once.
714-
# # -json: Output JSON format so that gotestfmt can parse it.
715-
# #
716-
# # tee /tmp/gotest-complement.log: We tee the output to a file so that we can re-process it
717-
# # later on for better formatting with gotestfmt. But we still want the command
718-
# # to output to the terminal as it runs so we can see what's happening in
719-
# # real-time.
720-
# run: |
721-
# set -o pipefail
722-
# COMPLEMENT_DIR=`pwd`/complement synapse/scripts-dev/complement.sh -p 1 -json 2>&1 | tee /tmp/gotest-complement.log
723-
# shell: bash
724-
# env:
725-
# POSTGRES: ${{ (matrix.database == 'Postgres') && 1 || '' }}
726-
# WORKERS: ${{ (matrix.arrangement == 'workers') && 1 || '' }}
727-
728-
# - name: Formatted Complement test logs
729-
# # Always run this step if we attempted to run the Complement tests.
730-
# if: always() && steps.run_complement_tests.outcome != 'skipped'
731-
# run: cat /tmp/gotest-complement.log | gotestfmt -hide "successful-downloads,empty-packages"
710+
- name: Run Complement Tests
711+
id: run_complement_tests
712+
# -p=1: We're using `-p 1` to force the test packages to run serially as GHA boxes
713+
# are underpowered and don't like running tons of Synapse instances at once.
714+
# -json: Output JSON format so that gotestfmt can parse it.
715+
#
716+
# tee /tmp/gotest-complement.log: We tee the output to a file so that we can re-process it
717+
# later on for better formatting with gotestfmt. But we still want the command
718+
# to output to the terminal as it runs so we can see what's happening in
719+
# real-time.
720+
run: |
721+
set -o pipefail
722+
COMPLEMENT_DIR=`pwd`/complement synapse/scripts-dev/complement.sh -p 1 -json 2>&1 | tee /tmp/gotest-complement.log
723+
shell: bash
724+
env:
725+
POSTGRES: ${{ (matrix.database == 'Postgres') && 1 || '' }}
726+
WORKERS: ${{ (matrix.arrangement == 'workers') && 1 || '' }}
727+
728+
- name: Formatted Complement test logs
729+
# Always run this step if we attempted to run the Complement tests.
730+
if: always() && steps.run_complement_tests.outcome != 'skipped'
731+
run: cat /tmp/gotest-complement.log | gotestfmt -hide "successful-downloads,empty-packages"
732732

733733
- name: Run in-repo Complement Tests
734734
id: run_in_repo_complement_tests

scripts-dev/complement.sh

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -340,16 +340,8 @@ main() {
340340
set -x
341341

342342
if [ -n "$use_in_repo_tests" ]; then
343-
# TODO: Remove CI debug
344-
echo "Script path: $0"
345-
echo "Script dir: $(dirname "$0")"
346-
echo "Current directory: $(pwd)"
347-
echo "Looking for: $(dirname "$0")/../complement"
348-
ls -la "$(dirname "$0")/../complement"
349-
350343
# Run the suite of Complement tests in the `./complement` directory in this repo
351-
in_repo_test_suite=${SUITE:-$(realpath $(dirname $0)/../complement)}
352-
cd "$in_repo_test_suite"
344+
cd "./complement"
353345
go test "${test_args[@]}" "$@" "${default_in_repo_complement_test_packages[@]}"
354346
else
355347
# Run the tests (from the Complement repo)!

0 commit comments

Comments
 (0)