Skip to content

Conversation

@ana-pantilie
Copy link
Contributor

Fixes #1767


Reviewer checklist
  • Test coverage: stack test --coverage
  • Public API documentation: stack haddock

(mkElemVar Mock.x)
initial = pure $
Mock.sigma (mkElemVar Mock.x) (Mock.plain10 (mkElemVar Mock.y))
actual <- applyRewriteRuleParallel_ initial axiomSigmaId
Copy link
Contributor Author

@ana-pantilie ana-pantilie Jun 16, 2020

Choose a reason for hiding this comment

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

For this test, applyRewriteRuleParallel_ will now throw the following error:

Expected function-like term, but found:
    /* Created: Test.Kore.Step.RewriteStep.sigma */
    /* Sfa */
    sigma{}(
        /* Fl Fn D Sfa */ Configx:testSort{},
        /* Created: Test.Kore.Step.RewriteStep.plain10 */
        /* Sfa */
        plain10{}(/* Fl Fn D Sfa */ Configy:testSort{})
    )
CallStack (from HasCallStack):
  error, called at src/Kore/Step/Step.hs:195:21 in kore-0.23.0.0-Kq7aVMskyU0HcW2wehu1H0:Kore.Step.Step

@ana-pantilie ana-pantilie requested a review from ttuegel June 16, 2020 17:49
@ana-pantilie ana-pantilie marked this pull request as ready for review June 17, 2020 07:03
test_unsupportedConstructs :: TestTree
test_unsupportedConstructs =
testCase "Unsupported constructs" $
andSimplifyFailure
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a difference now between andSimplifySuccess and andSimplifyFailure?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You're right, there isn't any. I've made the necessary changes. Thanks!

@ana-pantilie ana-pantilie merged commit e734b1c into runtimeverification:master Jun 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove UnificationError

2 participants