Changeset 32307 for branches/4.1/tests/phpunit/tests/comment.php

Timestamp:

04/27/2015 02:47:22 PM (11 years ago)

Author:

pento

Message:

WPDB: Sanity check that any strings being stored in the DB are not too long to store correctly.

Merge [32299] to the 4.1 and 4.2 branches.

File:

• branches/4.1/tests/phpunit/tests/comment.php (modified) (1 diff)

branches/4.1/tests/phpunit/tests/comment.php

@@ -43 +43 @@
                 $this->assertSame( array(), $found );
         }
+
+        public function test_comment_content_length() {
+                // `wp_new_comment()` checks REMOTE_ADDR, so we fake it to avoid PHP notices.
+                if ( isset( $_SERVER['REMOTE_ADDR'] ) ) {
+                        $remote_addr = $_SERVER['REMOTE_ADDR'];
+                } else {
+                        $_SERVER['REMOTE_ADDR'] = '';
+                }
+
+                $post_id = $this->factory->post->create();
+
+                $data = array(
+                        'comment_post_ID' => $post_id,
+                        'comment_author' => rand_str(),
+                        'comment_author_url' => '',
+                        'comment_author_email' => '',
+                        'comment_type' => '',
+                        'comment_content' => str_repeat( 'A', 65536 ),
+                        'comment_date' => '2011-01-01 10:00:00',
+                        'comment_date_gmt' => '2011-01-01 10:00:00',
+                );
+
+                $id = wp_new_comment( $data );
+
+                $this->assertFalse( $id );
+
+                // Cleanup.
+                if ( isset( $remote_addr ) ) {
+                        $_SERVER['REMOTE_ADDR'] = $remote_addr;
+                } else {
+                        unset( $_SERVER['REMOTE_ADDR'] );
+                }
+        }
 }