Skip to content

Commit

Permalink
use dockers for tests, use npm workspaces, add rejson & redisearch mo…
Browse files Browse the repository at this point in the history
…dules, fix some bugs
  • Loading branch information
leibale committed Nov 9, 2021
1 parent ecbd5b6 commit 3eb99db
Show file tree
Hide file tree
Showing 689 changed files with 5,321 additions and 1,712 deletions.
1 change: 0 additions & 1 deletion .github/release-drafter-config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,4 +41,3 @@ template: |
We'd like to thank all the contributors who worked on this release!
$CONTRIBUTORS
46 changes: 0 additions & 46 deletions .github/workflows/benchmark.yml

This file was deleted.

3 changes: 3 additions & 0 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,9 @@ jobs:
- name: Install Packages
run: npm ci

- name: Build tests tools
run: npm run build:tests-tools

- name: Run Tests
run: npm run test -- --forbid-only --redis-version=${{ matrix.redis-version }}

Expand Down
9 changes: 4 additions & 5 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
.vscode/
.idea/
node_modules/
dist/
.nyc_output/
.vscode/
coverage/
dump.rdb
documentation/
dist/
node_modules/
.DS_Store
dump.rdb
20 changes: 0 additions & 20 deletions .npmignore

This file was deleted.

4 changes: 0 additions & 4 deletions .nycrc.json

This file was deleted.

5 changes: 1 addition & 4 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -47,10 +47,7 @@ Node Redis has a full test suite with coverage setup.

To run the tests, run `npm install` to install dependencies, then run `npm test`.

Note that the test suite assumes that a few tools are installed in your environment, such as:

- redis (make sure redis-server is not running when starting the tests, it's part of the test-suite to start it and you'll end up with a "port already in use" error)
- stunnel (for TLS tests)
Note that the test suite assumes that [`docker`](https://www.docker.com/) is installed in your environment.

### Submitting Code for Review

Expand Down
Loading

0 comments on commit 3eb99db

Please sign in to comment.