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

PathAssert constructor uses PaintAssert? #191

Closed jcogilvie closed 8 years ago

jcogilvie commented 8 years ago 
public class PathAssert extends AbstractAssert<PathAssert, Path> {
  public PathAssert(Path actual) {
    super(actual, PaintAssert.class); // looks wrong?
  }
...
}

Shouldn't that be PathAssert?

JakeWharton commented 8 years ago

Yep!

On Tue, Feb 16, 2016 at 1:55 PM Jonathan Ogilvie notifications@github.com wrote:

public class PathAssert extends AbstractAssert<PathAssert, Path> { public PathAssert(Path actual) { super(actual, PaintAssert.class); // looks wrong? } ... }

Shouldn't that be PathAssert?

— Reply to this email directly or view it on GitHub https://github.com/square/assertj-android/issues/191.

JakeWharton commented 8 years ago

Want to send a pull request for it?

On Tue, Feb 16, 2016 at 2:06 PM Jake Wharton jakewharton@gmail.com wrote:

Yep!

On Tue, Feb 16, 2016 at 1:55 PM Jonathan Ogilvie notifications@github.com wrote:

public class PathAssert extends AbstractAssert<PathAssert, Path> { public PathAssert(Path actual) { super(actual, PaintAssert.class); // looks wrong? } ... }

Shouldn't that be PathAssert?

— Reply to this email directly or view it on GitHub https://github.com/square/assertj-android/issues/191.