[theme-reviewers] Bugs in the Unit Test Data

[Emil Uzelac]

All commercial, personal etc links are now http://example.org/ and let's
keep
it that way. Going over the rest now and once we're done, I'll ping Lance to
refresh the data for us.


On Mon, Jul 1, 2013 at 8:35 PM, Emil Uzelac <emil at uzelac.me> wrote:

> Hey Devin,
>
> Agreed on the non-commercial links, let's go with wordpress.org,
> automattic.com and example.com instead.
> No need to go above that. Shortcode test would not work and no need to
> test plugins with the Themes.
>
> BTW the Genesis titles from the demo need: word-break: break-word; ;)
>
> I will go over the data now.
>
> Emil
>
>
> On Mon, Jul 1, 2013 at 8:20 PM, Devin Price <devin at wptheming.com> wrote:
>
>> Hi.  Someone I follow on twitter (@GaryJ) noticed a few bugs in the Theme
>> Unit Test Data (http://codex.wordpress.org/**Theme_Unit_Test<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/<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<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>
>>
>> ______________________________**_________________
>> theme-reviewers mailing list
>> theme-reviewers at lists.**wordpress.org<theme-reviewers at lists.wordpress.org>
>> http://lists.wordpress.org/**mailman/listinfo/theme-**reviewers<http://lists.wordpress.org/mailman/listinfo/theme-reviewers>
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20130701/823a7dbc/attachment.html>