missed a test

broadinstitute · May 16, 2019 · 84f3683 · 84f3683
1 parent e8d9eb6
commit 84f3683
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/src/test/scala/org/broadinstitute/dsde/workbench/sam/google/GoogleExtensionSpec.scala b/src/test/scala/org/broadinstitute/dsde/workbench/sam/google/GoogleExtensionSpec.scala
@@ -664,8 +664,7 @@ class GoogleExtensionSpec(_system: ActorSystem) extends TestKit(_system) with Fl
 
     runAndWait(googleExtensions.onGroupUpdate(Seq(managedGroupRPN)))
 
-    // once when updating the managed group, and once when updating the policies it constrains
-    verify(mockGooglePubSubDAO, times(2)).publishMessages(any[String], any[Seq[String]])
+    verify(mockGooglePubSubDAO, times(1)).publishMessages(any[String], any[Seq[String]])
   }
 
   it should "trigger updates to constrained policies when updating a group that is a part of a managed group" in {
@@ -699,7 +698,6 @@ class GoogleExtensionSpec(_system: ActorSystem) extends TestKit(_system) with Fl
 
     runAndWait(googleExtensions.onGroupUpdate(Seq(WorkbenchGroupName(subGroupId))))
 
-    // once when updating the subgroup, and once when updating the policies constrained by the managed group
     verify(mockGooglePubSubDAO, times(1)).publishMessages(any[String], any[Seq[String]])
   }
 
@@ -735,7 +733,6 @@ class GoogleExtensionSpec(_system: ActorSystem) extends TestKit(_system) with Fl
 
     runAndWait(googleExtensions.onGroupUpdate(Seq(WorkbenchGroupName(subGroupId))))
 
-    // once when updating the subgroup, and once when updating the policies constrained by the managed group
     verify(mockGooglePubSubDAO, times(1)).publishMessages(any[String], any[Seq[String]])
   }