Revert "eth/filters: fix flaky test TestPendingTxFilterDeadlock (#28376)"

This reverts commit 492977c671.

eth/filters/filter_system_test.go

@@ -915,14 +915,10 @@ func TestPendingTxFilterDeadlock(t *testing.T) {
 
 	// Create a bunch of filters that will
 	// timeout either in 100ms or 200ms
-	subs := make([]*Subscription, 20)
+	fids := make([]rpc.ID, 20)
-	for i := 0; i < len(subs); i++ {
+	for i := 0; i < len(fids); i++ {
 		fid := api.NewPendingTransactionFilter(nil)
-		f, ok := api.filters[fid]
+		fids[i] = fid
-		if !ok {
-			t.Fatalf("Filter %s should exist", fid)
-		}
-		subs[i] = f.s
 		// Wait for at least one tx to arrive in filter
 		for {
 			hashes, err := api.GetFilterChanges(fid)
@@ -936,13 +932,21 @@ func TestPendingTxFilterDeadlock(t *testing.T) {
 		}
 	}
 
-	// Wait until filters have timed out and have been uninstalled.
+	// Wait until filters have timed out
-	for _, sub := range subs {
+	time.Sleep(3 * timeout)
-		select {
+
-		case <-sub.Err():
+	// If tx loop doesn't consume `done` after a second
-		case <-time.After(1 * time.Second):
+	// it's hanging.
-			t.Fatalf("Filter timeout is hanging")
+	select {
+	case done <- struct{}{}:
+		// Check that all filters have been uninstalled
+		for _, fid := range fids {
+			if _, err := api.GetFilterChanges(fid); err == nil {
+				t.Errorf("Filter %s should have been uninstalled\n", fid)
+			}
 		}
+	case <-time.After(1 * time.Second):
+		t.Error("Tx sending loop hangs")
 	}
 }