Re: [pgAdmin4][Patch]: Partition table fix

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][Patch]: Partition table fix
Date: 2018-07-20 16:01:42
Message-ID: CA+OCxoy6CsVgNq1hzGKfN-z6igWMdF_fnhmT612cqduc4e2N0g@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi

On Fri, Jul 20, 2018 at 10:42 AM, Khushboo Vashi <
khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:

> Hi,
>
> Please find the attached patch to fix the issue : The right panels
> (properties, SQL etc...) give error while clicking on the partitions of a
> table.
>

This fails tests for me:

HeadlessChrome 0.0.0 (Mac OS X 10.12.6): Executed 304 of 741 SUCCESS (0
secs / 4.711 secs)

HeadlessChrome 0.0.0 (Mac OS X 10.12.6) tree#node#getTreeNodeHierarchy
getTreeNodeHierarchy is called with aciTreeNode object When tree has table
when table has partition returns table with partition parameters FAILED

Expected object not to have properties

partition: Object({ some key: 'expected value', _type: 'partition',
some other key: 'some other value', priority: 0 })

at UserContext.<anonymous>
(regression/javascript/tree/webpack:/regression/javascript/tree/pgadmin_tree_node_spec.js:177:26)

HeadlessChrome 0.0.0 (Mac OS X 10.12.6): Executed 680 of 741 (1 FAILED) (0
secs / 6.981 secs)

HeadlessChrome 0.0.0 (Mac OS X 10.12.6) tree#node#getTreeNodeHierarchy
getTreeNodeHierarchy is called with aciTreeNode object When tree has table
when table has partition returns table with partition parameters FAILED

Expected object not to have properties

partition: Object({ some key: 'expected value', _type: 'partition',
some other key: 'some other value', priority: 0 })

HeadlessChrome 0.0.0 (Mac OS X 10.12.6) tree#node#getTreeNodeHierarchy
getTreeNodeHierarchy is called with TreeNode object When tree has table
when table has partition returns table with partition parameters FAILED

Expected object not to have properties

partition: Object({ some key: 'expected value', _type: 'partition',
some other key: 'some other value', priority: 0 })

at UserContext.<anonymous>
(regression/javascript/tree/webpack:/regression/javascript/tree/pgadmin_tree_node_spec.js:336:26)

HeadlessChrome 0.0.0 (Mac OS X 10.12.6): Executed 686 of 741 (2 FAILED) (0
secs / 6.999 secs)

HeadlessChrome 0.0.0 (Mac OS X 10.12.6) tree#node#getTreeNodeHierarchy
getTreeNodeHierarchy is called with TreeNode object When tree has table
when table has partition returns table with partition parameters FAILED

Expected object not to have properties

partition: Object({ some key: 'expected value', _type: 'partition',
some other key: 'some other value', priority: 0 })

HeadlessChrome 0.0.0 (Mac OS X 10.12.6): Executed 741 of 741 (2 FAILED)
(7.47 secs / 7.149 secs)

error Command failed with exit code 1.

info Visit *https://yarnpkg.com/en/docs/cli/run
<https://yarnpkg.com/en/docs/cli/run>* for documentation about this command.

make: *** [check] Error 1

--
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 Khushboo Vashi 2018-07-23 05:03:57 Re: [pgAdmin4][Patch]: Partition table fix
Previous Message Dave Page 2018-07-20 15:56:07 pgAdmin 4 commit: Remove comment that's not really needed any more.