Re: [Lxr-dev] [Lxr-commits] CVS: lxr/tests/test-src - New directory

[Malcolm B. <mal...@gm...>]

On Sun, Apr 19, 2009 at 9:58 AM, Adrian Issott <adr...@ho...> wrote:
> Hi Malcolm,
>
> To give you some additional background on my change, I saw the line "$root =
> "/Users/malcolmbox/dev/lxr-src/";" in tests\PlainTest.pm and assumed that
> this was something on your local machine that hadn't been submitted to CVS.

Yes it was/is - shows that I've been the only person running these
tests I guess!

> Anyhow, I'm happy to move the test-scr folder (or whatever you were using
> originally) into the lxr-tools module provided we update the tests to check
> the test src folder is actually present and die if it isn't.

I think that's a good plan - there are various other test files in
there already to test for some previous bugs (e.g. filenames with
spaces, directories containing directories of same name etc).

> I'm not sure about moving the test scripts themselves - we don't want to
> give people the impression there are no tests but then the scripts should
> really be in the same module as the test src so they are keep in sync. I
> know - how about we move everything to do with testing into the lxr-tools
> module and just leave a note in the current test dir in the lxr module
> saying how to get the tests?

Good plan - there's a readme in the test directory that is a perfect
place for that.  Also the makerelease.pl script in lxr-tools will need
updating as it calls the tests.

Malcolm

>
>> From: mal...@gm...
>> Date: Fri, 17 Apr 2009 11:29:11 +0100
>> To: lxr...@li...
>> Subject: Re: [Lxr-dev] [Lxr-commits] CVS: lxr/tests/test-src - New
>> directory
>>
>> Hi Adrian,
>>
>> I've been moving directories/files for testing to the lxr-tools module
>> in the test-data directory - it looks like this commit is adding some
>> test files into the main lxr module.
>>
>> The main reason for this is to separate out the development
>> infrastructure from the released code - although I haven't moved the
>> actual test code yet (perhaps this would be worth doing).
>>
>> How do you feel about moving this across to the lxr-tools module?
>>
>> Cheers,
>>
>> Malcolm
>>
>> On Fri, Apr 10, 2009 at 9:29 PM, AdrianIssott
>> <adr...@us...> wrote:
>> > Update of /cvsroot/lxr/lxr/tests/test-src
>> > In directory
>> > ddv4jf1.ch3.sourceforge.com:/tmp/cvs-serv10674/tests/test-src
>> >
>> > Log Message:
>> > Directory /cvsroot/lxr/lxr/tests/test-src added to the repository
>> >
>> >
>> >
>> >
>> > ------------------------------------------------------------------------------
>> > Stay on top of everything new and different, both inside and
>> > around Java (TM) technology - register by April 22, and save
>> > $200 on the JavaOne (SM) conference, June 2-5, 2009, San Francisco.
>> > 300 plus technical and hands-on sessions. Register today.
>> > Use priority code J9JMT32. http://p.sf.net/sfu/p
>> > _______________________________________________
>> > Lxr-commits mailing list
>> > Lxr...@li...
>> > https://lists.sourceforge.net/lists/listinfo/lxr-commits
>> >
>>
>>
>> ------------------------------------------------------------------------------
>> Stay on top of everything new and different, both inside and
>> around Java (TM) technology - register by April 22, and save
>> $200 on the JavaOne (SM) conference, June 2-5, 2009, San Francisco.
>> 300 plus technical and hands-on sessions. Register today.
>> Use priority code J9JMT32. http://p.sf.net/sfu/p
>> _______________________________________________
>> Lxr-developer mailing list
>> Lxr...@li...
>> https://lists.sourceforge.net/lists/listinfo/lxr-developer
>
> ________________________________
> Windows Live Messenger just got better. Find out more!