Skip to content

MigrateHamcrestToAssertJ mistake #861

@ChenyuWang98

Description

@ChenyuWang98

Why make this replacement?
assertThat(a,is(b)) =》assertThat(a).containsExactly(b)
closeTo(123.123,1) => isCloseTo(123.123,within(1))

I think it should
assertThat(a,is(b)) =》 assertThat(a).isEqualTo(b)
closeTo(123.123,1) => isCloseTo(123.123,within(1.0))

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    Status

    Done

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions