[theme-reviewers] Theme Unit Tests on GitHub?

[Emil Uzelac]

Not really sure what would be the point of that to be honest. Anyone can
freely obtain an official copy and ping us here with the changes.

Creating a ticket is also not needed. We can handle all at one place, which
is right here.

And again this is @Lance's call :)



On Friday, October 10, 2014, Tammie Lister <karmatosed at gmail.com> wrote:

>  I think having it (not forked) but accessible on GH could be good. We
> don't have to fork just to put on GH and for the record, I never suggested
> it was forked :)
>
> Regards
> Tammie Lister
>
> On 10/10/2014 15:56, Emil Uzelac wrote:
>
> I don't think we need to fork the unit. WPTest is not official copy or the
> same as ours. If you wish to improve, please by all means. Just ping me and
> I will go over the changes and make the update.
>
>  Unless @Lance thinks that this is ok to do, but I doubt that because
> someone tried few times already.
>
> On Friday, October 10, 2014, Gary Jones <gary at gamajo.com
> <javascript:_e(%7B%7D,'cvml','gary at gamajo.com');>> wrote:
>
>> Hey,
>>
>>  tl;dr:
>> 1. Could the theme unit tests file be put on public repo on GitHub?
>> 2. Anyone want to help tackle
>> https://core.trac.wordpress.org/ticket/22279 ?
>>
>>  Posting after Tammie suggested I do.
>>
>>  I'd like to see The Theme Unit Tests file on GitHub for the following
>> reasons:
>>
>>  * Access to commit history to see what exactly has changed version to
>> version. Currently, us mere mortals can only see when it was updated - the
>> information about what was not updated is not available (or not easy to
>> find) as it's part of Automattic's SVN repo.
>>  * Easier to see already-reported bugs and enhancements, rather than
>> wading through the mailing list archives.
>>
>>  There's a fork already on GH (https://github.com/manovotny/wptest)
>> which claims to have added extra edge cases. But I've currently got no idea
>> which cases these are, and why they weren't added to the original Tests
>> file. Michael also believes that further cases and fixes may have been
>> added to the official Tests since the fork that aren't in his, so now we
>> may have two sets of tests, mostly but not completely duplicated.
>>
>> If the official Tests were on GH, and open to pull requests on those edge
>> cases and other issues, then I'd hope that wptest could be discontinued to
>> just leave a single more-complete test suite that everyone can benefit from.
>>
>> The wptest file has fixed things like
>> https://github.com/manovotny/wptest/issues/27 (s/Isaac Newton/Albert
>> Einstein) and https://github.com/manovotny/wptest/issues/24
>> (s/Horizontal/Vertical on vertical featured image alt text) which the
>> official Tests are still bugged with. Both tests are bugged with missing
>> markup formatting tests for some elements,
>> https://github.com/manovotny/wptest/issues/37 (s/Headers/Headings) and a
>> side-effect of https://core.trac.wordpress.org/ticket/29621 (post titles
>> stripped of markup on export).
>>
>> One of the things that would make contributions *considerably* easier
>> would be to get https://core.trac.wordpress.org/ticket/22279 (mixed line
>> endings for XML vs cdata in export) fixed. That would enable direct editing
>> of the file without breaking line-endings, without having to set up a clean
>> WP install, import, amend and export. If anyone has experience of the
>> Exporter and could help on this, then I would personally be grateful, as
>> would potential GitHub contributors :-)
>>
>>  Gary
>>
>>   --
>> Gary Jones
>> Web Developer, Gamajo Tech
>>
>
>
> _______________________________________________
> theme-reviewers mailing listtheme-reviewers at lists.wordpress.org <javascript:_e(%7B%7D,'cvml','theme-reviewers at lists.wordpress.org');>http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
> --
> Thanks
> Tammie Lister
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20141010/9b67c927/attachment.html>