Add Test Coverage for Atom & Reorganize Feeds Unit Tests

[stevenkword]

In Make WordPress Core, we refer to the syndication components as the "Feeds Component". However, in our unit tests, we have the group labelled in its singular form, "Feed". I proposing that we rename this group to its plural form and relocate these test into the tests/phpunit/tests/feeds/ directory.

IMPORTANT -- I don't believe svn diff has the ability to delete directories, so if this proposal is accepted, we need to make sure to manually rename/delete the tests/phpunit/tests/feed directory before commit. Apologies in advance!

In addition to renaming the group, this ticket aims to separate generic syndication tests from the RSS-specific test and additionally provides test coverage for Atom feeds, which currently do not exists.

I feel it is also important to note that this will mandate a refresh of any feeds-related unit tests that are currently sitting in trac. I am willing to commit to refreshing those personally, but would prefer to hold on pending approval from the Committers.

Lastly, this patch proposes a new file, tests/phpunit/tests/feeds/common.php, which contains 3 brand new tests that will currently fail against trunk. I wanted to leave these in here to demonstrate what I was thinking with the common tests. These relate to ticket #30210. However, if I take them back out, the purpose of the common.php file becomes much less apparant. Let's discuss!

[stevenkword]

@jorbin -- You might be interested in this one.

[stevenkword]

Raising priority since this is blocking any ticket where Atom testing is required. e.g. #33591

[chriscct7]

Bug Scrub 1/8/2016 notes:

• The renaming of the group was favorable, with the note that when it is done, the associated PHP files also need to have the file docbloc item updated as well.
• Patch should be split as follows:

Phase 1: Add atom tests.
Phase 2: Rename the group.
Phase 3: Add the common.php tests AFTER the group is renamed, which currently fail and need to be fixed.

• Some of the patch functions have incomplete and/or missing docblocs.
• Some parts of the patch have formatting issues that need to be addressed.

[johnbillion]

A few notes:

• The post_count and excerpt_only properties aren't set in the Tests_Feeds_Atom class.

In both Tests_Feeds_Atom and Tests_Feeds_RSS2:

• The author assertion needs to test against the user's display name, not their user_login. Need to explicitly set a display name when creating the test user.
• The test posts should have some categories and tags assigned so the category and tag assertions operate on more than just the default category and an empty tag list.
• Creating more than 5 test posts might be a good idea, then an assertion can be added to ensure the feeds contain the correct number of posts.
• Unless the test posts have explicitly differentiating properties (which they don't), there's no need to loop over every post and run the assertions. Just running the assertions on the first post is enough.
• assertCount( 1, $x ) can be used instead of assertEquals( 1, count( $x ) ).

[stevenkword]

35160.atom.patch​ takes the suggestions made by @johnbillion into consideration. I'd rather just get the Atom tests in for now since they are blocking other tickets, and we can create new tickets down the road for dealing with common / RSS.

[stevenkword]

I've also added 35160.rss2.patch​ which improves usage of coding standards and doc, but also implements the recommended changes found in 35160.atom.patch​ . They are both in the same "style" at this time.

[stevenkword]

Fixes formatting per PHPCS

[stevenkword]

Clean-up for the RSS2 tests

[jorbin]

In 36519:

Improve Automated Feed Tests

Multiple improvements to the RSS2 automated tests along with the addition of Atom tests.

1. General whitespace cleanup (since the rss2 file serves as the base of the atom file).
2. Adds an author and category to the tests.
3. Since the content of the posts is the same, we don't need to test all of the post content.
4. Adds many posts so that the post count can be checked

Props stevenkword
Fixes #35160.

[johnbillion]

In 38924:

Feeds: Greatly reduce the number of dummy posts that are generated for the RSS2 tests. This speeds the tests up.

See #35160, #30210

[johnbillion]

In 38927:

Feeds: Greatly reduce the number of dummy posts that are generated for the Atom tests, following on from [38924].

See #35160