Return-Path: Delivered-To: apmail-incubator-directory-dev-archive@www.apache.org Received: (qmail 38669 invoked from network); 9 Feb 2005 04:16:50 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur-2.apache.org with SMTP; 9 Feb 2005 04:16:50 -0000 Received: (qmail 83327 invoked by uid 500); 9 Feb 2005 04:16:50 -0000 Delivered-To: apmail-incubator-directory-dev-archive@incubator.apache.org Received: (qmail 83267 invoked by uid 500); 9 Feb 2005 04:16:49 -0000 Mailing-List: contact directory-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "Apache Directory Developers List" Delivered-To: mailing list directory-dev@incubator.apache.org Received: (qmail 83253 invoked by uid 99); 9 Feb 2005 04:16:49 -0000 X-ASF-Spam-Status: No, hits=0.5 required=10.0 tests=FROM_ENDS_IN_NUMS,SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (hermes.apache.org: domain of aok123@bellsouth.net designates 205.152.59.65 as permitted sender) Received: from imf17aec.mail.bellsouth.net (HELO imf17aec.mail.bellsouth.net) (205.152.59.65) by apache.org (qpsmtpd/0.28) with ESMTP; Tue, 08 Feb 2005 20:16:45 -0800 Received: from [172.16.1.7] ([65.80.200.112]) by imf17aec.mail.bellsouth.net (InterMail vM.5.01.06.11 201-253-122-130-111-20040605) with ESMTP id <20050209041643.OAIT1992.imf17aec.mail.bellsouth.net@[172.16.1.7]> for ; Tue, 8 Feb 2005 23:16:43 -0500 Message-ID: <42098EAB.3@bellsouth.net> Date: Tue, 08 Feb 2005 23:16:43 -0500 From: Alex Karasulu User-Agent: Mozilla Thunderbird 1.0 (X11/20041206) X-Accept-Language: en-us, en MIME-Version: 1.0 To: Apache Directory Developers List Subject: Re: Getting rid of snacc dependency in snickers test cases References: <42083FCD.6060500@videotron.ca> <420989C1.9090404@videotron.ca> In-Reply-To: <420989C1.9090404@videotron.ca> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked X-Spam-Rating: minotaur-2.apache.org 1.6.2 0/1000/N Vincent Tence wrote: > Status > ------ > > A lot of the test cases in svn have been migrated to using > pre-computed PDUs. This goes for encoders as well as decoders tests. > Tests have also been simplified to rely on the equals() implementation > of message classes - it'll test it for the same price. > > However, some of the test cases are causing trouble. The offending > test cases have an TODO: tag that will help identify them. For those > test, the migrated version is in comment until I can find out what's > going on. > > So far here's what I've learned about tests failing when migrated: > > 1. All decoder tests that use a response containing a BUSY result code > fail. The decoded message shows a SUCCESS result code. Something must be wrong with type safe enum ResultCodeEnum somewhere. > > 2. ModifyRequestTest fails for an unknown reason Any ideas on how we may isolate the problem here? > > 3. I'm not sure how to migrate the SearchRequestTest What's presenting a problem? > > 4. ExtendedResponseEncoderTest is a copy of ExtendedRequestEncoderTest > instead of a test of its own. Np just blow it away if you don't have time to implement the test. > > 5. Encoders generate PDUs that look equivalent yet are different from > snacc one's - a CHOICE order maybe? Yeah there are differences in the way elements are order like the order of attributes in a search entry response. Snacc uses a HashMap to store these where we use an ArrayList. Hence the difference in order. The only way to conduct tests halfway properly is to put elements into a Map and make sure both elements exist in both PDUs. Comparisons based on order will fail. -Alex