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
Ticket/11250 Remake the unit tests for the BBCode parser #1142
Conversation
PHPBB3-11250
PHPBB3-11250
PHPBB3-11250
: . and [ are replaced by their html representation if they are text only PHPBB3-11250
…ones PHPBB3-11250
{ | ||
if ($incomplete) | ||
{ | ||
$this->markTestIncomplete('New bbcode parser has not been backported from feature/ascraeus-experiment yet.'); |
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.
You sure that this is text better than the personalized one? (like the one in line 240)
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.
Fixed
array( | ||
'PHPBB3-9364 - quoted: "test[/[/b]quote] test" / non-quoted: "[/quote] test" - also failed if layout distorted', | ||
'[quote]test[/[/b]quote] test [/quote][/quote] test', | ||
'[quote:]test[/[/b]quote] test [/quote:][/quote] test', |
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.
Is this correct?
Specially the part:
test [/quote:][
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.
Yes this is how it should be, see the mentioned ticket in the description http://tracker.phpbb.com/browse/PHPBB3-9364
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.
sorry, misread how the uid works.
* nickvergessen/ticket/11250: [ticket/11250] Use user defined message as incomplete message [ticket/11250] Add tests for simple nesting and code [ticket/11250] Run tests by default and add an option for incomplete ones [ticket/11250] Fix parsing result with special chars [ticket/11250] Add tests for size and attachment [ticket/11250] Fix method names [ticket/11250] Add some comments and fix a description [ticket/11250] Add some more tests for quotes [ticket/11250] Move quote special cases from class to unit tests [ticket/11250] Add parsing tests for our basic bbcodes in simple cases
Added some basic tests for 3.0 bbcodes and for the comments from message_parser.php
http://tracker.phpbb.com/browse/PHPBB3-11250