This is an automated email from the ASF dual-hosted git repository.
rcordier pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/james-project.git
The following commit(s) were added to refs/heads/master by this push:
new aba90e1b8b Update JamesServerExtension
aba90e1b8b is described below
commit aba90e1b8b126f8ec2b7ef22150855bb1023750b
Author: hung phan <[email protected]>
AuthorDate: Tue Mar 5 11:20:36 2024 +0700
Update JamesServerExtension
---
.../org/apache/james/JamesServerExtension.java | 50 ++++++++++++++++------
1 file changed, 36 insertions(+), 14 deletions(-)
diff --git
a/server/container/guice/common/src/test/java/org/apache/james/JamesServerExtension.java
b/server/container/guice/common/src/test/java/org/apache/james/JamesServerExtension.java
index e473bf9f47..85ff5ae53b 100644
---
a/server/container/guice/common/src/test/java/org/apache/james/JamesServerExtension.java
+++
b/server/container/guice/common/src/test/java/org/apache/james/JamesServerExtension.java
@@ -56,11 +56,22 @@ public class JamesServerExtension implements
BeforeAllCallback, BeforeEachCallba
public enum Lifecycle {
// Restarts the server for each class, including nested classes
- PER_CLASS(JamesServerExtension::start, (extension, context) -> { },
(extension, context) -> { }, JamesServerExtension::stop),
+ PER_CLASS(
+ (extension, context) -> {
+ extension.setup(context);
+ extension.start(context);
+ },
+ (extension, context) -> { },
+ (extension, context) -> { },
+ (extension, context) -> {
+ extension.stop(context);
+ extension.tearDown(context);
+ }),
// Restarts the server for the enclosing class, it will ignore nested
classes
PER_ENCLOSING_CLASS(
(extension, context) -> {
if (!isNested(context)) {
+ extension.setup(context);
extension.start(context);
}
},
@@ -69,10 +80,15 @@ public class JamesServerExtension implements
BeforeAllCallback, BeforeEachCallba
(extension, context) -> {
if (!isNested(context)) {
extension.stop(context);
+ extension.tearDown(context);
}
}),
// Restarts the server for each test (default)
- PER_TEST((extension, context) -> { }, JamesServerExtension::start,
JamesServerExtension::stop, (extension, context) -> { });
+ PER_TEST(
+ JamesServerExtension::setup,
+ JamesServerExtension::start,
+ JamesServerExtension::stop,
+ JamesServerExtension::tearDown);
private static boolean isNested(ExtensionContext context) {
return context.getTestClass()
@@ -121,7 +137,6 @@ public class JamesServerExtension implements
BeforeAllCallback, BeforeEachCallba
@Override
public void beforeAll(ExtensionContext extensionContext) throws Exception {
- registrableExtension.beforeAll(extensionContext);
lifecycle.beforeAll.accept(this, extensionContext);
}
@@ -130,6 +145,24 @@ public class JamesServerExtension implements
BeforeAllCallback, BeforeEachCallba
lifecycle.beforeEach.accept(this, extensionContext);
}
+ @Override
+ public void afterEach(ExtensionContext extensionContext) throws Exception {
+ lifecycle.afterEach.accept(this, extensionContext);
+ }
+
+ @Override
+ public void afterAll(ExtensionContext extensionContext) throws Exception {
+ lifecycle.afterAll.accept(this, extensionContext);
+ }
+
+ private void setup(ExtensionContext extensionContext) throws Exception {
+ registrableExtension.beforeAll(extensionContext);
+ }
+
+ private void tearDown(ExtensionContext extensionContext) throws Exception {
+ registrableExtension.afterAll(extensionContext);
+ }
+
private void start(ExtensionContext extensionContext) throws Exception {
folderRegistrableExtension.beforeEach(extensionContext);
registrableExtension.beforeEach(extensionContext);
@@ -150,23 +183,12 @@ public class JamesServerExtension implements
BeforeAllCallback, BeforeEachCallba
}
}
- @Override
- public void afterEach(ExtensionContext extensionContext) throws Exception {
- lifecycle.afterEach.accept(this, extensionContext);
- }
-
private void stop(ExtensionContext extensionContext) throws Exception {
guiceJamesServer.stop();
registrableExtension.afterEach(extensionContext);
folderRegistrableExtension.afterEach(extensionContext);
}
- @Override
- public void afterAll(ExtensionContext extensionContext) throws Exception {
- lifecycle.afterAll.accept(this, extensionContext);
- registrableExtension.afterAll(extensionContext);
- }
-
@Override
public boolean supportsParameter(ParameterContext parameterContext,
ExtensionContext extensionContext) throws ParameterResolutionException {
return (parameterContext.getParameter().getType() ==
GuiceJamesServer.class)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
