Do not pass in a default value.

matrix-org · Apr 9, 2020 · 9480cf3 · 9480cf3
1 parent 808d19d
commit 9480cf3
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 14 deletions.
diff --git a/tests/10apidoc/12device_management.pl b/tests/10apidoc/12device_management.pl
@@ -37,7 +37,7 @@ sub matrix_delete_device {
 }
 
 test "GET /device/{deviceId}",
-   requires => [ local_user_fixture( with_events => 0 ) ],
+   requires => [ local_user_fixture() ],
 
    do => sub {
       my ( $user ) = @_;
@@ -64,7 +64,7 @@ sub matrix_delete_device {
    };
 
 test "GET /device/{deviceId} gives a 404 for unknown devices",
-   requires => [ local_user_fixture( with_events => 0 ) ],
+   requires => [ local_user_fixture() ],
 
    do => sub {
       my ( $user ) = @_;
@@ -78,7 +78,7 @@ sub matrix_delete_device {
 
 
 test "GET /devices",
-   requires => [ local_user_fixture( with_events => 0 ) ],
+   requires => [ local_user_fixture() ],
 
    do => sub {
       my ( $user ) = @_;
@@ -136,7 +136,7 @@ sub matrix_delete_device {
    };
 
 test "PUT /device/{deviceId} updates device fields",
-   requires => [ local_user_fixture( with_events => 0 ) ],
+   requires => [ local_user_fixture() ],
 
    do => sub {
       my ( $user ) = @_;
@@ -172,7 +172,7 @@ sub matrix_delete_device {
    };
 
 test "PUT /device/{deviceId} gives a 404 for unknown devices",
-   requires => [ local_user_fixture( with_events => 0 ) ],
+   requires => [ local_user_fixture() ],
 
    do => sub {
       my ( $user ) = @_;
@@ -188,7 +188,7 @@ sub matrix_delete_device {
    };
 
 test "DELETE /device/{deviceId}",
-   requires => [ local_user_fixture( with_events => 0 ) ],
+   requires => [ local_user_fixture() ],
 
    do => sub {
       my ( $user ) = @_;
@@ -273,8 +273,8 @@ sub matrix_delete_device {
 #
 test "DELETE /device/{deviceId} requires UI auth user to match device owner",
    requires => [
-      local_user_fixture( with_events => 0 ),
-      local_user_fixture( with_events => 0 ),
+      local_user_fixture(),
+      local_user_fixture(),
    ],
 
    do => sub {
@@ -317,7 +317,7 @@ sub matrix_delete_device {
 
 
 test "DELETE /device/{deviceId} with no body gives a 401",
-   requires => [ local_user_fixture( with_events => 0 ) ],
+   requires => [ local_user_fixture() ],
 
    do => sub {
       my ( $user ) = @_;

diff --git a/tests/10apidoc/13ui-auth.pl b/tests/10apidoc/13ui-auth.pl
@@ -50,7 +50,7 @@ sub make_ticket_request
 }
 
 test "Interactive authentication types include SSO",
-   requires => [ local_user_fixture( with_events => 0 ) ],
+   requires => [ local_user_fixture() ],
 
    do => sub {
       my ( $user ) = @_;
@@ -84,7 +84,7 @@ sub make_ticket_request
 
 test "Can perform interactive authentication with SSO",
    requires => [
-      local_user_fixture( with_events => 0 ),
+      local_user_fixture(),
       $main::API_CLIENTS[0],
       $main::HOMESERVER_INFO[0],
    ],
@@ -143,7 +143,7 @@ sub make_ticket_request
 
 test "The user must be consistent through an interactive authentication session with SSO",
    requires => [
-      local_user_fixture( with_events => 0 ),
+      local_user_fixture(),
       $main::API_CLIENTS[0],
       $main::HOMESERVER_INFO[0],
    ],
@@ -212,7 +212,7 @@ sub make_ticket_request
 
 
 test "The operation must be consistent through an interactive authentication session",
-   requires => [ local_user_fixture( with_events => 0 ) ],
+   requires => [ local_user_fixture() ],
 
    do => sub {
       my ( $user ) = @_;

diff --git a/tests/30rooms/05aliases.pl b/tests/30rooms/05aliases.pl
@@ -345,7 +345,7 @@ sub _test_can_create_and_delete_alias {
    };
 
 test "Can delete canonical alias",
-   requires => [ local_user_fixture( with_events => 0 ), room_alias_fixture(),
+   requires => [ local_user_fixture(), room_alias_fixture(),
                  qw( can_create_room_alias )],
 
    do => sub {