Re: [pgAdmin4] Add API tests for Download CSV route in query tool

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4] Add API tests for Download CSV route in query tool
Date: 2019-04-11 12:57:00
Message-ID: CA+OCxowmGofs7pAOpX0J+MEeDd-BTKefQXDVpoL31HOyjHp9dQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi

On Thu, Apr 11, 2019 at 12:21 PM Murtuza Zabuawala <
murtuza(dot)zabuawala(at)enterprisedb(dot)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

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Murtuza Zabuawala 2019-04-11 14:29:59 Re: [pgAdmin4] Add API tests for Download CSV route in query tool
Previous Message Dave Page 2019-04-11 12:51:53 Re: little messages update