+global\Tests_Feeds_Atom::test_entry_elements()

Complexity

Conditions	10
Paths	73

Size

Total Lines	77
Code Lines	39

Duplication

Lines	4
Ratio	5.19 %

Importance

Changes

0

<?php
    function test_entry_elements() 
    {
        $this->go_to('/?feed=atom');
        $feed = $this->do_atom();
        $xml = xml_to_array($feed);

        // Get all the <entry> child elements of the <feed> element.
        $entries = xml_find($xml, 'feed', 'entry');

        // Verify we are displaying the correct number of posts.
        $this->assertCount($this->post_count, $entries);

        // We Really only need to test X number of entries unless the content is different
        $entries = array_slice($entries, 1);

        // Check each of the desired entries against the known post data.
        foreach ( $entries as $key => $entry ) {

            // Get post for comparison
            $id = xml_find($entries[$key]['child'], 'id');

Array keys should be surrounded by spaces unless they contain a string or an integer.

            preg_match('/\?p=(\d+)/', $id[0]['content'], $matches);
            $post = get_post($matches[1]);

Overridding WordPress globals is prohibited

            // Author
            $author = xml_find($entries[$key]['child'], 'author', 'name');

Array keys should be surrounded by spaces unless they contain a string or an integer.

            $user = new WP_User($post->post_author);
            $this->assertEquals($user->display_name, $author[0]['content']);

            // Title
            $title = xml_find($entries[$key]['child'], 'title');

Array keys should be surrounded by spaces unless they contain a string or an integer.

            $this->assertEquals($post->post_title, $title[0]['content']);

            // Link rel="alternate"
            $link_alts = xml_find($entries[$key]['child'], 'link');

Array keys should be surrounded by spaces unless they contain a string or an integer.

            foreach ( $link_alts as $link_alt ) {
                if ('alternate' == $link_alt['attributes']['rel'] ) {
                    $this->assertEquals(get_permalink($post), $link_alt['attributes']['href']);
                }
            }

            // Id
            $guid = xml_find($entries[$key]['child'], 'id');

Array keys should be surrounded by spaces unless they contain a string or an integer.

            $this->assertEquals($post->guid, $id[0]['content']);

            // Updated
            $updated = xml_find($entries[$key]['child'], 'updated');

Array keys should be surrounded by spaces unless they contain a string or an integer.

            $this->assertEquals(strtotime($post->post_modified_gmt), strtotime($updated[0]['content']));

            // Published
            $published = xml_find($entries[$key]['child'], 'published');

Array keys should be surrounded by spaces unless they contain a string or an integer.

            $this->assertEquals(strtotime($post->post_date_gmt), strtotime($published[0]['content']));

            // Category
            foreach ( get_the_category($post->ID) as $term ) {
                $terms[] = $term->name;

$terms was never initialized. Although not strictly required by PHP, it is generally a good practice to add $terms = array(); before regardless.

            }
            $categories = xml_find($entries[$key]['child'], 'category');

Array keys should be surrounded by spaces unless they contain a string or an integer.

            foreach ( $categories as $category ) {
                $this->assertTrue(in_array($category['attributes']['term'], $terms));

The variable $terms does not seem to be defined for all execution paths leading up to this point.

            }
            unset($terms);

            // Content
            if (! $this->excerpt_only ) {

This code seems to be duplicated across your project.

Expected 1 space before "!"; 0 found

                $content = xml_find($entries[$key]['child'], 'content');

Array keys should be surrounded by spaces unless they contain a string or an integer.

                $this->assertEquals(trim(apply_filters('the_content', $post->post_content)), trim($content[0]['content']));
            }

            // Link rel="replies"
            $link_replies = xml_find($entries[$key]['child'], 'link');

Array keys should be surrounded by spaces unless they contain a string or an integer.

            foreach ( $link_replies as $link_reply ) {
                if ('replies' == $link_reply['attributes']['rel'] && 'application/atom+xml' == $link_reply['attributes']['type'] ) {
                    $this->assertEquals(get_post_comments_feed_link($post->ID, 'atom'), $link_reply['attributes']['href']);
                }
            }
        }
    }
}