Skip to content

assertj refasters could fix up allMatch #36

@carterkozak

Description

@carterkozak

I think using .isTrue() on any kind of complex assertion is probably a codesmell, so might be worth replacing stuff like this:

-assertThat(result.getTestResults().stream().allMatch(
-        testResult -> testResult.getType().equals(RegressionTestResultType.NO_REGRESSION))).isTrue();
+assertThat(result.getTestResults()).allMatch(
+        testResult -> testResult.getType().equals(RegressionTestResultType.NO_REGRESSION));

Migrated from palantir/gradle-baseline#953

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions