← Back to team overview

nunit-core team mailing list archive

Re: [Bug 1057911] Re: Trap incorrect input for nunit-console.exe options.

 

I saw that. Thanks.

On Sun, Oct 21, 2012 at 2:26 PM, Simone Busoli
<1057911@xxxxxxxxxxxxxxxxxx> wrote:
> I pushed to trunk already btw.
>
> On Sun, Oct 21, 2012 at 10:57 PM, Charlie Poole <charlie@xxxxxxxxx>
> wrote:
>
>> Thanks, I'll do a final check after I merge everything.
>>
>> Charlie
>>
>> On Sun, Oct 21, 2012 at 1:42 PM, Simone Busoli
>> <1057911@xxxxxxxxxxxxxxxxxx> wrote:
>> > Should work now, and I shall not have broken the 1.1 build, or at least I
>> > hope so.
>> >
>> > On Sun, Oct 21, 2012 at 9:43 PM, Charlie Poole <charlie@xxxxxxxxx>
>> > wrote:
>> >
>> >> That's good. Meanwhile I'm working on cleaning up to prepare for the
>> >> actual release.
>> >>
>> >> Charlie
>> >>
>> >> On Sun, Oct 21, 2012 at 12:10 PM, Simone Busoli
>> >> <1057911@xxxxxxxxxxxxxxxxxx> wrote:
>> >> > I see, I can definitely create the files in the temp dir then.
>> >> >
>> >> > Simone
>> >> >
>> >> > On Sun, Oct 21, 2012 at 5:24 PM, Charlie Poole <charlie@xxxxxxxxx>
>> >> > wrote:
>> >> >
>> >> >> Because we distribute the tests and tell users to validate the
>> >> >> installation by running them!
>> >> >>
>> >> >> Charlie
>> >> >>
>> >> >> On Sun, Oct 21, 2012 at 8:02 AM, Simone Busoli
>> >> >> <1057911@xxxxxxxxxxxxxxxxxx> wrote:
>> >> >> > Charlie, these are files which are used during the test. Why would
>> you
>> >> >> run
>> >> >> > the tests from the installed package?
>> >> >> >
>> >> >> > On Sun, Oct 21, 2012 at 4:46 PM, Charlie Poole <charlie@xxxxxxxxx>
>> >> >> > wrote:
>> >> >> >
>> >> >> >> Simone, This fix is causing a problem in the final release
>> package,
>> >> when
>> >> >> >> NUnit is installed in the Program Files directory. We can't access
>> >> the
>> >> >> >> two text files you added due to security concerns. The best
>> solution
>> >> >> >> I've found in similar cases is to create the files on the fly in
>> the
>> >> >> >> temp directory and delete them afterwards. That would be best, if
>> you
>> >> >> >> have time. Otherwise, we can just drop those tests from the
>> release.
>> >> >> >>
>> >> >> >> Charlie
>> >> >> >>
>> >> >> >> --
>> >> >> >> You received this bug notification because you are a bug assignee.
>> >> >> >> https://bugs.launchpad.net/bugs/1057911
>> >> >> >>
>> >> >> >> Title:
>> >> >> >>   Trap incorrect input for nunit-console.exe options.
>> >> >> >>
>> >> >> >> Status in NUnit V2 Test Framework:
>> >> >> >>   Fix Committed
>> >> >> >>
>> >> >> >> Bug description:
>> >> >> >>   nunit-console.exe should handle incorrect options gracefully.
>>  As
>> >> of
>> >> >> >>   now, it either runs 0 tests or throws an unhandled exception.
>> >> >> >>
>> >> >> >>   1) Specifically, for the /runlist option, the following is
>> thrown:
>> >> >> >>
>> >> >> >>   /cygdrive/c/Program\ Files/NUnit\ 2.6.1/bin/nunit-console.exe
>> >> >> >> /runlist=./EmptyLibrary/Class1.cs ./NUnit/bin/Debug/NUnit.dll
>> >> >> >>   NUnit-Console version 2.6.1.12217
>> >> >> >>   Copyright (C) 2002-2012 Charlie Poole.
>> >> >> >>   Copyright (C) 2002-2004 James W. Newkirk, Michael C. Two,
>> Alexei A.
>> >> >> >> Vorontsov.
>> >> >> >>   Copyright (C) 2000-2002 Philip Craig.
>> >> >> >>   All Rights Reserved.
>> >> >> >>
>> >> >> >>   Runtime Environment -
>> >> >> >>      OS Version: Microsoft Windows NT 5.1.2600 Service Pack 3
>> >> >> >>     CLR Version: 2.0.50727.3634 ( Net 3.5 )
>> >> >> >>
>> >> >> >>   ProcessModel: Default    DomainUsage: Single
>> >> >> >>   Execution Runtime: net-3.5
>> >> >> >>   Run list: ./EmptyLibrary/Class1.cs
>> >> >> >>   Unhandled Exception:
>> >> >> >>   System.IndexOutOfRangeException: Index was outside the bounds of
>> >> the
>> >> >> >> array.
>> >> >> >>      at NUnit.ConsoleRunner.ConsoleUi.Execute(ConsoleOptions
>> options)
>> >> >> >>      at NUnit.ConsoleRunner.Runner.Main(String[] args)
>> >> >> >>
>> >> >> >>   2) If /runlist path is incorrect, than 0 tests are run.  This
>> is a
>> >> >> >>   silent error that should also be handled by notifying users
>> that no
>> >> >> >>   such path exists.
>> >> >> >>
>> >> >> >> To manage notifications about this bug go to:
>> >> >> >> https://bugs.launchpad.net/nunitv2/+bug/1057911/+subscriptions
>> >> >> >>
>> >> >> >
>> >> >> > --
>> >> >> > You received this bug notification because you are subscribed to
>> NUnit
>> >> >> > Extended Testing Platform.
>> >> >> > https://bugs.launchpad.net/bugs/1057911
>> >> >> >
>> >> >> > Title:
>> >> >> >   Trap incorrect input for nunit-console.exe options.
>> >> >> >
>> >> >> > To manage notifications about this bug go to:
>> >> >> > https://bugs.launchpad.net/nunitv2/+bug/1057911/+subscriptions
>> >> >>
>> >> >> --
>> >> >> You received this bug notification because you are a bug assignee.
>> >> >> https://bugs.launchpad.net/bugs/1057911
>> >> >>
>> >> >> Title:
>> >> >>   Trap incorrect input for nunit-console.exe options.
>> >> >>
>> >> >> Status in NUnit V2 Test Framework:
>> >> >>   Fix Committed
>> >> >>
>> >> >> Bug description:
>> >> >>   nunit-console.exe should handle incorrect options gracefully.  As
>> of
>> >> >>   now, it either runs 0 tests or throws an unhandled exception.
>> >> >>
>> >> >>   1) Specifically, for the /runlist option, the following is thrown:
>> >> >>
>> >> >>   /cygdrive/c/Program\ Files/NUnit\ 2.6.1/bin/nunit-console.exe
>> >> >> /runlist=./EmptyLibrary/Class1.cs ./NUnit/bin/Debug/NUnit.dll
>> >> >>   NUnit-Console version 2.6.1.12217
>> >> >>   Copyright (C) 2002-2012 Charlie Poole.
>> >> >>   Copyright (C) 2002-2004 James W. Newkirk, Michael C. Two, Alexei A.
>> >> >> Vorontsov.
>> >> >>   Copyright (C) 2000-2002 Philip Craig.
>> >> >>   All Rights Reserved.
>> >> >>
>> >> >>   Runtime Environment -
>> >> >>      OS Version: Microsoft Windows NT 5.1.2600 Service Pack 3
>> >> >>     CLR Version: 2.0.50727.3634 ( Net 3.5 )
>> >> >>
>> >> >>   ProcessModel: Default    DomainUsage: Single
>> >> >>   Execution Runtime: net-3.5
>> >> >>   Run list: ./EmptyLibrary/Class1.cs
>> >> >>   Unhandled Exception:
>> >> >>   System.IndexOutOfRangeException: Index was outside the bounds of
>> the
>> >> >> array.
>> >> >>      at NUnit.ConsoleRunner.ConsoleUi.Execute(ConsoleOptions options)
>> >> >>      at NUnit.ConsoleRunner.Runner.Main(String[] args)
>> >> >>
>> >> >>   2) If /runlist path is incorrect, than 0 tests are run.  This is a
>> >> >>   silent error that should also be handled by notifying users that no
>> >> >>   such path exists.
>> >> >>
>> >> >> To manage notifications about this bug go to:
>> >> >> https://bugs.launchpad.net/nunitv2/+bug/1057911/+subscriptions
>> >> >>
>> >> >
>> >> > --
>> >> > You received this bug notification because you are subscribed to NUnit
>> >> > Extended Testing Platform.
>> >> > https://bugs.launchpad.net/bugs/1057911
>> >> >
>> >> > Title:
>> >> >   Trap incorrect input for nunit-console.exe options.
>> >> >
>> >> > To manage notifications about this bug go to:
>> >> > https://bugs.launchpad.net/nunitv2/+bug/1057911/+subscriptions
>> >>
>> >> --
>> >> You received this bug notification because you are a bug assignee.
>> >> https://bugs.launchpad.net/bugs/1057911
>> >>
>> >> Title:
>> >>   Trap incorrect input for nunit-console.exe options.
>> >>
>> >> Status in NUnit V2 Test Framework:
>> >>   Fix Committed
>> >>
>> >> Bug description:
>> >>   nunit-console.exe should handle incorrect options gracefully.  As of
>> >>   now, it either runs 0 tests or throws an unhandled exception.
>> >>
>> >>   1) Specifically, for the /runlist option, the following is thrown:
>> >>
>> >>   /cygdrive/c/Program\ Files/NUnit\ 2.6.1/bin/nunit-console.exe
>> >> /runlist=./EmptyLibrary/Class1.cs ./NUnit/bin/Debug/NUnit.dll
>> >>   NUnit-Console version 2.6.1.12217
>> >>   Copyright (C) 2002-2012 Charlie Poole.
>> >>   Copyright (C) 2002-2004 James W. Newkirk, Michael C. Two, Alexei A.
>> >> Vorontsov.
>> >>   Copyright (C) 2000-2002 Philip Craig.
>> >>   All Rights Reserved.
>> >>
>> >>   Runtime Environment -
>> >>      OS Version: Microsoft Windows NT 5.1.2600 Service Pack 3
>> >>     CLR Version: 2.0.50727.3634 ( Net 3.5 )
>> >>
>> >>   ProcessModel: Default    DomainUsage: Single
>> >>   Execution Runtime: net-3.5
>> >>   Run list: ./EmptyLibrary/Class1.cs
>> >>   Unhandled Exception:
>> >>   System.IndexOutOfRangeException: Index was outside the bounds of the
>> >> array.
>> >>      at NUnit.ConsoleRunner.ConsoleUi.Execute(ConsoleOptions options)
>> >>      at NUnit.ConsoleRunner.Runner.Main(String[] args)
>> >>
>> >>   2) If /runlist path is incorrect, than 0 tests are run.  This is a
>> >>   silent error that should also be handled by notifying users that no
>> >>   such path exists.
>> >>
>> >> To manage notifications about this bug go to:
>> >> https://bugs.launchpad.net/nunitv2/+bug/1057911/+subscriptions
>> >>
>> >
>> > --
>> > You received this bug notification because you are subscribed to NUnit
>> > Extended Testing Platform.
>> > https://bugs.launchpad.net/bugs/1057911
>> >
>> > Title:
>> >   Trap incorrect input for nunit-console.exe options.
>> >
>> > To manage notifications about this bug go to:
>> > https://bugs.launchpad.net/nunitv2/+bug/1057911/+subscriptions
>>
>> --
>> You received this bug notification because you are a bug assignee.
>> https://bugs.launchpad.net/bugs/1057911
>>
>> Title:
>>   Trap incorrect input for nunit-console.exe options.
>>
>> Status in NUnit V2 Test Framework:
>>   Fix Committed
>>
>> Bug description:
>>   nunit-console.exe should handle incorrect options gracefully.  As of
>>   now, it either runs 0 tests or throws an unhandled exception.
>>
>>   1) Specifically, for the /runlist option, the following is thrown:
>>
>>   /cygdrive/c/Program\ Files/NUnit\ 2.6.1/bin/nunit-console.exe
>> /runlist=./EmptyLibrary/Class1.cs ./NUnit/bin/Debug/NUnit.dll
>>   NUnit-Console version 2.6.1.12217
>>   Copyright (C) 2002-2012 Charlie Poole.
>>   Copyright (C) 2002-2004 James W. Newkirk, Michael C. Two, Alexei A.
>> Vorontsov.
>>   Copyright (C) 2000-2002 Philip Craig.
>>   All Rights Reserved.
>>
>>   Runtime Environment -
>>      OS Version: Microsoft Windows NT 5.1.2600 Service Pack 3
>>     CLR Version: 2.0.50727.3634 ( Net 3.5 )
>>
>>   ProcessModel: Default    DomainUsage: Single
>>   Execution Runtime: net-3.5
>>   Run list: ./EmptyLibrary/Class1.cs
>>   Unhandled Exception:
>>   System.IndexOutOfRangeException: Index was outside the bounds of the
>> array.
>>      at NUnit.ConsoleRunner.ConsoleUi.Execute(ConsoleOptions options)
>>      at NUnit.ConsoleRunner.Runner.Main(String[] args)
>>
>>   2) If /runlist path is incorrect, than 0 tests are run.  This is a
>>   silent error that should also be handled by notifying users that no
>>   such path exists.
>>
>> To manage notifications about this bug go to:
>> https://bugs.launchpad.net/nunitv2/+bug/1057911/+subscriptions
>>
>
> --
> You received this bug notification because you are subscribed to NUnit
> Extended Testing Platform.
> https://bugs.launchpad.net/bugs/1057911
>
> Title:
>   Trap incorrect input for nunit-console.exe options.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/nunitv2/+bug/1057911/+subscriptions

-- 
You received this bug notification because you are a member of NUnit
Developers, which is subscribed to NUnit V2.
https://bugs.launchpad.net/bugs/1057911

Title:
  Trap incorrect input for nunit-console.exe options.

Status in NUnit V2 Test Framework:
  Fix Committed

Bug description:
  nunit-console.exe should handle incorrect options gracefully.  As of
  now, it either runs 0 tests or throws an unhandled exception.

  1) Specifically, for the /runlist option, the following is thrown:

  /cygdrive/c/Program\ Files/NUnit\ 2.6.1/bin/nunit-console.exe /runlist=./EmptyLibrary/Class1.cs ./NUnit/bin/Debug/NUnit.dll
  NUnit-Console version 2.6.1.12217
  Copyright (C) 2002-2012 Charlie Poole.
  Copyright (C) 2002-2004 James W. Newkirk, Michael C. Two, Alexei A. Vorontsov.
  Copyright (C) 2000-2002 Philip Craig.
  All Rights Reserved.

  Runtime Environment -
     OS Version: Microsoft Windows NT 5.1.2600 Service Pack 3
    CLR Version: 2.0.50727.3634 ( Net 3.5 )

  ProcessModel: Default    DomainUsage: Single
  Execution Runtime: net-3.5
  Run list: ./EmptyLibrary/Class1.cs
  Unhandled Exception:
  System.IndexOutOfRangeException: Index was outside the bounds of the array.
     at NUnit.ConsoleRunner.ConsoleUi.Execute(ConsoleOptions options)
     at NUnit.ConsoleRunner.Runner.Main(String[] args)

  2) If /runlist path is incorrect, than 0 tests are run.  This is a
  silent error that should also be handled by notifying users that no
  such path exists.

To manage notifications about this bug go to:
https://bugs.launchpad.net/nunitv2/+bug/1057911/+subscriptions


References