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

Adding a test to test overall persistence of userId and anonymousId. Change setCachedSettings: to not statically dispatch_once #623

Merged
merged 1 commit into from
Oct 22, 2016

Conversation

tonyxiao
Copy link
Contributor

@tonyxiao tonyxiao commented Oct 22, 2016

This helps ensure bug like #615 not happen again

cc @sperand-io @f2prateek

…Change setCachedSettings: to not statically dispatch_once
@codecov-io
Copy link

codecov-io commented Oct 22, 2016

Current coverage is 46.61% (diff: 80.00%)

Merging #623 into dev will increase coverage by 8.53%

@@                dev       #623   diff @@
==========================================
  Files            79         79          
  Lines          2479       2484     +5   
  Methods         619        619          
  Messages          0          0          
  Branches        264        266     +2   
==========================================
+ Hits            944       1158   +214   
+ Misses         1419       1182   -237   
- Partials        116        144    +28   

Powered by Codecov. Last update f26b145...6036abf

@f2prateek
Copy link
Contributor

LGTM

@f2prateek f2prateek merged commit b1d957e into dev Oct 22, 2016
@f2prateek f2prateek deleted the test-persistence branch October 22, 2016 04:58
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