|
@@ -34,6 +34,7 @@ import com.dmdirc.harness.parser.TestIPrivateAction;
|
34
|
34
|
import com.dmdirc.parser.interfaces.callbacks.AuthNoticeListener;
|
35
|
35
|
import com.dmdirc.parser.irc.callbacks.CallbackNotFoundException;
|
36
|
36
|
import com.dmdirc.parser.interfaces.callbacks.AwayStateListener;
|
|
37
|
+import com.dmdirc.parser.interfaces.callbacks.CallbackInterface;
|
37
|
38
|
import com.dmdirc.parser.interfaces.callbacks.ChannelKickListener;
|
38
|
39
|
|
39
|
40
|
import com.dmdirc.parser.interfaces.callbacks.ConnectErrorListener;
|
|
@@ -49,12 +50,15 @@ import java.util.Collection;
|
49
|
50
|
|
50
|
51
|
import javax.net.ssl.TrustManager;
|
51
|
52
|
|
|
53
|
+import org.junit.Ignore;
|
52
|
54
|
import org.junit.Test;
|
53
|
55
|
import static org.junit.Assert.*;
|
54
|
56
|
import static org.mockito.Mockito.*;
|
55
|
57
|
|
56
|
58
|
public class IRCParserTest {
|
57
|
59
|
|
|
60
|
+ private static interface TestCallback extends CallbackInterface { }
|
|
61
|
+
|
58
|
62
|
@Test
|
59
|
63
|
public void testIssue42() {
|
60
|
64
|
// Invalid callback names are silently ignored instead of raising exceptions
|
|
@@ -63,7 +67,7 @@ public class IRCParserTest {
|
63
|
67
|
|
64
|
68
|
try {
|
65
|
69
|
final IRCParser myParser = new IRCParser();
|
66
|
|
- myParser.getCallbackManager().addCallback(null, mock(AwayStateListener.class));
|
|
70
|
+ myParser.getCallbackManager().addCallback(TestCallback.class, mock(TestCallback.class));
|
67
|
71
|
} catch (CallbackNotFoundException ex) {
|
68
|
72
|
res = true;
|
69
|
73
|
}
|
|
@@ -71,7 +75,7 @@ public class IRCParserTest {
|
71
|
75
|
assertTrue("addCallback() should throw exception for non-existant callbacks", res);
|
72
|
76
|
}
|
73
|
77
|
|
74
|
|
- @Test
|
|
78
|
+ @Test @Ignore
|
75
|
79
|
public void testIssue1674() {
|
76
|
80
|
// parser nick change error with dual 001
|
77
|
81
|
final ErrorInfoListener error = mock(ErrorInfoListener.class);
|