Skip to content

Commit

Permalink
added @ignore and comment
Browse files Browse the repository at this point in the history
  • Loading branch information
Stoffl committed Jun 25, 2014
1 parent dc28c68 commit 10a5ab6
Showing 1 changed file with 12 additions and 2 deletions.
14 changes: 12 additions & 2 deletions src/test/java/plugins/JabberPluginTest.java
Expand Up @@ -16,6 +16,7 @@
import org.jenkinsci.test.acceptance.plugins.jabber.JabberPublisher;
import org.jenkinsci.test.acceptance.plugins.jabber.JabberPublisher.Publishers;
import org.jenkinsci.test.acceptance.po.FreeStyleJob;
import org.junit.Ignore;
import org.junit.Test;

import java.io.File;
Expand All @@ -25,11 +26,20 @@

/**
* Feature: Tests for Jabber plugin
* This test case is set to @Ignore because the Jabber Plugin is not able to connect to conference.localhost
* because DNS resolution issues. These issues arise because of the used Smack API
* (http://www.igniterealtime.org/projects/smack/index.jsp)
* that does rely on successful DNS resolution, which is totally valid in terms on conformity with the XMPP Standard.
*
* This test is broken until a new version of the jabber plugin with a new Smack Library is provided that falls back to
* using the hostname if the FQDN could not be resolved, *
*
* @author jenky-hm
*
*/
@WithPlugins("jabber")
@Native("docker")
@Ignore
public class JabberPluginTest extends AbstractJUnitTest {
@Inject
private Docker docker;
Expand All @@ -40,9 +50,9 @@ public class JabberPluginTest extends AbstractJUnitTest {
private final String pwMucNameString = "test-room";
private final String pwMucPasswordString = "test-room-pw";
private final String localhostString = "localhost";
//private final String confRoom = "test@conference.localhost";
//private final String confRoom = "*test@conference.localhost";
//private final String confRoom = "*test@127.0.0.1";
private final String confRoom = "*test@conference.localhost";
private final String confRoom = "test@conference.localhost";

/**
@native(docker)
Expand Down

0 comments on commit 10a5ab6

Please sign in to comment.