Add test for a normal dispatch activity

This commit is contained in:
Alex Kirk 2022-12-09 13:16:34 +01:00
parent 4d05d3710b
commit 05575fe6e7
2 changed files with 67 additions and 33 deletions

View file

@ -30,14 +30,8 @@ class Activity_Dispatcher {
$activitypub_activity = new \Activitypub\Model\Activity( 'Create', \Activitypub\Model\Activity::TYPE_FULL ); $activitypub_activity = new \Activitypub\Model\Activity( 'Create', \Activitypub\Model\Activity::TYPE_FULL );
$activitypub_activity->from_post( $activitypub_post ); $activitypub_activity->from_post( $activitypub_post );
$sent = array(); $inboxes = \Activitypub\get_follower_inboxes( $user_id );
foreach ( \Activitypub\get_follower_inboxes( $user_id ) as $inbox => $to ) {
$sent[ $to ] = true;
$activitypub_activity->set_to( $to );
$activity = $activitypub_activity->to_json(); // phpcs:ignore
\Activitypub\safe_remote_post( $inbox, $activity, $user_id );
}
$followers_url = \get_rest_url( null, '/activitypub/1.0/users/' . intval( $user_id ) . '/followers' ); $followers_url = \get_rest_url( null, '/activitypub/1.0/users/' . intval( $user_id ) . '/followers' );
foreach ( $activitypub_activity->get_cc() as $cc ) { foreach ( $activitypub_activity->get_cc() as $cc ) {
if ( $cc === $followers_url ) { if ( $cc === $followers_url ) {
@ -47,11 +41,17 @@ class Activity_Dispatcher {
if ( ! $inbox || \is_wp_error( $inbox ) ) { if ( ! $inbox || \is_wp_error( $inbox ) ) {
continue; continue;
} }
if ( isset( $sent[ $cc ] ) ) { // init array if empty
continue; if ( ! isset( $inboxes[ $inbox ] ) ) {
$inboxes[ $inbox ] = array();
} }
$sent[ $cc ] = true; $inboxes[ $inbox ][] = $cc;
$activity = $activitypub_activity->to_json(); // phpcs:ignore }
foreach ( $inboxes as $inbox => $to ) {
$to = array_unique( $to );
$activitypub_activity->set_to( $to );
$activity = $activitypub_activity->to_json();
\Activitypub\safe_remote_post( $inbox, $activity, $user_id ); \Activitypub\safe_remote_post( $inbox, $activity, $user_id );
} }

View file

@ -1,6 +1,44 @@
<?php <?php
class Test_Activitypub_Activity_Dispatcher extends WP_UnitTestCase { class Test_Activitypub_Activity_Dispatcher extends ActivityPub_TestCase_Cache_HTTP {
public static $users = array(); public static $users = array(
'username@example.org' => array(
'url' => 'https://example.org/users/username',
'inbox' => 'https://example.org/users/username/inbox',
'name' => 'username',
),
'jon@example.com' => array(
'url' => 'https://example.com/author/jon',
'inbox' => 'https://example.com/author/jon/inbox',
'name' => 'jon',
),
);
public function test_dispatch_activity() {
$followers = array( 'https://example.com/author/jon', 'https://example.org/users/username' );
\update_user_meta( 1, 'activitypub_followers', $followers );
$post = \wp_insert_post(
array(
'post_author' => 1,
'post_content' => 'hello',
)
);
$pre_http_request = new MockAction();
add_filter( 'pre_http_request', array( $pre_http_request, 'filter' ), 10, 3 );
$activitypub_post = new \Activitypub\Model\Post( $post );
\Activitypub\Activity_Dispatcher::send_post_activity( $activitypub_post );
$this->assertSame( 2, $pre_http_request->get_call_count() );
$all_args = $pre_http_request->get_args();
$first_call_args = array_shift( $all_args );
$this->assertEquals( 'https://example.com/author/jon/inbox', $first_call_args[2] );
$second_call_args = array_shift( $all_args );
$this->assertEquals( 'https://example.org/users/username/inbox', $second_call_args[2] );
remove_filter( 'pre_http_request', array( $pre_http_request, 'filter' ), 10 );
}
public function test_dispatch_mentions() { public function test_dispatch_mentions() {
$post = \wp_insert_post( $post = \wp_insert_post(
array( array(
@ -41,21 +79,11 @@ class Test_Activitypub_Activity_Dispatcher extends WP_UnitTestCase {
public function set_up() { public function set_up() {
parent::set_up(); parent::set_up();
add_filter( 'pre_http_request', array( get_called_class(), 'pre_http_request' ), 10, 3 );
add_filter( 'http_response', array( get_called_class(), 'http_response' ), 10, 3 );
add_filter( 'http_request_host_is_external', array( get_called_class(), 'http_request_host_is_external' ), 10, 2 );
add_filter( 'http_request_args', array( get_called_class(), 'http_request_args' ), 10, 2 );
add_filter( 'pre_get_remote_metadata_by_actor', array( get_called_class(), 'pre_get_remote_metadata_by_actor' ), 10, 2 ); add_filter( 'pre_get_remote_metadata_by_actor', array( get_called_class(), 'pre_get_remote_metadata_by_actor' ), 10, 2 );
_delete_all_posts(); _delete_all_posts();
} }
public function tear_down() { public function tear_down() {
remove_filter( 'pre_http_request', array( get_called_class(), 'pre_http_request' ) );
remove_filter( 'http_response', array( get_called_class(), 'http_response' ) );
remove_filter( 'http_request_host_is_external', array( get_called_class(), 'http_request_host_is_external' ) );
remove_filter( 'http_request_args', array( get_called_class(), 'http_request_args' ) );
remove_filter( 'pre_get_remote_metadata_by_actor', array( get_called_class(), 'pre_get_remote_metadata_by_actor' ) ); remove_filter( 'pre_get_remote_metadata_by_actor', array( get_called_class(), 'pre_get_remote_metadata_by_actor' ) );
parent::tear_down(); parent::tear_down();
} }
@ -64,6 +92,11 @@ class Test_Activitypub_Activity_Dispatcher extends WP_UnitTestCase {
if ( isset( self::$users[ $actor ] ) ) { if ( isset( self::$users[ $actor ] ) ) {
return self::$users[ $actor ]; return self::$users[ $actor ];
} }
foreach ( self::$users as $username => $data ) {
if ( $data['url'] === $actor ) {
return $data;
}
}
return $pre; return $pre;
} }
@ -79,6 +112,7 @@ class Test_Activitypub_Activity_Dispatcher extends WP_UnitTestCase {
} }
return $args; return $args;
} }
public static function pre_http_request( $preempt, $request, $url ) { public static function pre_http_request( $preempt, $request, $url ) {
return array( return array(
'headers' => array( 'headers' => array(