Changeset 11885 for trunk/tests/phpunit/includes/loader.php

Timestamp:

03/05/2018 12:00:25 AM (7 years ago)

Author:

r-a-y

Message:

Activity: Conditionally load action and screen functions.

This commit conditionally loads action and screen function code for the
Activity component, utilizing the 'bp_late_include' hook introduced in
r11884.

Previously, we loaded these functions at all times, which is unnecessary
when a user is not on an BuddyPress activity page. Now, we only load this
code when needed.

A special case is required for PHPUnit as PHPUnit doesn't like conditional-
included files when running multiple unit tests. So for PHPUnit, we
include all action and screen code at once, just like before.

See #7218.

File:

• trunk/tests/phpunit/includes/loader.php (modified) (1 diff)

trunk/tests/phpunit/includes/loader.php

@@ -17 +17 @@
 }
 tests_add_filter( 'bp_send_email_delivery_class', '_bp_mock_mailer' );
+
+/**
+ * Load up activity action and screen code.
+ *
+ * In BuddyPress, this is loaded conditionally, but PHPUnit needs all files
+ * loaded at the same time to prevent weird load order issues.
+ */
+add_action( 'bp_activity_includes', function() {
+    $dirs = array(
+        buddypress()->plugin_dir . 'bp-activity/actions/',
+        buddypress()->plugin_dir . 'bp-activity/screens/',
+    );
+
+    foreach ( $dirs as $dir ) {
+        foreach ( glob( $dir . "*.php" ) as $file ) {
+            require $file;
+        }
+    }
+} );