On Thu, Jul 22, 2010 at 11:53:22AM +0200, Romulo Goncalves wrote:
> Changeset: 7a88e64c34b1 for MonetDB
> URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=7a88e64c34b1
> Modified Files:
>       sql/src/test/BugTracker-2010/Tests/connectto.Bug-2548.stable.err
>       sql/src/test/BugTracker-2010/Tests/connectto.Bug-2548.stable.out
> Branch: Jun2010
> Log Message:
> 
> It seems the err output for the test was not complete.
> In the approval of the test two sql statements were missing, therefore, the 
> err output was created without the errors related to that two statements.

Thanks for taking care of this.
My reason for not approving the error messages in the first place was that I
was not sure about the intention of the test, and hence, whether and which
error messages are to be expected.
I now understand from your approval that the error messages indeed match the
intention of the test.

I wonder, though, whether the error message is indeed what it should be.
Saying "failed: Success" (see below) seems at least "odd" to me ...

Stefan

> 
> diffs (32 lines):
> 
> diff -r 6908fb1d95e4 -r 7a88e64c34b1 
> sql/src/test/BugTracker-2010/Tests/connectto.Bug-2548.stable.err
> --- a/sql/src/test/BugTracker-2010/Tests/connectto.Bug-2548.stable.err        
> Thu Jul 22 11:02:07 2010 +0200
> +++ b/sql/src/test/BugTracker-2010/Tests/connectto.Bug-2548.stable.err        
> Thu Jul 22 11:53:14 2010 +0200
> @@ -71,9 +71,15 @@
>  # 13:12:55 >  mclient -lsql -ftest -i -e --host=rig --port=39884 
>  # 13:12:55 >  
>  
> -MAPI  = mone...@rig:39884
> +MAPI  = mone...@alviss:36310
>  QUERY = connect to default;
>  ERROR = !CONNECT TO: DEFAULT is not supported!
> +MAPI  = mone...@alviss:36310
> +QUERY = connect to 'whatever' port 50001 database 'nonexisting' USER 
> 'monetdb' PASSWORD 'monetdb' LANGUAGE 'mal';
> +ERROR = !IOException:mapi.connect:Could not connect: gethostbyname failed: 
> Success
                                                                      ^^^^^^  
^^^^^^^
                                                                      !!!!!!  
???????
> +MAPI  = mone...@alviss:36310
> +QUERY = disconnect 'whatever';
> +ERROR = !DISCONNECT CATALOG: no such db_alias 'whatever'
>  
>  # 13:12:55 >  
>  # 13:12:55 >  Done.
> diff -r 6908fb1d95e4 -r 7a88e64c34b1 
> sql/src/test/BugTracker-2010/Tests/connectto.Bug-2548.stable.out
> --- a/sql/src/test/BugTracker-2010/Tests/connectto.Bug-2548.stable.out        
> Thu Jul 22 11:02:07 2010 +0200
> +++ b/sql/src/test/BugTracker-2010/Tests/connectto.Bug-2548.stable.out        
> Thu Jul 22 11:53:14 2010 +0200
> @@ -23,8 +23,6 @@
>  # 13:12:55 >  mclient -lsql -ftest -i -e --host=rig --port=39884 
>  # 13:12:55 >  
>  
> -! to be provided / approved !
> -
>  # 13:12:55 >  
>  # 13:12:55 >  Done.
>  # 13:12:55 >  
> _______________________________________________
> Checkin-list mailing list
> checkin-l...@monetdb.org
> http://mail.monetdb.org/mailman/listinfo/checkin-list
> 

-- 
| Dr. Stefan Manegold | mailto:stefan.maneg...@cwi.nl |
| CWI,  P.O.Box 94079 | http://www.cwi.nl/~manegold/  |
| 1090 GB Amsterdam   | Tel.: +31 (20) 592-4212       |
| The Netherlands     | Fax : +31 (20) 592-4199       |

------------------------------------------------------------------------------
This SF.net email is sponsored by Sprint
What will you do first with EVO, the first 4G phone?
Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first
_______________________________________________
Monetdb-developers mailing list
Monetdb-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/monetdb-developers

Reply via email to