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 for DDLogmessage #498

Merged
merged 5 commits into from
Jun 24, 2015
Merged

Tests for DDLogmessage #498

merged 5 commits into from
Jun 24, 2015

Conversation

pk
Copy link
Contributor

@pk pk commented Apr 18, 2015

Hi from Cocoapods TestJam,

I'm trying to add some test coverage for the project. Started with the DDLogMessage as it was simple enough.

There are 2 failing tests because I've wrote tests according to the documentation which mentions using DDLogMessageOptions to force copy the file & function strings but the code is actually missing in the implementation.

@bpoplauschi bpoplauschi added this to the 2.0.1 milestone Apr 20, 2015
@rivera-ernesto
Copy link
Member

This one has some more Travis errors than #499 and #500.

@pk
Copy link
Contributor Author

pk commented Apr 20, 2015

Hi Ernesto,

this one won't actually build in Travis until the #499 is merged in. I wrote the tests to comply with the documentation in the DDLogMessage so it must fail. Two errors actually where I check file & function attributes copy behaviour.

I think I need to figure out why the #499 is failing first, fix that and then merge this one in.

@bpoplauschi bpoplauschi merged commit 28ba285 into CocoaLumberjack:master Jun 24, 2015
bpoplauschi added a commit that referenced this pull request Jun 24, 2015
@bpoplauschi
Copy link
Member

Thanks @pk for all the unit tests you created and sorry for the delay of adding them into the project.

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.

None yet

3 participants