Skip to content
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

Use for the tests wherever possible. #69



None yet
2 participants
Copy link

commented Mar 24, 2014

The Markdown Test Suite test suite is already huge for the Original markdown spec: note how it allowed this pull request to reduce the existing test file to almost nothing.

Support for extensions has been added recently by yours truly, so there aren't many implemented yet, but additions are now welcome.

@karlcow was really nice and accepted pull requests quickly, so I think it is a good idea to collaborate with him.

If we ever feel that the projects are starting to diverge in their goals we can just fork off, but for now contributing seems like the best option to me.

The only downside of this is that it requires test creators to also code the expected output for the test which is not required for tests here. We could however just accept tests directly here if contributors don't have the time to pull request there separately, and if ever they are implemented there we can remove them from here.


This comment has been minimized.

Copy link

commented Mar 24, 2014

There was an interesting idea for testing which was given to me in the past but I have never come to try it.

Instead of testing raw markup output which might have zillions of syntactic rules for spaces, etc. It might be better to test the rendered DOM.

Once a test markdown file is being converted to HTML, it is possible to create how it is represented in the DOM. This DOM can be more easily compared in between different conversion tools, because HTML5 specification defines now exactly how the serialization of HTML as a DOM should be done.

@cirosantilli cirosantilli changed the title Use for the tests. Use for the tests wherever possible. Mar 29, 2014


This comment has been minimized.

Copy link
Collaborator Author

commented Apr 7, 2014

2 weeks have passed, so I'm merging this. Will undo at any time if anyone sees a problem.

cirosantilli added a commit that referenced this pull request Apr 7, 2014

@cirosantilli cirosantilli merged commit 290d61b into plasticboy:master Apr 7, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.