JAMES-1904 Increase test coverage on XMLDomainList
Project: http://git-wip-us.apache.org/repos/asf/james-project/repo Commit: http://git-wip-us.apache.org/repos/asf/james-project/commit/3e5a16d4 Tree: http://git-wip-us.apache.org/repos/asf/james-project/tree/3e5a16d4 Diff: http://git-wip-us.apache.org/repos/asf/james-project/diff/3e5a16d4 Branch: refs/heads/master Commit: 3e5a16d405c0ee4dbd353fb7c91698f33cf9528a Parents: 06b6e48 Author: Antoine Duprat <adup...@linagora.com> Authored: Wed Apr 19 13:47:49 2017 +0200 Committer: benwa <btell...@linagora.com> Committed: Thu Apr 20 07:47:38 2017 +0700 ---------------------------------------------------------------------- .../james/domainlist/xml/XMLDomainListTest.java | 34 ++++++++++++++++++++ 1 file changed, 34 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/james-project/blob/3e5a16d4/server/data/data-file/src/test/java/org/apache/james/domainlist/xml/XMLDomainListTest.java ---------------------------------------------------------------------- diff --git a/server/data/data-file/src/test/java/org/apache/james/domainlist/xml/XMLDomainListTest.java b/server/data/data-file/src/test/java/org/apache/james/domainlist/xml/XMLDomainListTest.java index 5d14c11..15a07ff 100644 --- a/server/data/data-file/src/test/java/org/apache/james/domainlist/xml/XMLDomainListTest.java +++ b/server/data/data-file/src/test/java/org/apache/james/domainlist/xml/XMLDomainListTest.java @@ -29,11 +29,17 @@ import org.apache.commons.configuration.DefaultConfigurationBuilder; import org.apache.commons.configuration.HierarchicalConfiguration; import org.apache.james.dnsservice.api.DNSService; import org.apache.james.dnsservice.api.mock.MockDNSService; +import org.apache.james.domainlist.api.DomainListException; +import org.junit.Rule; import org.junit.Test; +import org.junit.rules.ExpectedException; import org.slf4j.LoggerFactory; public class XMLDomainListTest { + @Rule + public ExpectedException expectedException = ExpectedException.none(); + private HierarchicalConfiguration setUpConfiguration(boolean auto, boolean autoIP, List<String> names) { DefaultConfigurationBuilder configuration = new DefaultConfigurationBuilder(); @@ -117,4 +123,32 @@ public class XMLDomainListTest { assertThat(dom.getDomains()).describedAs("One domain found").hasSize(1); } + + @Test + public void addDomainShouldFailWhenAlreadyConfigured() throws Exception { + expectedException.expect(DomainListException.class); + + List<String> domains = new ArrayList<String>(); + domains.add("domain1"); + + XMLDomainList testee = new XMLDomainList(); + testee.setLog(LoggerFactory.getLogger("MockLog")); + testee.configure(setUpConfiguration(true, false, domains)); + + testee.addDomain("newDomain"); + } + + @Test + public void removeDomainShouldFailWhenAlreadyConfigured() throws Exception { + expectedException.expect(DomainListException.class); + + List<String> domains = new ArrayList<String>(); + domains.add("domain1"); + + XMLDomainList testee = new XMLDomainList(); + testee.setLog(LoggerFactory.getLogger("MockLog")); + testee.configure(setUpConfiguration(true, false, domains)); + + testee.removeDomain("newDomain"); + } } --------------------------------------------------------------------- To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org