From 231bad3560f402228d88c6c8b2445eb4a0ba783e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Richard=20K=C3=B6rber?= Date: Mon, 16 Apr 2018 19:43:30 +0200 Subject: [PATCH] Enable unit test for Pebble account deactivation --- .../shredzone/acme4j/it/pebble/AccountIT.java | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/acme4j-it/src/test/java/org/shredzone/acme4j/it/pebble/AccountIT.java b/acme4j-it/src/test/java/org/shredzone/acme4j/it/pebble/AccountIT.java index c3924c0f..76292ed7 100644 --- a/acme4j-it/src/test/java/org/shredzone/acme4j/it/pebble/AccountIT.java +++ b/acme4j-it/src/test/java/org/shredzone/acme4j/it/pebble/AccountIT.java @@ -210,7 +210,6 @@ public class AccountIT extends PebbleITBase { * Deactivate an account. */ @Test - @Ignore // TODO PEBBLE: missing public void testDeactivate() throws AcmeException { KeyPair keyPair = createKeyPair(); Session session = new Session(pebbleURI()); @@ -223,10 +222,18 @@ public class AccountIT extends PebbleITBase { acct.deactivate(); - Session session2 = new Session(pebbleURI()); - Account acct2 = session2.login(location, keyPair).getAccount(); - assertThat(acct2.getLocation(), is(location)); - assertThat(acct2.getStatus(), is(Status.DEACTIVATED)); + // Make sure it is deactivated now... + assertThat(acct.getStatus(), is(Status.DEACTIVATED)); + + // Make sure account cannot be accessed any more... + try { + Session session2 = new Session(pebbleURI()); + Account acct2 = session2.login(location, keyPair).getAccount(); + acct2.update(); + fail("Account can still be accessed"); + } catch (AcmeUnauthorizedException ex) { + assertThat(ex.getMessage(), is("Account has been deactivated")); + } } }