Re: [pgadmin-hackers] Re: Server side cursor limitations for ondemand loading of data in query tool [RM2137] [pgAdmin4]

Hi,

On Sat, May 13, 2017 at 12:35 AM, Joao Pedro De Almeida Pereira <jdealmeidapereira@pivotal.io> wrote:

We were only able to apply the patch on 1f903ba2 (were seeing patch does not apply due to sqleditor.js conflicts)

The javascript tests passed, but we were unable to copy rows or columns or cells when running the application. Could you run feature tests?

There are three modes sqleditor can be launched

1. Query tool  (Tools menus -> Query Tool)

2. Datagrid.  (Right click on any table/view  -> View Data -> View All/First 100/Last 100/Filtered rows)

3. Scripts (Right click on any table/view -> INSERT/CREATE/UPDATE/DELETE/SELECT)

Paste functionality is only enabled in Datagrid and table has Primary key otherwise it's disabled. In your case row might have been copied but you were unable paste because you might be trying to paste the rows in Query tool. Please try again in Datagrid with table having Primary key.

 

Now that more functionality is being added to sqleditor.js, this may be a good time to extract the functionality to separate files. This will increase readability, and encourage separation of concerns. It will also make changes easier to test in isolation.

Ok. Let me check if I can separate out ant functionalities.

It's probably a good idea to test the changes made to the python as well as javascript code. In this case, the new behavior of poll() in sqleditor __init__ should be tested.

At this point we don't have any python unit tests that can test sqleditor backend (python code).

@Dave should I include python unit test cases in this patch?