Return-Path: Delivered-To: apmail-incubator-harmony-dev-archive@www.apache.org Received: (qmail 14116 invoked from network); 11 Apr 2006 10:11:41 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 11 Apr 2006 10:11:41 -0000 Received: (qmail 1516 invoked by uid 500); 11 Apr 2006 10:11:31 -0000 Delivered-To: apmail-incubator-harmony-dev-archive@incubator.apache.org Received: (qmail 1468 invoked by uid 500); 11 Apr 2006 10:11:30 -0000 Mailing-List: contact harmony-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: harmony-dev@incubator.apache.org Delivered-To: mailing list harmony-dev@incubator.apache.org Received: (qmail 1457 invoked by uid 99); 11 Apr 2006 10:11:30 -0000 Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 11 Apr 2006 03:11:30 -0700 X-ASF-Spam-Status: No, hits=1.4 required=10.0 tests=SPF_NEUTRAL X-Spam-Check-By: apache.org Received-SPF: neutral (asf.osuosl.org: 64.74.244.71 is neither permitted nor denied by domain of geir@pobox.com) Received: from [64.74.244.71] (HELO chi.mobile-health-diary.com) (64.74.244.71) by apache.org (qpsmtpd/0.29) with SMTP; Tue, 11 Apr 2006 03:11:28 -0700 Received: (qmail 9685 invoked from network); 11 Apr 2006 10:11:06 -0000 Received: from ool-43560634.dyn.optonline.net (HELO ?192.168.2.3?) (geir@67.86.6.52) by b014.internal.mobile-health-diary.com with SMTP; 11 Apr 2006 10:11:06 -0000 Message-ID: <443B8034.1040000@pobox.com> Date: Tue, 11 Apr 2006 06:08:52 -0400 From: Geir Magnusson Jr Reply-To: geir@pobox.com User-Agent: Thunderbird 1.5 (Windows/20051201) MIME-Version: 1.0 To: harmony-dev@incubator.apache.org Subject: Re: Long,long testcase name... References: <5c8e69f0604101940q6843d941n7ad41b05d78d3617@mail.gmail.com> <443B1CF7.5060003@pobox.com> <5c8e69f0604102043n17331ad6o75d56fc354d441ea@mail.gmail.com> In-Reply-To: <5c8e69f0604102043n17331ad6o75d56fc354d441ea@mail.gmail.com> Content-Type: text/plain; charset=GB2312 Content-Transfer-Encoding: 8bit X-Virus-Checked: Checked by ClamAV on apache.org X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N LvJimmy��Jing wrote: > 2006/4/11, Geir Magnusson Jr : >> testRequestPasswordAuthentication1() >> >> I mean, why do we want to embed all that crap in the testcase name? Who >> cares? The only person that will care is someone fixing when a testcase >> breaks, and they'll read the code... > > > To make our code easy and clear to read, a good naming convention is > necessary as I think, that's why I dare say > testRequestPasswordAuthentication1() is not so good for people to tell which > method is tested here. The current naming convention is not bad but take too > mang characters. > Your opinion?:) I think you are tugging my chain. You are suggesting that : test_requestPasswordAuthentication_java_lang_String_java_net_InetAddress_int_java_lang_String_java_lang_String_java_lang_String_java_net_URL_java_net_Authenticator_RequestorType() makes the code easy and clear to read, better than testRequestPasswordAuthentication1() (for example) especially since when you go look at the code you will see all package and argument information? Is there any situation you can imagine where knowing all of that information when reviewing a test run report is more useful than knowing that method X in class Y in package Z failed, where X, Y and Z are canconical forms of each? geir > > geir >> >> LvJimmy��Jing wrote: >>> Hi all: >>> >>> Following our testcase naming convention, I've find a name as: (hope >> it >>> won't break your screen :)) >>> public void >>> >> test_requestPasswordAuthentication_java_lang_String_java_net_InetAddress_int_java_lang_String_java_lang_String_java_lang_String_java_net_URL_java_net_Authenticator_RequestorType() >>> which is the testcase for >>> java.net.Authenticator.requestPasswordAuthentication( >>> String rHost, InetAddress rAddr, int rPort, String >> rProtocol, >>> String rPrompt, String rScheme, URL rURL, >>> Authenticator.RequestorType reqType); >>> and the class has another two method named >> requestPasswordAuthentication, >>> and only this method take URL and RequestorType as its parameters. >>> The name is somehow too long to see and read, and I guess we can >> find a >>> shorter one for it, >>> e.g.test_requestPasswordAuthentication_URL_RequestorType. As URL and >>> RequestorType can identify the exactly >>> method to test. Or a adjusted naming convention shall be take. Any >> opinions? >>> -- >>> >>> Best Regards! >>> >>> Jimmy, Jing Lv >>> China Software Development Lab, IBM >>> >> --------------------------------------------------------------------- >> Terms of use : http://incubator.apache.org/harmony/mailing.html >> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org >> For additional commands, e-mail: harmony-dev-help@incubator.apache.org >> >> > > > -- > > Best Regards! > > Jimmy, Jing Lv > China Software Development Lab, IBM --------------------------------------------------------------------- Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org For additional commands, e-mail: harmony-dev-help@incubator.apache.org