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

tests(settings): adding tests for FindHome() and SetHome() #245

Merged
merged 3 commits into from
Sep 15, 2016

Conversation

ultimateboy
Copy link
Member

No description provided.

@deis-bot
Copy link

@Joshua-Anderson and @aboyett are potential reviewers of this pull request based on my analysis of git blame information. Thanks @ultimateboy!

@codecov-io
Copy link

codecov-io commented Sep 15, 2016

Current coverage is 28.09% (diff: 100%)

Merging #245 into master will increase coverage by 1.28%

@@             master       #245   diff @@
==========================================
  Files            57         57          
  Lines          3905       3905          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           1047       1097    +50   
+ Misses         2716       2651    -65   
- Partials        142        157    +15   

Powered by Codecov. Last update 2e26124...1fee0ac

@mboersma mboersma added the LGTM1 label Sep 15, 2016
@mboersma mboersma added this to the v2.6 milestone Sep 15, 2016
@ultimateboy ultimateboy merged commit d5c5e7b into deis:master Sep 15, 2016
@ultimateboy ultimateboy deleted the home-tests branch September 15, 2016 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants