Skip to content

Test for member type consistency [depends-on: #4021] #4023

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 1, 2019

Requires #4021 and #4020 -- please only review the last commit. This adds a test for ec41608 which went in with @thk123's type consistency improvements -- revert that commit (requires reverting a few subsequent refactors) to see it fail.

@tautschnig tautschnig changed the title Test for member type consistency Test for member type consistency [depends-on: #4020, #4021] Feb 1, 2019
@tautschnig tautschnig changed the title Test for member type consistency [depends-on: #4020, #4021] Test for member type consistency [depends-on: #4021] Feb 12, 2019
tautschnig added a commit that referenced this pull request Mar 7, 2019
…xpressions

Verify member expressions [blocks: #4023]
@smowton smowton force-pushed the smowton/admin/test-for-member-type-consistency branch from a284810 to b1efed4 Compare March 7, 2019 13:43
@smowton
Copy link
Contributor Author

smowton commented Mar 7, 2019

Rebased

@tautschnig tautschnig merged commit 9a8dab9 into diffblue:develop Mar 7, 2019
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: b1efed4).
Build URL: https://travis-ci.com/diffblue/test-gen/builds/103549141
Status will be re-evaluated on next push.
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 compatibility was already broken by an earlier merge.

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.

4 participants