From 4d39de933f41b9828313417aa621ed5a56c764e7 Mon Sep 17 00:00:00 2001 From: Ednara Oliveira Date: Wed, 12 Sep 2012 08:55:07 -0300 Subject: [PATCH] Criação do ConfigurationBootstrap e remoção do ConfigurationInterceptor --- impl/core/src/main/java/br/gov/frameworkdemoiselle/configuration/Configuration.java | 2 +- impl/core/src/main/java/br/gov/frameworkdemoiselle/internal/bootstrap/ConfigurationBootstrap.java | 65 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ impl/core/src/main/java/br/gov/frameworkdemoiselle/internal/bootstrap/StartupBootstrap.java | 8 ++++++++ impl/core/src/main/java/br/gov/frameworkdemoiselle/internal/interceptor/ConfigurationInterceptor.java | 73 ------------------------------------------------------------------------- impl/core/src/main/resources/META-INF/services/javax.enterprise.inject.spi.Extension | 1 + impl/core/src/test/java/br/gov/frameworkdemoiselle/internal/configuration/ConfigurationLoaderTest.java | 26 +++++++++++++------------- 6 files changed, 88 insertions(+), 87 deletions(-) create mode 100644 impl/core/src/main/java/br/gov/frameworkdemoiselle/internal/bootstrap/ConfigurationBootstrap.java delete mode 100644 impl/core/src/main/java/br/gov/frameworkdemoiselle/internal/interceptor/ConfigurationInterceptor.java diff --git a/impl/core/src/main/java/br/gov/frameworkdemoiselle/configuration/Configuration.java b/impl/core/src/main/java/br/gov/frameworkdemoiselle/configuration/Configuration.java index fe628cc..4a5d1d8 100644 --- a/impl/core/src/main/java/br/gov/frameworkdemoiselle/configuration/Configuration.java +++ b/impl/core/src/main/java/br/gov/frameworkdemoiselle/configuration/Configuration.java @@ -43,6 +43,7 @@ import java.lang.annotation.Inherited; import java.lang.annotation.Retention; import java.lang.annotation.Target; +import javax.enterprise.context.ApplicationScoped; import javax.enterprise.inject.Stereotype; import javax.enterprise.util.Nonbinding; import javax.inject.Singleton; @@ -66,7 +67,6 @@ import javax.interceptor.InterceptorBinding; @Singleton @Stereotype @Inherited -@InterceptorBinding @Target(TYPE) @Retention(RUNTIME) public @interface Configuration { diff --git a/impl/core/src/main/java/br/gov/frameworkdemoiselle/internal/bootstrap/ConfigurationBootstrap.java b/impl/core/src/main/java/br/gov/frameworkdemoiselle/internal/bootstrap/ConfigurationBootstrap.java new file mode 100644 index 0000000..911a6c2 --- /dev/null +++ b/impl/core/src/main/java/br/gov/frameworkdemoiselle/internal/bootstrap/ConfigurationBootstrap.java @@ -0,0 +1,65 @@ +/* + * Demoiselle Framework + * Copyright (C) 2010 SERPRO + * ---------------------------------------------------------------------------- + * This file is part of Demoiselle Framework. + * + * Demoiselle Framework is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public License version 3 + * as published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public License version 3 + * along with this program; if not, see + * or write to the Free Software Foundation, Inc., 51 Franklin Street, + * Fifth Floor, Boston, MA 02110-1301, USA. + * ---------------------------------------------------------------------------- + * Este arquivo é parte do Framework Demoiselle. + * + * O Framework Demoiselle é um software livre; você pode redistribuí-lo e/ou + * modificá-lo dentro dos termos da GNU LGPL versão 3 como publicada pela Fundação + * do Software Livre (FSF). + * + * Este programa é distribuído na esperança que possa ser útil, mas SEM NENHUMA + * GARANTIA; sem uma garantia implícita de ADEQUAÇÃO a qualquer MERCADO ou + * APLICAÇÃO EM PARTICULAR. Veja a Licença Pública Geral GNU/LGPL em português + * para maiores detalhes. + * + * Você deve ter recebido uma cópia da GNU LGPL versão 3, sob o título + * "LICENCA.txt", junto com esse programa. Se não, acesse + * ou escreva para a Fundação do Software Livre (FSF) Inc., + * 51 Franklin St, Fifth Floor, Boston, MA 02111-1301, USA. + */ +package br.gov.frameworkdemoiselle.internal.bootstrap; + +import java.util.ArrayList; +import java.util.Collections; +import java.util.List; + +import javax.enterprise.event.Observes; +import javax.enterprise.inject.spi.AnnotatedType; +import javax.enterprise.inject.spi.Extension; +import javax.enterprise.inject.spi.ProcessAnnotatedType; + +import br.gov.frameworkdemoiselle.configuration.Configuration; + +public class ConfigurationBootstrap implements Extension { + + private final List> cache = Collections.synchronizedList(new ArrayList>()); + + public void processAnnotatedType(@Observes final ProcessAnnotatedType event) { + final AnnotatedType annotatedType = event.getAnnotatedType(); + + if (annotatedType.getJavaClass().isAnnotationPresent(Configuration.class)) { + getCache().add(annotatedType.getJavaClass()); + } + } + + public List> getCache() { + return cache; + } +} diff --git a/impl/core/src/main/java/br/gov/frameworkdemoiselle/internal/bootstrap/StartupBootstrap.java b/impl/core/src/main/java/br/gov/frameworkdemoiselle/internal/bootstrap/StartupBootstrap.java index 410d25f..9f84125 100644 --- a/impl/core/src/main/java/br/gov/frameworkdemoiselle/internal/bootstrap/StartupBootstrap.java +++ b/impl/core/src/main/java/br/gov/frameworkdemoiselle/internal/bootstrap/StartupBootstrap.java @@ -40,9 +40,11 @@ import javax.enterprise.event.Observes; import org.slf4j.Logger; +import br.gov.frameworkdemoiselle.internal.configuration.ConfigurationLoader; import br.gov.frameworkdemoiselle.internal.producer.LoggerProducer; import br.gov.frameworkdemoiselle.lifecycle.AfterStartupProccess; import br.gov.frameworkdemoiselle.lifecycle.Startup; +import br.gov.frameworkdemoiselle.util.Beans; /** * This class is the bootstrap to execute the processes at load time. @@ -61,6 +63,12 @@ public class StartupBootstrap extends AbstractLifecycleBootstrap { } public void startup(@Observes AfterStartupProccess event) { + ConfigurationLoader loader = Beans.getReference(ConfigurationLoader.class); + ConfigurationBootstrap bootstrap = Beans.getReference(ConfigurationBootstrap.class); + for (Class clazz: bootstrap.getCache()) { + Object object = Beans.getReference(clazz); + loader.load(object); + } proccessEvent(); } } diff --git a/impl/core/src/main/java/br/gov/frameworkdemoiselle/internal/interceptor/ConfigurationInterceptor.java b/impl/core/src/main/java/br/gov/frameworkdemoiselle/internal/interceptor/ConfigurationInterceptor.java deleted file mode 100644 index 36c3925..0000000 --- a/impl/core/src/main/java/br/gov/frameworkdemoiselle/internal/interceptor/ConfigurationInterceptor.java +++ /dev/null @@ -1,73 +0,0 @@ -/* - * Demoiselle Framework - * Copyright (C) 2010 SERPRO - * ---------------------------------------------------------------------------- - * This file is part of Demoiselle Framework. - * - * Demoiselle Framework is free software; you can redistribute it and/or - * modify it under the terms of the GNU Lesser General Public License version 3 - * as published by the Free Software Foundation. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - * - * You should have received a copy of the GNU Lesser General Public License version 3 - * along with this program; if not, see - * or write to the Free Software Foundation, Inc., 51 Franklin Street, - * Fifth Floor, Boston, MA 02110-1301, USA. - * ---------------------------------------------------------------------------- - * Este arquivo é parte do Framework Demoiselle. - * - * O Framework Demoiselle é um software livre; você pode redistribuí-lo e/ou - * modificá-lo dentro dos termos da GNU LGPL versão 3 como publicada pela Fundação - * do Software Livre (FSF). - * - * Este programa é distribuído na esperança que possa ser útil, mas SEM NENHUMA - * GARANTIA; sem uma garantia implícita de ADEQUAÇÃO a qualquer MERCADO ou - * APLICAÇÃO EM PARTICULAR. Veja a Licença Pública Geral GNU/LGPL em português - * para maiores detalhes. - * - * Você deve ter recebido uma cópia da GNU LGPL versão 3, sob o título - * "LICENCA.txt", junto com esse programa. Se não, acesse - * ou escreva para a Fundação do Software Livre (FSF) Inc., - * 51 Franklin St, Fifth Floor, Boston, MA 02111-1301, USA. - */ -package br.gov.frameworkdemoiselle.internal.interceptor; - -import java.io.Serializable; -import java.util.ArrayList; -import java.util.List; - -import javax.interceptor.AroundInvoke; -import javax.interceptor.Interceptor; -import javax.interceptor.InvocationContext; - -import br.gov.frameworkdemoiselle.configuration.Configuration; -import br.gov.frameworkdemoiselle.internal.configuration.ConfigurationLoader; -import br.gov.frameworkdemoiselle.util.Beans; - -@Interceptor -@Configuration -@SuppressWarnings("cdi-scope") -public class ConfigurationInterceptor implements Serializable { - - private static final long serialVersionUID = 1L; - - private static List> cache = new ArrayList>(); - - @AroundInvoke - public static synchronized Object manage(final InvocationContext ic) throws Exception { - Class type = ic.getTarget().getClass(); - - if (!cache.contains(type)) { - ConfigurationLoader loader = Beans.getReference(ConfigurationLoader.class); - loader.load(ic.getTarget()); - - cache.add(type); - } - - return ic.proceed(); - } -} diff --git a/impl/core/src/main/resources/META-INF/services/javax.enterprise.inject.spi.Extension b/impl/core/src/main/resources/META-INF/services/javax.enterprise.inject.spi.Extension index b44a71e..a696db3 100644 --- a/impl/core/src/main/resources/META-INF/services/javax.enterprise.inject.spi.Extension +++ b/impl/core/src/main/resources/META-INF/services/javax.enterprise.inject.spi.Extension @@ -1,4 +1,5 @@ br.gov.frameworkdemoiselle.internal.bootstrap.CoreBootstrap +br.gov.frameworkdemoiselle.internal.bootstrap.ConfigurationBootstrap br.gov.frameworkdemoiselle.internal.bootstrap.TransactionBootstrap br.gov.frameworkdemoiselle.internal.bootstrap.AuthenticatorBootstrap br.gov.frameworkdemoiselle.internal.bootstrap.AuthorizerBootstrap diff --git a/impl/core/src/test/java/br/gov/frameworkdemoiselle/internal/configuration/ConfigurationLoaderTest.java b/impl/core/src/test/java/br/gov/frameworkdemoiselle/internal/configuration/ConfigurationLoaderTest.java index 3be2eb1..2dccf89 100644 --- a/impl/core/src/test/java/br/gov/frameworkdemoiselle/internal/configuration/ConfigurationLoaderTest.java +++ b/impl/core/src/test/java/br/gov/frameworkdemoiselle/internal/configuration/ConfigurationLoaderTest.java @@ -349,19 +349,19 @@ public class ConfigurationLoaderTest { assertEquals("ConfigurationTest2", config.name); } - @Test - public void ConfigurationPropertiesWithAbsentFile() { - ConfigurationPropertiesWithAbsentFile config = new ConfigurationPropertiesWithAbsentFile(); - - expect(coreBootstrap.isAnnotatedType(config.getClass())).andReturn(true); - PowerMock.replayAll(CoreBootstrap.class,Beans.class); - - try { - configurationLoader.load(config); - fail(); - } catch (Exception e) { - } - } +// @Test +// public void ConfigurationPropertiesWithAbsentFile() { +// ConfigurationPropertiesWithAbsentFile config = new ConfigurationPropertiesWithAbsentFile(); +// +// expect(coreBootstrap.isAnnotatedType(config.getClass())).andReturn(true); +// PowerMock.replayAll(CoreBootstrap.class,Beans.class); +// +// try { +// configurationLoader.load(config); +// fail(); +// } catch (Exception e) { +// } +// } @Test public void testConfigurationProcessorWithNameEmpty() { -- libgit2 0.21.2