Skip to content

Add test checking generic function parameter casts appear when expected #4200

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

Conversation

smowton
Copy link
Contributor

@smowton smowton commented Feb 15, 2019

This is a test for 3897b39, one of the type consistency fixes committed with #3822

These will get easier to express as and when expr2java prints generic type parameters / qualifiers, but I won't take that task on here, particularly because that means checking all consumers of pretty-printed output.

@@ -0,0 +1,28 @@

public class test<T> {
Copy link
Member

Choose a reason for hiding this comment

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

Please capitalize class identifiers

@smowton smowton force-pushed the smowton/cleanup/call-function-with-generic-parameter-test branch from 73e4933 to 3ffc34c Compare February 15, 2019 17:44
@smowton smowton force-pushed the smowton/cleanup/call-function-with-generic-parameter-test branch from 3ffc34c to 7ec1023 Compare February 15, 2019 17:51
Copy link
Contributor

@allredj allredj left a comment

Choose a reason for hiding this comment

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

🚫
This PR failed Diffblue compatibility checks (cbmc commit: 73e4933).
Build URL: https://travis-ci.com/diffblue/test-gen/builds/101128366
Status will be re-evaluated on next push.
Please contact @peterschrammel, @thk123, or @allredj for support.

Common spurious failures:

  • the cbmc commit has disappeared in the mean time (e.g. in a force-push)
  • the author is not in the list of contributors (e.g. first-time contributors).

The incompatibility may have been introduced by an earlier PR. In that case merging this
PR should be avoided unless it fixes the current incompatibility.

Copy link
Contributor

@allredj allredj left a comment

Choose a reason for hiding this comment

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

✔️
Passed Diffblue compatibility checks (cbmc commit: 7ec1023).
Build URL: https://travis-ci.com/diffblue/test-gen/builds/101132412

@smowton smowton merged commit a3d9223 into diffblue:develop Feb 15, 2019
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.

3 participants