Return-Path: X-Original-To: apmail-incubator-ooo-dev-archive@minotaur.apache.org Delivered-To: apmail-incubator-ooo-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 13204918A for ; Mon, 16 Jul 2012 05:40:18 +0000 (UTC) Received: (qmail 31311 invoked by uid 500); 16 Jul 2012 05:40:17 -0000 Delivered-To: apmail-incubator-ooo-dev-archive@incubator.apache.org Received: (qmail 31206 invoked by uid 500); 16 Jul 2012 05:40:16 -0000 Mailing-List: contact ooo-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: ooo-dev@incubator.apache.org Delivered-To: mailing list ooo-dev@incubator.apache.org Received: (qmail 31166 invoked by uid 99); 16 Jul 2012 05:40:15 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 16 Jul 2012 05:40:15 +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 jogischmidt@googlemail.com designates 209.85.212.177 as permitted sender) Received: from [209.85.212.177] (HELO mail-wi0-f177.google.com) (209.85.212.177) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 16 Jul 2012 05:40:09 +0000 Received: by wibhm11 with SMTP id hm11so1808489wib.0 for ; Sun, 15 Jul 2012 22:39:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=googlemail.com; s=20120113; h=message-id:date:from:user-agent:mime-version:to:subject:references :in-reply-to:content-type:content-transfer-encoding; bh=mdXHC2hkuHuvxGY/q9q4qXTQj55Wqgt9Nypf7ljrcI4=; b=PluVRxwiXP8rYfyZMl0Vwfu+hTP5TZXHhVeovPB8OfzZuBsiPYuSo/g963Ih3ItM3v sy+bbcdPPOP5EtDoyoR9WzbTKRvYhOOJXHJieZB7bxDwcyHJaUQaUxt4hlxguDvWnhpz z3/erR4OWxEtyCC3z64P/ZmJDyR/xlvQBsGmM7p3L9SYaF4T7oAelkzBTmbQxey0SETw OCozPpQB8/dYnNNfCk9htgJ4OCqzAZqb7YtsjbvXApS9/QRBiOJZ3/YGm2ByeiwwNtYl fLx3hlmp67FEzmahTvCsHL1qyNtupKFUKzrOrLnfsohHu4UsOj6U4OHq9QJNhpbu93pv 6+4A== Received: by 10.180.81.138 with SMTP id a10mr15339934wiy.7.1342417188095; Sun, 15 Jul 2012 22:39:48 -0700 (PDT) Received: from [9.155.131.47] (deibp9eh1--blueice2n2.emea.ibm.com. [195.212.29.172]) by mx.google.com with ESMTPS id y2sm19751774wix.7.2012.07.15.22.39.47 (version=SSLv3 cipher=OTHER); Sun, 15 Jul 2012 22:39:47 -0700 (PDT) Message-ID: <5003A92B.8030404@googlemail.com> Date: Mon, 16 Jul 2012 07:39:55 +0200 From: =?ISO-8859-1?Q?J=FCrgen_Schmidt?= User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:13.0) Gecko/20120614 Thunderbird/13.0.1 MIME-Version: 1.0 To: ooo-dev@incubator.apache.org Subject: Re: [QA][DISCUSS]Refactor automated test code. References: In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org On 7/16/12 4:41 AM, Zhe Liu wrote: > Hi, > Our project is loaded with automated testing code, but it's in a mess > and out of maintain. Even the most are not stable enough to give > useful results. The testing I mentioned here requires a running > OpenOffice instances, not is the low level unit test during building, > e.g. UNO API test, complex test and gui test. > I propose to refactor it. I wrote a wiki page to introduce it. > http://wiki.services.openoffice.org/wiki/Test_Refactor > The key points include: > 1. Keep only Java as programming language and base on JUnit 4. It can > reduce our maintain effort. Currently our test code is written in 3 > languages (Basic, C++ and Java). > 2. Reduces modules related to test. > 3. Readable code and unified style. For UNO API, if test code is > enough readable, it's also a good code example for OpenOffice > developers. > One problem is that the refactor task requires us to do a lot of work. > What's your opinion? > > refactoring the whole area makes a lot of sense to me. We can only benefit from a cleaner tooling here. Something that is maintained and documented. Hopefully writing some new tests becomes easy enough that people get started to contribute to this effort ;-) Juergen