[theme-reviewers] Theme Unit Tests on GitHub?

[Gary Jones]

Thanks Lance.

Any opinions on a little Grunt file for the purpose of easily and
consistently removing the WP.com bits? (and whatever other tasks on the
file might need doing between source and distributable file).

Note that this isn't introducing any NodeJS dependency for *using* the XML
file, but it reduces the task of searching and replacing several regex
strings down to a single command line call.

I'd envisage a structure of something like:

* src/theme-unit-tests.xml
* dist/theme-unit-tests.xml
* dist/theme-unit-tests-wpcom.xml
* gruntfile.js
* package.json
* README.md

The dist files would be updated by Grunt from the src file, and adapted by
Grunt for WP.com and everyone else accordingly.
----
Gary Jones
Web Developer,  Gamajo Tech
On 16 Oct 2014 18:45, "Emil Uzelac" <emil at uzelac.me> wrote:

> Thanks kind lady :)
>
> On Thursday, October 16, 2014, Tammie Lister <karmatosed at gmail.com> wrote:
>
>>  Sure, I'll pop it over next week.
>>
>> Thanks
>> Tammie
>>
>> On 16/10/2014 18:42, Emil Uzelac wrote:
>>
>> Tammie would you like to start?
>>
>> On Thursday, October 16, 2014, Tammie Lister <karmatosed at gmail.com>
>> wrote:
>>
>>>  Hi
>>>
>>> I'm down with putting it on GH but we do need to make sure we have a
>>> process and particularly how it's prepared.
>>>
>>> Thanks
>>> Tammie
>>>
>>> On 16/10/2014 18:29, Edward Caissie wrote:
>>>
>>> Fine by me ...
>>>
>>>
>>> Edward Caissie
>>> aka Cais.
>>>
>>> On Thu, Oct 16, 2014 at 1:23 PM, Emil Uzelac <emil at uzelac.me> wrote:
>>>
>>>> Awesome and thanks. So guys should we push over to TRT GH or?
>>>>
>>>>
>>>> On Thursday, October 16, 2014, Lance Willett <nanobar at gmail.com> wrote:
>>>>
>>>>> Here's the checklist for preparing a new XML file, next time you need
>>>>> to do it.
>>>>>
>>>>> Often when I get the new XML file from
>>>>> http://wpthemetestdata.wordpress.com/ it needs a bit of massaging
>>>>> before commit to our pub repo.
>>>>>
>>>>> Here’s the process
>>>>>
>>>>> 1. WPTRT admins make changes to the test site on WP.com.
>>>>> 2. Go to http://wpthemetestdata.wordpress.com/ and export the full XML
>>>>> file, and open it locally in a text editor.
>>>>> 3. Clean it up.
>>>>> 4. Diff it with the previous version to be sure of no weird changes or
>>>>> additions, other than the expected ones.
>>>>> 5. Push to GitHub.
>>>>> 6. Go to http://codex.wordpress.org/Theme_Unit_Test, edit the page,
>>>>> change the UTC timestamp to “now” to show the most current update
>>>>> timestamp. **You can stop doing this probably since GitHub will have a
>>>>> history going forward.**
>>>>>
>>>>> Notes for the XML clean up, step 3.
>>>>>
>>>>> Remove the following metadata elements that are specific to WP.com.
>>>>>
>>>>> superawesome
>>>>> _edit_last
>>>>> email_notification
>>>>> _searchme
>>>>> jabber_published
>>>>> geo_public
>>>>> _elasticsearch_indexed_on
>>>>>
>>>>> I look for these patterns using a text editor like TextMate or Sublime
>>>>> Text, with regex search turned on:
>>>>>
>>>>> 1. https://gist.github.com/lancewillett/a7c849026e2e403399a0
>>>>> 2. https://gist.github.com/lancewillett/127085c444ec6ff398d1
>>>>>
>>>>> Then search/replace with an empty string to remove from the XML file.
>>>>>
>>>>> It's not end of the world if they make it into the Unit Test XML file,
>>>>> but it's just clutter.
>>>>>
>>>>> – Lance
>>>>>
>>>>> On Thu, Oct 16, 2014 at 11:15 AM, Lance Willett <nanobar at gmail.com>
>>>>> wrote:
>>>>> > The history of the file changes is public, it's part of our WP.com
>>>>> > public themes SVN repo.
>>>>> >
>>>>> > You can see the history by doing something like:
>>>>> > svn log
>>>>> https://wpcom-themes.svn.automattic.com/demo/theme-unit-test-data.xml
>>>>> >
>>>>> > I'm totally happy for the file to move to GitHub. If WPTRT has an
>>>>> > account and you'd like to maintain it going forward—please just grab
>>>>> > the latest copy and start from there; I don't need to do anything
>>>>> > further.
>>>>> >
>>>>> > Then edit the Codex page at
>>>>> http://codex.wordpress.org/Theme_Unit_Test
>>>>> > to point to the GitHub repo instead of the SVN repo.
>>>>> >
>>>>> > Done. :)
>>>>> >
>>>>> > – Lance
>>>>> _______________________________________________
>>>>> theme-reviewers mailing list
>>>>> theme-reviewers at lists.wordpress.org
>>>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>>>
>>>>
>>>> _______________________________________________
>>>> theme-reviewers mailing list
>>>> theme-reviewers at lists.wordpress.org
>>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>>
>>>>
>>>
>>>
>>> _______________________________________________
>>> theme-reviewers mailing listtheme-reviewers at lists.wordpress.orghttp://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>
>>>
>>> --
>>> Thanks
>>> Tammie Lister
>>>
>>>
>>
>> _______________________________________________
>> theme-reviewers mailing listtheme-reviewers at lists.wordpress.orghttp://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>
>>
>> --
>> Thanks
>> Tammie Lister
>>
>>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20141016/00f49be3/attachment.html>