Changeset 10647 for trunk/tests/phpunit/testcases/admin/functions.php

Timestamp:

03/09/2016 08:32:09 PM (9 years ago)

Author:

djpaul

Message:

Emails: fix situations not being correctly assigned to email posts.

Previously, we were using wp_set_post_terms() to fetch the terms to assign to an email post. However, it turns that that function returns term taxonomy IDs, not term IDs. Generally, term_id and term_taxonomy_id will be the same for newly created terms like these, but not when the database still contains shared taxonomy terms.

Fixes #6936

Props chherbst, boonebgorges

File:

• trunk/tests/phpunit/testcases/admin/functions.php (modified) (1 diff)

trunk/tests/phpunit/testcases/admin/functions.php

@@ -219 +219 @@
         $bp->admin->notices = $reset_admin_notices;
     }
+
+    /**
+     * @ticket BP6936
+     */
+    public function test_email_type_descriptions_should_match_when_split_terms_exist() {
+        global $wpdb;
+
+        // Delete all existing email types and descriptions.
+        $emails = get_posts( array(
+            'fields' => 'ids',
+            'post_type' => bp_get_email_post_type(),
+        ) );
+        foreach ( $emails as $email ) {
+            wp_delete_post( $email, true );
+        }
+
+        $descriptions = get_terms( bp_get_email_tax_type(), array(
+            'fields' => 'ids',
+            'hide_empty' => false,
+        ) );
+        foreach ( $descriptions as $description ) {
+            wp_delete_term( (int) $description, bp_get_email_tax_type() );
+        }
+
+        // Fake the existence of split terms by offsetting the term_taxonomy table.
+        $wpdb->insert( $wpdb->term_taxonomy, array( 'term_id' => 9999, 'taxonomy' => 'post_tag', 'description' => 'foo description', 'parent' => 0, 'count' => 0 ) );
+
+        require_once( BP_PLUGIN_DIR . '/bp-core/admin/bp-core-admin-schema.php' );
+        bp_core_install_emails();
+
+        $d_terms = get_terms( bp_get_email_tax_type(), array(
+            'hide_empty' => false,
+        ) );
+
+        $correct_descriptions = bp_email_get_type_schema();
+        foreach ( $d_terms as $d_term ) {
+            $correct_description = $correct_descriptions[ $d_term->slug ];
+            $this->assertSame( $correct_description, $d_term->description );
+        }
+    }
 }