supermarin / ObjectiveSugar

ObjectiveC additions for humans. Ruby style.
MIT License
2.17k stars 190 forks source link

Add slather for test coverage #87

Closed marklarr closed 9 years ago

marklarr commented 10 years ago

Computes test coverage using slather. Allows you to see what code is tested, and also gives you a cute little badge. Just need to enable the repo at https://coveralls.io

Coverage Status

supermarin commented 10 years ago

@marklarr thanks for the PR!

Left you some line notes before merging

marklarr commented 9 years ago

https://github.com/supermarin/ObjectiveSugar/pull/91

supermarin commented 9 years ago

Yo @marklarr what happened to this and #91?

would be great to just have a reminder why we killed it :)

marklarr commented 9 years ago

Hey @supermarin, I shut this one down because it used the old Pods integration,

# Podfile

begin
  require 'slather'
  Slather.prepare_pods(self)
rescue LoadError
  puts 'Slather has been disabled (not installed).'
end

which was sort of a drag. #91 takes advantages of @kylef's cocoapods integration, making the above not necessary anymore. Unfortunately, though, although #91 appears to be working fine locally for me, when it runs on travis it reports 0% test coverage, and I haven't quite been able to figure it out yet.

supermarin commented 9 years ago

:+1: thanks!

Feel free to contribute back when you figure it out