Return-Path: X-Original-To: apmail-camel-dev-archive@www.apache.org Delivered-To: apmail-camel-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 36E549B0B for ; Tue, 24 Jan 2012 15:36:44 +0000 (UTC) Received: (qmail 39725 invoked by uid 500); 24 Jan 2012 15:36:44 -0000 Delivered-To: apmail-camel-dev-archive@camel.apache.org Received: (qmail 39522 invoked by uid 500); 24 Jan 2012 15:36:43 -0000 Mailing-List: contact dev-help@camel.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@camel.apache.org Delivered-To: mailing list dev@camel.apache.org Received: (qmail 39508 invoked by uid 99); 24 Jan 2012 15:36:43 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 24 Jan 2012 15:36:43 +0000 X-ASF-Spam-Status: No, hits=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of hzbarcea@gmail.com designates 209.85.213.45 as permitted sender) Received: from [209.85.213.45] (HELO mail-yw0-f45.google.com) (209.85.213.45) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 24 Jan 2012 15:36:38 +0000 Received: by yhr47 with SMTP id 47so1900682yhr.32 for ; Tue, 24 Jan 2012 07:36:17 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=message-id:date:from:user-agent:mime-version:to:subject:references :in-reply-to:content-type:content-transfer-encoding; bh=FW//JIOhbCPrm9SpihX/SE94j0E523jb0+aIh9CQaKc=; b=fpekEWzOWvAEbGuBKotd6NNGtE4rD4yUFCq0RsffvWIANT87POp7vjy/4MVGi5jelQ i3twuDrUlHExKJzA3pBZKu8DQX2fRocd9cNv1bTzCZ/17CjexqnBr2QzLfdpi9lXz4sM P4NVWyd6LzKdy8sQApS5p8VvlPzMNsyolFAL8= Received: by 10.236.139.193 with SMTP id c41mr18994708yhj.24.1327419377342; Tue, 24 Jan 2012 07:36:17 -0800 (PST) Received: from [10.40.58.206] (cpe-071-070-182-182.nc.res.rr.com. [71.70.182.182]) by mx.google.com with ESMTPS id a24sm45646086ana.13.2012.01.24.07.36.15 (version=SSLv3 cipher=OTHER); Tue, 24 Jan 2012 07:36:15 -0800 (PST) Message-ID: <4F1ECFEE.2040105@gmail.com> Date: Tue, 24 Jan 2012 10:36:14 -0500 From: Hadrian Zbarcea User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:8.0) Gecko/20111124 Thunderbird/8.0 MIME-Version: 1.0 To: dev@camel.apache.org Subject: Re: [DISCUSS] - Add camel-test-spring component References: <4F1EC9B0.6050503@gmail.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 8bit Ok, let me clarify, there are a few points made here. Making camel-test standalone w/o spring dependencies is great, we should do that. Having a camel-test-spring is ok too and as you mentioned the primary audience for that would be Spring *users*. The point I was making is that, internally, we should only use camel-test-spring for integration tests (that's where we would 'eat our own dog food'), other than that it is not necessary (maybe with very few exceptions, like jms) and we should use just use camel-test (w/o spring). As cmueller already mentioned, we badly need to improve the testing time. As a very notable example, camel-cxf went for being the 4th test time consuming component (16+ min) to test in a bit over 1 min all thanks to dkulp's improvements. It's not only adding features that's important. Hadrian On 01/24/2012 10:14 AM, Claus Ibsen wrote: > On Tue, Jan 24, 2012 at 4:09 PM, Hadrian Zbarcea wrote: >> Actully, imo, a camel-test-spring should only be used for integration and by >> users, not for unit tests. >> > > Do you mean Camel unit testing itself? Why should we not eat our own dog food? > > > camel-test-spring, is for any kind of Camel + Spring testing, whether > its integration or unit test, etc. > The test kit, has a base class, CamelSpringTestSupport, that you > extend, and then make it easier to test. > Just as we got today. > > The proposal is to make the camel-test standalone, so it does not drag > in Spring JARs, for the growing number > of people who are not using Spring. And have a camel-test-spring > component for the Spring users. > > > >> Hadrian >> >> >> On 01/24/2012 06:16 AM, Claus Ibsen wrote: >>> >>> On Tue, Jan 24, 2012 at 11:59 AM, Christian M�ller >>> wrote: >>>> >>>> In generall a good idea. But I doesn't like to lose the possibilities to >>>> use the annotations like @EnpointInject, ... >>>> >>> >>> https://issues.apache.org/jira/browse/CAMEL-4934 will allow to use the >>> Camel @EndpointInject annotations in pure Java code (eg no Spring). >>> I am working on this right now. >>> >>> >>>> One of my bigger intentions for Camel 2.10.0 is to speed up the tests (if >>>> it's possible). In the past we was successful to reduce the time our unit >>>> test needs. At present I don't have a good idea how, but I have a few >>>> things I will try. >>>> One of the things I like on the pure Spring test support is the >>>> "@DirtiesContext" annotation. May we could have simmilar things in Camel >>>> to >>>> only boot up a new CamelContext if it's needed. >>>> Another thing is to remove duplicated tests, combine similar tests, use >>>> plain junit tests where it's possible, ... >>>> But this is outside auf this thread scope... >>>> >>>> Best, >>>> Christian >>>> >>>> On Tue, Jan 24, 2012 at 10:15 AM, Claus Ibsen >>>> wrote: >>>> >>>>> Hi >>>>> >>>>> In the recent work by GNodet to add a new camel-test-blueprint, which >>>>> I have recently polished. I noticed that the camel-test classes for >>>>> CamelTestSupport has dependency on Spring JARs. This is not the >>>>> intent, as there is a CamelSpringTestSupport people should use if they >>>>> use Spring. >>>>> >>>>> So I wonder if it would make sense for us to split camel-test into >>>>> - camel-test >>>>> - camel-test-spring >>>>> >>>>> eg to move the Spring Test support to a new component. >>>>> >>>>> Then we have a vanilla camel-test component that has just dependency on >>>>> JUnit. >>>>> This may aid end users, who are not using Spring, that we drag in >>>>> Spring JARs when they use camel-test. >>>>> >>>>> Any thoughts? >>>>> >>>>> >>>>> >>>>> >>>>> -- >>>>> Claus Ibsen >>>>> ----------------- >>>>> FuseSource >>>>> Email: cibsen@fusesource.com >>>>> Web: http://fusesource.com >>>>> Twitter: davsclaus, fusenews >>>>> Blog: http://davsclaus.blogspot.com/ >>>>> Author of Camel in Action: http://www.manning.com/ibsen/ >>>>> >>> >>> >>> >> >> -- >> Hadrian Zbarcea >> Principal Software Architect >> Talend, Inc >> http://coders.talend.com/ >> http://camelbot.blogspot.com/ > > > -- Hadrian Zbarcea Principal Software Architect Talend, Inc http://coders.talend.com/ http://camelbot.blogspot.com/