Skip to content

Commit c4ec981

Browse files
committed
Fix test to run without broker in the background
1 parent ae02f2c commit c4ec981

File tree

1 file changed

+20
-1
lines changed

1 file changed

+20
-1
lines changed

src/test/java/com/rabbitmq/client/test/ConnectionFactoryTest.java

Lines changed: 20 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ protected AMQConnection createConnection(ConnectionParams params, FrameHandler f
6060
}
6161

6262
@Override
63-
protected synchronized FrameHandlerFactory createFrameHandlerFactory() throws IOException {
63+
protected synchronized FrameHandlerFactory createFrameHandlerFactory() {
6464
return mock(FrameHandlerFactory.class);
6565
}
6666
};
@@ -87,6 +87,11 @@ protected AMQConnection createConnection(ConnectionParams params, FrameHandler f
8787
createCalled.set(true);
8888
return connection;
8989
}
90+
91+
@Override
92+
protected synchronized FrameHandlerFactory createFrameHandlerFactory() {
93+
return mock(FrameHandlerFactory.class);
94+
}
9095
};
9196
connectionFactory.setCredentialsProvider(provider);
9297
connectionFactory.setAutomaticRecoveryEnabled(false);
@@ -114,7 +119,14 @@ protected AddressResolver createAddressResolver(List<Address> addresses) {
114119
addressResolver.set(super.createAddressResolver(addresses));
115120
return addressResolver.get();
116121
}
122+
123+
@Override
124+
protected synchronized FrameHandlerFactory createFrameHandlerFactory() {
125+
return mock(FrameHandlerFactory.class);
126+
}
117127
};
128+
// connection recovery makes the creation path more complex
129+
connectionFactory.setAutomaticRecoveryEnabled(false);
118130

119131
doNothing().when(connection).start();
120132
connectionFactory.newConnection();
@@ -138,7 +150,14 @@ protected AddressResolver createAddressResolver(List<Address> addresses) {
138150
addressResolver.set(super.createAddressResolver(addresses));
139151
return addressResolver.get();
140152
}
153+
154+
@Override
155+
protected synchronized FrameHandlerFactory createFrameHandlerFactory() {
156+
return mock(FrameHandlerFactory.class);
157+
}
141158
};
159+
// connection recovery makes the creation path more complex
160+
connectionFactory.setAutomaticRecoveryEnabled(false);
142161

143162
doNothing().when(connection).start();
144163
connectionFactory.useSslProtocol();

0 commit comments

Comments
 (0)