Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(tests): ioredis being able to successfully run tests, make it ready to be part of CI #459

Merged
merged 1 commit into from
Nov 6, 2022

Conversation

boazsade
Copy link
Contributor

@boazsade boazsade commented Nov 3, 2022

This commit do not include any changes in functionalities, but enable successfully running ioredis tests

  1. Add scripts that allow running ioredis using docker image
  2. some twinks to the docker file
  3. update to the readme.
    Please note that since ioredis has coverage for features that we are not supporting yet, using the script to run it on the container is the safest way to currently execute these tests right now. Also as we add more features, these scripts need to be updated accordingly.

Copy link
Contributor

@lsvmello lsvmello left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo fixes

tests/README.md Show resolved Hide resolved
tests/README.md Show resolved Hide resolved
tests/integration/run_ioredis_on_docker.sh Show resolved Hide resolved
@romange
Copy link
Collaborator

romange commented Nov 3, 2022

@lsvmello Thank you for your comments! please join our discord and say hello - I will add you to our #dev channel :)

@boazsade boazsade force-pushed the ioredis-howto branch 2 times, most recently from 9cb17ef to 0229211 Compare November 6, 2022 10:53
…ady to be part of CI

Signed-off-by: Boaz Sade <boaz@dragonflydb.io>
@romange romange merged commit 2ed4d34 into dragonflydb:main Nov 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants