search

square / assertj-android

A set of AssertJ helpers geared toward testing Android.
https://square.github.io/assertj-android/
Apache License 2.0
1.58k stars 156 forks source link

Adding not null Assertion for IntentAssert #87

Closed donnfelker closed 10 years ago

donnfelker commented 10 years ago

During development of a deep linking feature for a client I ran into an issue where android-fest assertion was throwing a NullPointerException. This happens when you check for the component name in an IntentAssert but for some reason the ComponentName was not set (for whatever reason). This change adds a null check for component assertion as sometimes component can be null.

Just an FYI - Since this repo is now gradle based, the Contributing markdown file seems to be out of date as it specifies mvn clean verify prior to pull request submission. :smile:

JakeWharton commented 10 years ago

Thanks, Donn. I'll get the docs updated too.

donnfelker commented 10 years ago

:thumbsup: