Skip to content
This repository was archived by the owner on Jul 27, 2023. It is now read-only.

Fixes #207 - notify listeners even if key doesn't exists in consul #232

Merged
merged 1 commit into from
Apr 27, 2017

Conversation

maqdev
Copy link
Contributor

@maqdev maqdev commented Apr 27, 2017

No description provided.

Verified

This commit was signed with the committer’s verified signature.
jrjohnson Jon Johnson
…onsul
@maqdev
Copy link
Contributor Author

maqdev commented Apr 27, 2017

I don't know why, but mvn doesn't executes all tests. It reports only Tests run: 58, Failures: 0, Errors: 0, Skipped: 10 however if I run in IDEA there is 97 tests, and 10 are failing.

@rickfast
Copy link
Owner

Idea might not respect @Ignore. Either way, it's no problem

@rickfast rickfast merged commit 18a8f45 into rickfast:master Apr 27, 2017
@maqdev
Copy link
Contributor Author

maqdev commented Apr 27, 2017

@rickfast it seems that mvn only executes top-level tests. For example ConsulCacheTest isn't executed and are failing already.
I'm trying to add more tests for ServiceHealthCache

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants