Mailing-List: contact harmony-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: harmony-dev@incubator.apache.org
Received-SPF: pass (asf.osuosl.org: domain of stepan.mishura@gmail.com
 designates 66.249.92.175 as permitted sender)
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
        s=beta; d=gmail.com;
        h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:references;
        b=MsEbmu6RA40V+99gYBzT+8cvkN3cUAmVaz6BmjjbKIIvu7gshGxOcdaBrBbO4OlGuVq2KlOcijPr6qBKAkQ7Nb1wuwYityubuKmgQJITbSL1yHZ6H84hONuCi17h4sLY6Ah2mnktAleBpeUZFptckAZ5ZIaQf1qbiSXy0JZblGs=
Message-ID: <6e47b64f0606260008q360793cta561ea8efba72104@mail.gmail.com>
Date: Mon, 26 Jun 2006 14:08:25 +0700
From: "Stepan Mishura" <stepan.mishura@gmail.com>
To: harmony-dev@incubator.apache.org
Subject: Re: [classlib] Merging frameworks for testing serialization - first
 step
In-Reply-To: <4d0b24970606232015t269be753ib80a583c0a46afec@mail.gmail.com>
MIME-Version: 1.0
Content-Type: multipart/alternative;
	boundary="----=_Part_6753_3113663.1151305705686"
References: <6e47b64f0606200111s3e702238hcf6a7e49482bf991@mail.gmail.com>
	 <6e47b64f0606222054l24b05625y7cdbc16f7f8d9b1b@mail.gmail.com>
	 <4d0b24970606232015t269be753ib80a583c0a46afec@mail.gmail.com>

------=_Part_6753_3113663.1151305705686
Content-Type: text/plain; charset=WINDOWS-1252; format=flowed
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline

On 6/24/06, Andrew Zhang wrote:
>
> Welldone stepan!
>
> I have a small question about the sample from serialization page[1]:
>
>
> public void testSerializationCompatibility()
>        throws Exception {
>
>    SerializationTest.verifyGolden(new SomeSerializableClass());
> }
>
> Any argument for  ****.golden.ser?  Typing error?


Good catch! I missed 'TestCase' param in the example - fixed in r417133.

Thanks,
Stepan.

Thanks!
> On 6/23/06, Stepan Mishura <stepan.mishura@gmail.com> wrote:
> >
> > Hi,
> >
> > I've updated framework for testing serialization page[1] - I added
> > guidelines
> > for developing serialization tests. Also I've removed confusing
> 'TestCase'
> > parameter in SerializationTest.verifySelf() methods.
> >
> > If there are no objections I'm going in next two days to move
> > SerializationTest.java from 'security' module to support folder. So new
> > location will be:
> > support/src/test/java/org/apache/harmony/testframework/serialization
> > folder.
> > Class name won't change.
> >
> > Thoughts?
> >
> > Thanks,
> > Stepan.
> >
> > [1]
> >
> >
> http://incubator.apache.org/harmony/subcomponents/classlibrary/ser_testin=
g.html
> >
> > On 6/20/06, Stepan Mishura  wrote:
> > >
> > >  Hi,
> > >
> > > I'm going to start merging existing frameworks for testing
> > serialization.
> > >
> > > As first step I've updated 'security' framework. The updated framewor=
k
> > > searches and loads resource files according [1] and eliminates
> > requirement
> > > to extend SerializationTest. Also to provide smooth frameworks mergin=
g
> > I've
> > > put stub to let the framework search resources in the 'old' way ( i.e=
.
> > via
> > > "RESOURCE_DIR" system property). The stub will be removed after
> > completing
> > > the merge.
> > >
> > > The updated framework suggests the following way for testing
> > > serialization:
> > >
> > > a) Compatibility =96 4 new static methods are introduced.
> > >     verifyGolden(TestCase, Object)
> > >     verifyGolden(TestCase, Object, SerializableAssert)
> > >     verifyGolden(TestCase, Object[])
> > >     verifyGolden(TestCase, Object[], SerializableAssert)
> > >
> > > A test should invoke one of above methods, for example,
> > > public void testCompatibility() throws Exception {
> > >     SerializationTest.verifyGolden(this, new SomeSerializableClass
> ());
> > > }
> > >
> > > b) Self-testing: the same as for compatibility =96 there are 4 new
> static
> > > methods that should be invoked from a test:
> > >     verifySelf(TestCase, Object)
> > >     verifySelf(Object, SerializableAssert)
> > >     verifySelf(TestCase, Object[])
> > >     verifySelf(Object[], SerializableAssert)
> > >
> > > For example,
> > > public void testSelf() throws Exception {
> > >     SerializationTest.verifySelf(new SomeSerializableClass(), new
> > > MyComparator());
> > > }
> > >
> > > To complete frameworks merging I'd like to suggest the next steps:
> > > 2) Reviewing the update and the suggested way for testing
> serialization
> > by
> > > the community. Please let me know if it is acceptable and what can be
> > > improved.
> > > 3) Replace SerializationTester class with SerializationTest. I'm goin=
g
> > to
> > > add more stubs to let existing tests work in the 'old' way.
> > > 4) Adjusting existing serialization tests (moving and renaming
> resource
> > > files, replacing stubs invocation with new methods)
> > > 5) Removing stubs.
> > >
> > > Thanks,
> > > Stepan Mishura
> > > Intel Middleware Products Division
> > >
> > > [1]
> > >
> >
> http://incubator.apache.org/harmony/subcomponents/classlibrary/ser_testin=
g.html
> > >
> > >
> > > ------------------------------------------------------
> > > 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.or=
g
> > >
> >
> >
> >
> > --
> > Thanks,
> > Stepan Mishura
> > Intel Middleware Products Division
> >
> > ------------------------------------------------------
> > 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
> >
> >
>
>
> --
> Andrew Zhang
> China Software Development Lab, IBM
>
>


--=20
Thanks,
Stepan Mishura
Intel Middleware Products Division

------------------------------------------------------
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

------=_Part_6753_3113663.1151305705686--