Skip to content
This repository has been archived by the owner on May 8, 2019. It is now read-only.

Test profile jobs #1162

Merged
merged 10 commits into from
Dec 2, 2016
Merged

Test profile jobs #1162

merged 10 commits into from
Dec 2, 2016

Conversation

anamariamv
Copy link
Contributor

This PR solves #1138 , added tests for ProfileJobs.
@javipacheco could you review, please?
Thanks!!

@47degdev
Copy link

47degdev commented Dec 1, 2016

Current coverage is 32.06% (diff: 42.85%)

Merging #1162 into master will increase coverage by 0.95%

@@             master      #1162   diff @@
==========================================
  Files           422        422          
  Lines         15200      15202     +2   
  Methods       14825      14831     +6   
  Messages          0          0          
  Branches        359        355     -4   
==========================================
+ Hits           4729       4875   +146   
+ Misses        10471      10327   -144   
  Partials          0          0          

Powered by Codecov. Last update 713c181...dfc729a

@javipacheco
Copy link
Collaborator

LGTM!

@javipacheco javipacheco merged commit 10c808e into master Dec 2, 2016
@javipacheco javipacheco deleted the am-test-profileJobs branch December 2, 2016 08:14
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.

3 participants