Skip to content

Conversation

@afsafzal
Copy link
Member

Using Z3 to validate the specifications and a symbolic execution module + delta debugging.


@attr.s
class Specification(object):
name = attr.ib(type=str)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Use attrs (i.e., use the code that was deleted).

Copy link
Member Author

Choose a reason for hiding this comment

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

I don't exactly know how to specify that the argument should be str but I want the value to be set as Expression(precondition) in this format. If you know how, let me know so that I can change it.

self.__name = name
self.__precondition = Expression(precondition)
self.__postcondition = Expression(postcondition)
super().__init__()
Copy link
Collaborator

Choose a reason for hiding this comment

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

This does nothing.

return 1.0

def get_constraint(self, command: 'Command', state: State,
postfix: str = '')\
Copy link
Collaborator

Choose a reason for hiding this comment

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

No need for \. Also, one arg per line.

name: str,
precondition: str,
postcondition: str
) -> None:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Where's the timeout lambda?

@coveralls
Copy link

coveralls commented Sep 24, 2018

Coverage Status

Coverage increased (+2.6%) to 50.776% when pulling 8751a82 on symex into 8513d6c on master.

@ChrisTimperley
Copy link
Collaborator

Coverage Status

Coverage decreased (-37.2%) to 10.966% when pulling a49ff4d on symex into d6559a4 on master.

That's a surprisingly big drop, @afsafzal?

@afsafzal afsafzal merged commit 4113cae into master Sep 30, 2018
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