Skip to content

Commit

Permalink
test fix
Browse files Browse the repository at this point in the history
  • Loading branch information
DomPeliniAerospike committed Oct 21, 2024
1 parent b706f93 commit 9bbd1ab
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 9 deletions.
4 changes: 1 addition & 3 deletions .github/assets/aerospike.conf
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
service {
cluster-name vector-search
feature-key-file /etc/aerospike/features.conf
cluster-name nodejs-client
}
logging {
console {
Expand All @@ -24,7 +23,6 @@ network {
port 3001
}
}

namespace test {
allow-ttl-without-nsup true
replication-factor 2
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/build-bindings.yml
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ jobs:
if: ${{ !contains(github.event.pull_request.labels.*.name, 'new-server-features') }}
run: |
npm i @types/mocha @types/yargs @types/semver @types/chai
docker run -d --name aerospike -p 3000-3002:3000-3002 aerospike/aerospike-server
run: docker run -d -v $(pwd)/.github/assets/aerospike.conf:/etc/mail/aerospike.conf --name aerospike -p 3000-3002:3000-3002 aerospike/aerospike-server --config-file /etc/mail/aerospike.conf
Expand Down
10 changes: 5 additions & 5 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -217,7 +217,7 @@ jobs:

- name: Run Aerospike server
if: ${{ !contains(github.event.pull_request.labels.*.name, 'new-server-features') }}
run: docker run -d -v $(pwd)/.github/assets/aerospike.conf:/etc/aerospike/aerospike.conf --name aerospike -p 3000-3002:3000-3002 aerospike/aerospike-server --config-file /etc/aerospike/aerospike.conf
run: docker run -d -v $(pwd)/.github/assets/aerospike.conf:/etc/mail/aerospike.conf --name aerospike -p 3000-3002:3000-3002 aerospike/aerospike-server --config-file /etc/mail/aerospike.conf

- name: Wait for database to be ready
# Should be ready after 3 seconds
Expand All @@ -229,7 +229,7 @@ jobs:
npm install typescript --save-dev;
npx tsc;
cd ..;
npm run ts-test
npm run ts-test dist/
test-lowest-supported-server:
runs-on: ubuntu-latest
Expand Down Expand Up @@ -275,7 +275,7 @@ jobs:
npm install typescript --save-dev;
npx tsc;
cd ..;
npm run ts-test -- --t 20000
npm run ts-test dist/ -- --t 20000
test-ee:
runs-on: ubuntu-latest
Expand Down Expand Up @@ -332,7 +332,7 @@ jobs:
npm install typescript --save-dev;
npx tsc;
cd ..;
npm run ts-test tests/admin.js -- --h localhost --U admin --P admin --t 40000
npm run ts-test dist/admin.js -- --h localhost --U admin --P admin --t 40000
test-valgrind:
runs-on: ubuntu-latest
Expand Down Expand Up @@ -387,7 +387,7 @@ jobs:

- name: Run Aerospike server
if: ${{ !contains(github.event.pull_request.labels.*.name, 'new-server-features') }}
run: docker run -d -v $(pwd)/.github/assets/aerospike.conf:/etc/aerospike/aerospike.conf --name aerospike -p 3000-3002:3000-3002 aerospike/aerospike-server
run: docker run -d -v $(pwd)/.github/assets/aerospike.conf:/etc/mail/aerospike.conf --name aerospike -p 3000-3002:3000-3002 aerospike/aerospike-server --config-file /etc/mail/aerospike.conf

- name: Wait for database to be ready
# Should be ready after 3 seconds
Expand Down

0 comments on commit 9bbd1ab

Please sign in to comment.