-
-
Notifications
You must be signed in to change notification settings - Fork 2
Conversation
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, though I did not run the tests locally. But from static code analysis only the describe.skip
instead of xdescribe
caught my attention. Good stuff.
}) | ||
// Pending since index pages have console error | ||
// see https://github.com/eXist-db/monex/issues/60 | ||
xdescribe('profiling', function () { |
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.
Have you tried describe.skip(...)
here?
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.
both work the same, for consistency i stuck with x
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.
Oh really, interesting!
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.
I still prefer .skip being more readable and more obvious. Also, it is not mentioned in the mocha docs front page and could therefore be considered discouraged style. Moreover, is there a alternative representation for .only
?
So, in the long run I would prefer to switch to .skip globally.
cy.contains('/db/apps/monex/controller.xql') | ||
}) | ||
}) | ||
xdescribe('indexes', function () { |
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.
see above
functional tests of the new and old dashboards.
old package-manager has a skipped test, fixing it seems not worth the effort. If anybody feels like having a go at it be my guest.
The 2 skipped monex test still point to actual errors
close #8 #1