-
Notifications
You must be signed in to change notification settings - Fork 31
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
adapt to latest ex-aws #25
Conversation
Thanks for the PR. Looks good, mind adding a quick test? |
@@ -10,7 +10,7 @@ defmodule CloudWatchTest do | |||
use ExUnit.Case, async: false | |||
|
|||
setup_all do | |||
{:ok, _} = Cycler.start_link() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I ran into a race condition when running tests locally where this match was failing here or in log_stream_name_test depending on which ran first since the tests don't wait for the cycler process to exit before running the next one.
@@ -10,7 +10,7 @@ defmodule CloudWatchTest do | |||
use ExUnit.Case, async: false | |||
|
|||
setup_all do | |||
{:ok, _} = Cycler.start_link() | |||
start_supervised(Cycler) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the fix for the above. It makes the test wait for the cycler to exit before running the next test.
@kuiro5 I've added a test. |
@kuiro5 hi, sorry to bother again. I wonder if it would be possible to cut a release for current master? |
@ruslandoga @msmithstubbs v0.4.1 released, apologies on the delay. |
closes #24