WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Last modified 7 years ago

#34128 new defect (bug)

Tests_Feed_RSS2::test_channel fails when WP_TESTS_TITLE contains an apostrophe

Reported by: miqrogroove Owned by:
Priority: normal Milestone:
Component: Feeds Version:
Severity: normal Keywords: needs-patch needs-unit-tests
Cc: Focuses:

Description

If the site configuration contains an apostrophe in the site title, the test_channel unit fails.

1) Tests_Feed_RSS2::test_channel
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'Rob's Test Blog'
+'Rob's Test Blog'

The happens with either of the following configurations:

define( 'WP_TESTS_TITLE', 'Rob\'s Test Blog' );

and

define( 'WP_TESTS_TITLE', 'Rob's Test Blog' );

Change History (4)

#1 @netweb
11 years ago

  • Keywords needs-patch added
  • Milestone Awaiting Review → Future Release
  • Summary Tests_Feed_RSS2::test_channel Failure → Tests_Feed_RSS2::test_channel fails when WP_TESTS_TITLE contains an apostrophe

Sounds reasonable, kind of surprised more tests didn't fail

#3 @stevenkword
11 years ago

  • Keywords needs-unit-tests added

#4 @stevenkword
10 years ago

I did a little digging and believe the problem is in the xml_to_array() method of /tests/phpunit/includes/utils.php. This function is taking the encoded HTML entities and translating them back into unencoded characters.

Note: See TracTickets for help on using tickets.