2 @before methods, not a good idea (#108502)

Fixes IpFilterRemoteAddressFilterTests testThatFilteringWorksByIp. There
were 2 "@ Before"-annotated methods that didn't play well together.

Fixes: #108441

Author: albertzaharovits

x-pack/plugin/security/src/test/java/org/elasticsearch/xpack/security/transport/netty4/IpFilterRemoteAddressFilterTests.java

@@ -41,13 +41,6 @@ public class IpFilterRemoteAddressFilterTests extends ESTestCase {
     private IpFilterRemoteAddressFilter handler;
     private ThreadPool threadPool;
 
-    @Override
-    @Before
-    public void setUp() throws Exception {
-        super.setUp();
-        this.threadPool = new TestThreadPool(getTestName());
-    }
-
     @After
     public void terminate() throws Exception {
         terminate(threadPool);
@@ -95,14 +88,14 @@ public void init() throws Exception {
             ipFilter.setBoundHttpTransportAddress(httpTransport.boundAddress());
         }
 
+        this.threadPool = new TestThreadPool(getTestName());
         if (isHttpEnabled) {
             handler = new IpFilterRemoteAddressFilter(ipFilter, IPFilter.HTTP_PROFILE_NAME, threadPool.getThreadContext());
         } else {
             handler = new IpFilterRemoteAddressFilter(ipFilter, "default", threadPool.getThreadContext());
         }
     }
 
-    @AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/108441")
     public void testThatFilteringWorksByIp() throws Exception {
         InetSocketAddress localhostAddr = new InetSocketAddress(InetAddresses.forString("127.0.0.1"), 12345);
         assertThat(handler.accept(mock(ChannelHandlerContext.class), localhostAddr), is(true));