Is there an automated testing framework for Joplin?
Backend test coverage is ok and thanks to Mic’s efforts we’re starting to have frontend coverage too. I’d also like to check Spectron at some point to see if we can use it for integration testing https://www.electronjs.org/spectron
On running the tests as @mic704b has pointed out, I get the following errors: Link to Pastebin
I have edited the file
StatusScreen.jsx to have a retry-all link, but I don’t think that should trigger these 8 errors. What is causing this?
@RedDocMD I am looking into the problems. In the mean time, please remove the test files starting
integration_XXX and try again. Please let me know if that resolves the problem.
Are you running on Windows?
These seem to be due to a similar problem. Work is in progress on a fix.
If you notice any other spurious test failures, please post them here too.
@laurent, in the test
MdToHtml, there is a failure on the file
sanitize_ 2.md. Is this an issue with the sanitization, for it seems that the close tag
</a> is inserted at the wrong point. Or maybe it is the problem with Markdown that we have discussed?
No, actually I think it is a problem with the Markdown rendering engine. Referring to the following screenshot:
The raw HTML itself (before sanitizing) seems to have the problem. The sanitizing does nothing to it.
@mic704b, what do you think about it?
Yes, the raw html looks wrong in your screenshot. Interesting, when I run this test case it passes.
That looks like an old version of Joplin.
Funny, I thought I was on the master branch. Anyway, I will rebase and try again
Yes, the markdown sanitize errors go away on merging the latest commits
[snappy comment on]
Another person who loves to take screenshots of text. Seriously, this is getting ridiculous.
[snappy comment off]
I get your point @tessus, but I wonder how I would have pasted that text here without creating a mess…
Besides, if you have noticed, my username on my laptop is madguy. :
I don’t understand. Discourse uses markdown. Markdown has code blocks. These code blocks have syntax highlighting.
@tessus, I didn’t know that you could have a multi-line code block. Thanks for pointing this out. I will use it in future comments.