Re: [pgAdmin4] Add API tests for Download CSV route in query tool - Mailing list pgadmin-hackers

From Murtuza Zabuawala
Subject Re: [pgAdmin4] Add API tests for Download CSV route in query tool
Date
Msg-id CAKKotZRaABY70hNrR987SbJhvvtr3LWPTFuSvvnkgiT-nYDZ6A@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4] Add API tests for Download CSV route in query tool  (Dave Page <dpage@pgadmin.org>)
Responses Re: [pgAdmin4] Add API tests for Download CSV route in query tool  (Dave Page <dpage@pgadmin.org>)
List pgadmin-hackers
Hi Dave,

PFA updated patch with suggested changes.

--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



On Thu, Apr 11, 2019 at 6:27 PM Dave Page <dpage@pgadmin.org> wrote:
Hi

On Thu, Apr 11, 2019 at 12:21 PM Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi,

PFA patch to add API tests for Download CSV route in query tool.
> python runtests.py --pkg tools.sqleditor.tests --exclude feature_tests

Nice addition, but can you modify it so it doesn't spit the "table doesn't exist" error out to the terminal? We're trying to clean up the test output at the moment.

Oh, and it should be this_table_does_not_exist (without the trailing s :-) )

runTest (pgadmin.tools.sqleditor.tests.test_download_csv_query_tool.TestDownloadCSV)
Download csv URL with wrong query ... 2019-04-11 13:54:38,710: ERROR flask.app: failed to execute query ((with server cursor) for the server #4 - CONN:3783391 (query-id: 5309318):
error message:ERROR:  relation "this_table_does_not_exists" does not exist
LINE 1: SELECT * FROM this_table_does_not_exists
                      ^

ok
 
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Attachment

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: [pgAdmin4] Add API tests for Download CSV route in query tool
Next
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Add tests for the CSV download.