-
Notifications
You must be signed in to change notification settings - Fork 49
Improve test navigation #114
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
Improve test navigation #114
Conversation
bd65789
to
a3473ba
Compare
This change adds support for: * nesting tests (deep hierarchy) * direct navigation to failed block (cypher or cypher params) * let augmentation-tests provides a diff on failure that can be viewed in IntelliJ
a3473ba
to
48f0169
Compare
testSet.line = lineNr + 1 | ||
currentDepth = depth | ||
} | ||
line.startsWith("CAUTION:") -> ignore = true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should probably also ignore some others like NOTE:
etc.
var title: String? = null | ||
var current: StringBuilder? = null | ||
|
||
var codeBlocks = mutableMapOf<String, ParsedBlock>() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it would still be nice to have an object for these.
} | ||
} | ||
|
||
companion object { | ||
val DEBUG = false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should come from an external config or system property so it can be changed externally for test runs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks a lot !
Please add the configurability of DEBUG as a separate PR. |
This change adds support for: