Please wrap all your lines at 80 characters, otherwise it's very difficult to inspect diffs on github.

Also note that when this assertion fails, you will get rather unhelpful error message - you won't know what value was returned.

Much better way of writing test assertions is to use `assertThat` and hamcrest matchers:

```java
assertThat("Incorrect value returned.", response.getString("data"), equalTo("shhh!"));
```

See e.g. this [SO answer](http://stackoverflow.com/a/1464411/69868) and this [wiki page](https://github.com/junit-team/junit/wiki/Matchers-and-assertthat#assertthat) for more details.

Even better, you can write your [custom matcher](https://code.google.com/p/hamcrest/wiki/Tutorial#Writing_custom_matchers) so that you don't have to repeat yourself so much. The assertion can be a nice short on-liner then:

```java
@Override
public void onSuccess(JSONObject response) throws JSONException {
    assertThat(response, allOf(notNullValue(), hasData("shhh!"));
    // or even
    assertThat(response, hasData("shhh!");
}
```