Skip to content

Conversation

@nastra
Copy link
Contributor

@nastra nastra commented Jun 17, 2021

.withFailMessage(..) was mistakenly used and was therefore overriding
the actual error reporting, making debugging difficult.

`.withFailMessage(..)` was mistakenly used and was therefore overriding
the actual error reporting, making debugging difficult.
@github-actions github-actions bot added the API label Jun 17, 2021
Copy link
Contributor

@jackye1995 jackye1995 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! this error formatting looks good to me:

Expecting throwable message:
  "Cannot add field unknown as an identifier field: not found in current schema or added columns"
to contain:
  "not found in current schemas or added columns"
but did not.

@nastra
Copy link
Contributor Author

nastra commented Jun 17, 2021

@rdblue could you review/commit this please?

@rdblue rdblue merged commit b2ebf22 into apache:master Jun 17, 2021
@nastra nastra deleted the assertj-assertion-description branch June 18, 2021 06:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants