[theme-reviewers] Bugs in the Unit Test Data

Devin Price devin at wptheming.com
Tue Jul 2 01:20:38 UTC 2013


Hi.  Someone I follow on twitter (@GaryJ) noticed a few bugs in the 
Theme Unit Test Data (http://codex.wordpress.org/Theme_Unit_Test). I 
followed up with him and verified all the errors:

1) There's a post title called "Markup: Title with Markup", but the 
export seems to have stripped the markup, or the <b> and <i>, or 
<strong> and <em> were never actually included. Modifying the post so it 
becomes "Markup: Title <i>with</i> <b>Markup</b>" now enables potential 
passing of the test. See 
http://demo.gamajo.com/sample/2013/01/markup-title-with-markup/ (if you 
change the h1 font-weight to 100, you'll see that the final word is 
indeed bold).

2) This image 404s from the import data: 
http://wpthemetestdata.files.wordpress.com/2008/06/dsc20050813_115856_52.jpg.

3) There's a comment by John Saddington that uses a [youtube ...] 
shortcode. I'm not sure if this is supported in JetPack, but if so, it's 
not mentioned within the comment, and if not, then perhaps a [video...] 
shortcode or oEmbeded link might be better.

4) On the Comments post, there's a 2-3 posts by Michael Novotny that say 
it is an Author comment as well. Obviously trickier if the post is 
imported and assigned to a new or existing user, but the comments seem 
to be made by someone who isn't any of the default authors, so I don't 
think it would ever match up. There's also another comment which 
mentions "Author comment" made by Alex, yet his user ID is 0, so that 
would never the relevant post author class to appear either.

--

I also noticed the commenters on the "Template: Comments" changed to be 
mostly folks from the 8bit team.  They're great guys, but I think it 
might be more appropriate to have the unit test data be completely 
non-commercial (remove links to companies, etc).  (No mail list drama 
y'all.  Just +1 if you like.)

I believe a theme team admin will need to make these changes, so just 
wanted to pass it on.

-- 

Devin Price
WP Theming
@devinsays
<http://twitter.com/devinsays>



More information about the theme-reviewers mailing list