Project

General

Profile

Actions

Patch #28207

closed

Test improvements for footnotes formatting syntax

Added by Mischa The Evil almost 7 years ago. Updated almost 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Text formatting
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

This originates from #28119#note-2.

  1. We might want to add a simple test (like ApplicationHelperTest#test_footnotes, but than in source:trunk/test/unit/lib/redmine/wiki_formatting/markdown_formatter_test.rb) for markdown footnotes as enabled per #16072.
  2. And while looking at the above, the question of 'how many other textile syntax tests are there remaining, lurking outside of source:/trunk/test/unit/lib/redmine/wiki_formatting/textile_formatter_test.rb and what do we want to do with them?' came up...
    • The answer to the first part would be 'several' (among them e.g. ApplicationHelperTest#test_footnotes). My answer to the second part of the question would be 'move them to the dedicated TextileFormatterTest class'.
I'll leave two patches against source:/trunk@17198:
  • 0001-Add-footnotes-test-for-markdown-formatter-16072.patch
    • This patch adds a footnotes test for the markdown formatter (MarkdownFormatterTest#test_footnotes).
  • 0002-Move-textile-footnotes-test-to-TextileFormatterTest.patch
    • This patch moves ApplicationHelperTest#test_footnotes to become TextileFormatterTest#test_footnotes.

In case there is consensus on my answer to the question raised under point 2, I'll open a dedicated container issue for it.

Note: I won't target this issue to any version, as I don't think this is of any use for the changelogs.


Files


Related issues

Related to Redmine - Feature #16072: Markdown footnote supportClosedJean-Philippe Lang

Actions
Actions #1

Updated by Mischa The Evil almost 7 years ago

Actions #2

Updated by Go MAEDA almost 7 years ago

  • Target version set to 4.0.0
Actions #3

Updated by Go MAEDA almost 7 years ago

  • Status changed from New to Closed
  • Assignee set to Go MAEDA

Committed. Thank you for improving the test suite.

Actions

Also available in: Atom PDF