Merge pull request #648 from Gizra/remove-dead-code

Remove dead code in OgAccessHookTest

tests/src/Unit/OgAccessHookTest.php

@@ -44,17 +44,7 @@ public function testNotContentEntity($operation) {
   public function testGetEntityGroups($operation) {
     $this->user->hasPermission(OgAccess::ADMINISTER_GROUP_PERMISSION)->willReturn(TRUE);
     $user_entity_access = og_entity_access($this->groupContentEntity->reveal(), $operation, $this->user->reveal());
-
-    // @todo This is strange, 'view' is not part of the operations supplied by
-    //   ::permissionsProvider(). And why would a group administrator be allowed
-    //   access to all operations, except 'view'? Shouldn't this also return
-    //   'allowed'?
-    if ($operation == 'view') {
-      $this->assertTrue($user_entity_access->isNeutral());
-    }
-    else {
-      $this->assertTrue($user_entity_access->isAllowed());
-    }
+    $this->assertTrue($user_entity_access->isAllowed());
   }
 
 }