Skip to content

Commit

Permalink
Update cli test build
Browse files Browse the repository at this point in the history
Add step to install tools in default path before running tests (#242).
  • Loading branch information
lgrz authored and elshize committed Nov 29, 2024
1 parent f3d8727 commit fffd1c9
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 9 deletions.
5 changes: 5 additions & 0 deletions .github/workflows/commit.yml
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,11 @@ jobs:
working-directory: ${{runner.workspace}}/build
run: cmake --build . --config Debug -- -j 4

- name: Install
shell: bash
working-directory: ${{runner.workspace}}/build
run: sudo make install

- name: Test CLI
shell: bash
working-directory: ${{runner.workspace}}/build
Expand Down
6 changes: 4 additions & 2 deletions test/cli/setup.sh
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,10 @@
# This script should be executed within the build directory that is directly
# in the project directory, e.g., /path/to/pisa/build

PISA_BIN="./bin"
export PATH="$PISA_BIN:$PATH"
command -v compress_inverted_index >/dev/null 2>&1 || {
echo >&2 "tools not available in default path"
exit 1
}

test_dir=${TEST_DIR:-../test}

Expand Down
2 changes: 0 additions & 2 deletions test/cli/test_count_postings.sh
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,6 @@

set +x

PISA_BIN="bin"
export PATH="$PISA_BIN:$PATH"
DIR=$(dirname "$0")

@test "Extract posting counts" {
Expand Down
2 changes: 0 additions & 2 deletions test/cli/test_taily_stats.sh
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,6 @@

set +x

PISA_BIN="bin"
export PATH="$PISA_BIN:$PATH"
DIR=$(dirname "$0")

echo_int () {
Expand Down
3 changes: 0 additions & 3 deletions test/cli/test_wand_data.sh
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,6 @@

set +x

PISA_BIN="bin"
export PATH="$PISA_BIN:$PATH"

queries=$(cat <<HERE
301:International Organized Crime
302:Poliomyelitis and Post-Polio
Expand Down

0 comments on commit fffd1c9

Please sign in to comment.