Skip to content

Commit

Permalink
test: Remove find workaround (#955)
Browse files Browse the repository at this point in the history
Now that `cds-test` walks symbolic links it is no longer needed to have
the `find` workaround. This allows now for simpler test file selection.

```
npm t -w sqlite SELECT.test.js           // runs /cds-dbs/sqlite/test/compliance/compliance/SELECT.test.js
npm t -w postgres bookshop/read.test.js  // runs /cds-dbs/postgres/test/compliance/scenarios/bookshop/read.test.js
npm t -w hana sflight                    // runs /cds-dbs/hana/test/compliance/scenarios/sflight/*.test.js
```
  • Loading branch information
BobdenOs authored Dec 18, 2024
1 parent 7379df1 commit 6aa7878
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 5 deletions.
2 changes: 1 addition & 1 deletion hana/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
"CHANGELOG.md"
],
"scripts": {
"test": "(([ -z \"${HANA_HOST}\" ] && npm start) || true) && cds-test $(../test/find)",
"test": "(([ -z \"${HANA_HOST}\" ] && npm start) || true) && cds-test",
"test:remote": "cds-test",
"start": "npm run start:hce || npm run start:hxe",
"start:hce": "cd ./tools/docker/hce/ && ./start.sh",
Expand Down
3 changes: 2 additions & 1 deletion postgres/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
"CHANGELOG.md"
],
"scripts": {
"test": "npm start && cds-test $(../test/find)",
"test": "npm start && cds-test",
"start": "docker compose -f pg-stack.yml up -d"
},
"dependencies": {
Expand Down Expand Up @@ -59,6 +59,7 @@
},
"postgres": {
"impl": "@cap-js/postgres",
"kind": "sqlite",
"dialect": "postgres",
"vcap": {
"label": "postgresql-db"
Expand Down
2 changes: 1 addition & 1 deletion sqlite/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
"CHANGELOG.md"
],
"scripts": {
"test": "cds-test $(../test/find)"
"test": "cds-test"
},
"dependencies": {
"@cap-js/db-service": "^1.14.1",
Expand Down
2 changes: 0 additions & 2 deletions test/find

This file was deleted.

0 comments on commit 6aa7878

Please sign in to comment.