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

Use AssertionError(String, Throwable) instead of supplying the cause later. #1297

Merged
1 commit merged into from
May 29, 2024

Conversation

copybara-service[bot]
Copy link
Contributor

Use AssertionError(String, Throwable) instead of supplying the cause later.

I could have done this as part of cl/609415936 and cl/604677493, but I'd missed that AssertionError(String, Throwable) was added in API Level 19.

(The code in Platform in cl/609415936 still needs to use initCause because it uses ComparisonFailure, which does not expose a Throwable-accepting constructor.)

RELNOTES=n/a

…e later.

I could have done this as part of cl/609415936 and cl/604677493, but I'd missed that `AssertionError(String, Throwable)` was [added in API Level 19](https://developer.android.com/reference/java/lang/AssertionError#AssertionError(java.lang.String,%20java.lang.Throwable)).

(The code in `Platform` in cl/609415936 still needs to use `initCause` because it uses `ComparisonFailure`, which does not expose a `Throwable`-accepting constructor.)

RELNOTES=n/a
PiperOrigin-RevId: 638335715
@copybara-service copybara-service bot closed this pull request by merging all changes into master in 7a1c72f May 29, 2024
@copybara-service copybara-service bot deleted the test_638262992 branch May 29, 2024 17:38
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.

1 participant