Skip to content
This repository has been archived by the owner on Jul 30, 2018. It is now read-only.

Improve error information on a test operation #151

Open
kitsonk opened this issue Dec 29, 2017 · 0 comments
Open

Improve error information on a test operation #151

kitsonk opened this issue Dec 29, 2017 · 0 comments

Comments

@kitsonk
Copy link
Member

kitsonk commented Dec 29, 2017

Enhancement

When attempting to apply an undo operation that is failing for some reason because the test operation does not pass, currently all you get is:

Error: Test operation failure. Unable to apply any operations.

An error message the provides more context would be more useful, like:

Error: Test operation failed.
    Expected "somevalue" for at path "/foo/bar/baz" but got "othervalue"

Potentially looking AssertionError and potentially moving that to core. The other advantage is it will utilise V8's diffing functionality when thrown, improving the information available in Chrome and Node.

@kitsonk kitsonk added this to the rc.1 milestone Dec 29, 2017
@agubler agubler removed the rc label Jan 4, 2018
@agubler agubler removed this from the rc.1 milestone Jan 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants