Fix failures with empty room keys.

Signed-off-by: Nicolas Werner <nicolas.werner@hotmail.de>
deepbluev7 · May 1, 2021 · 5d840b5 · 5d840b5
1 parent d0de4de
commit 5d840b5
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 11 deletions.
diff --git a/tests/30rooms/06invite.pl b/tests/30rooms/06invite.pl
@@ -159,8 +159,11 @@ sub invited_user_can_reject_invite
 
       # Check that invitee no longer sees the invite
 
-      assert_json_object( $body->{rooms}{invite} );
-      keys %{ $body->{rooms}{invite} } and die "Expected empty dictionary";
+      if( exists $body->{rooms} and exists $body->{rooms}{invite} ) {
+         assert_json_object( $body->{rooms}{invite} );
+         keys %{ $body->{rooms}{invite} } and die "Expected empty dictionary";
+      }
+
       Future->done(1);
    });
 }
@@ -203,8 +206,10 @@ sub invited_user_can_reject_invite_for_empty_room
 
       # Check that invitee no longer sees the invite
 
-      assert_json_object( $body->{rooms}{invite} );
-      keys %{ $body->{rooms}{invite} } and die "Expected empty dictionary";
+      if( exists $body->{rooms} and exists $body->{rooms}{invite} ) {
+         assert_json_object( $body->{rooms}{invite} );
+         keys %{ $body->{rooms}{invite} } and die "Expected empty dictionary";
+      }
       Future->done(1);
    });
 }
@@ -235,8 +240,12 @@ sub invited_user_can_reject_invite_for_empty_room
          my ( $body ) = @_;
 
          log_if_fail "Sync body", $body;
-         assert_json_object( $body->{rooms}{invite} );
-         keys %{ $body->{rooms}{invite} } and die "Expected empty dictionary";
+
+         if( exists $body->{rooms} and exists $body->{rooms}{invite} ) {
+            assert_json_object( $body->{rooms}{invite} );
+            keys %{ $body->{rooms}{invite} } and die "Expected empty dictionary";
+         }
+
          Future->done(1);
       });
    };

diff --git a/tests/31sync/09archived.pl b/tests/31sync/09archived.pl
@@ -258,10 +258,12 @@
       })->then( sub {
          my ( $body ) = @_;
 
-         my $leave = $body->{rooms}{leave};
+         if( exists $body->{rooms} and exists $body->{rooms}{leave} ) {
+            my $leave = $body->{rooms}{leave};
 
-         assert_json_object( $leave );
-         keys %$leave == 0 or die "Expected no rooms in 'leave' state";
+            assert_json_object( $leave );
+            keys %$leave == 0 or die "Expected no rooms in 'leave' state";
+         }
 
          Future->done(1);
       });

diff --git a/tests/31sync/15lazy-members.pl b/tests/31sync/15lazy-members.pl
@@ -190,8 +190,10 @@
          matrix_sync_again( $alice, filter => $filter_id );
       })->then( sub {
          my ( $body ) = @_;
-         my $joined_rooms = $body->{rooms}{join};
-         assert_deeply_eq($joined_rooms, {});
+         if( exists $body->{rooms} and exists $body->{rooms}{join} ) {
+            my $joined_rooms = $body->{rooms}{join};
+            assert_deeply_eq($joined_rooms, {});
+         }
          Future->done(1);
       });
    };