fix: update the pubsub fake executor to cancel a pending callables future when cancelled

hannahrogers-google · hannahrogers-google · commit a1e4a4b2ce3f · 2022-12-14T22:50:22.000Z
diff --git a/google-cloud-pubsub/src/test/java/com/google/cloud/pubsub/v1/FakeScheduledExecutorService.java b/google-cloud-pubsub/src/test/java/com/google/cloud/pubsub/v1/FakeScheduledExecutorService.java
@@ -258,6 +258,7 @@ public int compareTo(Delayed o) {
         @Override
         public boolean cancel(boolean mayInterruptIfRunning) {
           synchronized (this) {
+            future.cancel(mayInterruptIfRunning);
             cancelled.set(true);
             return !done.get();
           }

Original file line number	Diff line number	Diff line change
`@@ -258,6 +258,7 @@ public int compareTo(Delayed o) {`
`258`	`258`	`@Override`
`259`	`259`	`public boolean cancel(boolean mayInterruptIfRunning) {`
`260`	`260`	`synchronized (this) {`
	`261`	`+ future.cancel(mayInterruptIfRunning);`
`261`	`262`	`cancelled.set(true);`
`262`	`263`	`return !done.get();`
`263`	`264`	`}`