Skip to content

Conversation

@cjihrig
Copy link
Contributor

@cjihrig cjihrig commented Aug 22, 2020

No description provided.

@geek
Copy link
Member

geek commented Aug 23, 2020

Looks like the message returned from redis changed, can probably update

expect(consoleMessage).to.contain('Redis server does not require a password, but a password was supplied');
to something like: "does not require a password, but a password was supplied"

@cjihrig cjihrig force-pushed the master branch 2 times, most recently from abebbab to 578f586 Compare August 23, 2020 21:13
This commit moves more of the redis setup into the docker-compose file.
@cjihrig
Copy link
Contributor Author

cjihrig commented Aug 24, 2020

@geek I updated the test whose error message had changed. I also got rid of the Travis dependency on redis by pushing everything into the docker-compose. As a result, I ended up mocking the one Unix Domain Socket test because Docker didn't seem to allow the host and container to share the same UDS.

This commit updates the only test that uses Unix Domain Sockets to connect to a mock Node.js server instead of a Redis container.
@lloydbenson
Copy link
Contributor

This is great work thanks so much @cjihrig.

@lloydbenson lloydbenson self-requested a review August 24, 2020 15:15
@cjihrig cjihrig merged commit 038bc68 into hapijs:master Aug 24, 2020
@cjihrig cjihrig added this to the 6.1.1 milestone Sep 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants