From: | Joao Pedro De Almeida Pereira <jdealmeidapereira(at)pivotal(dot)io> |
---|---|
To: | Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, Matthew Kleiman <mkleiman(at)pivotal(dot)io> |
Subject: | Re: [pgAdmin4][PATCH] To fix the issue of menu visibility when node is hidden |
Date: | 2017-05-11 14:49:38 |
Message-ID: | CAE+jjam=XPddpy27YzZLg5ehT-5A3arckcbYGiRnrRJ6fmAxnA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Murtuza,
Since you are adding a new function to the browser.js component, we would
encourage you to extract the function out of the templated javascript file
so that it can be tested. A good example of this would be the
pgadmin/static/js/size_prettify.js.
For this specific function we found one function that is used from
browser.js, and you can Stub that out using
createSpyObject(browser, 'get_preference');
We think the object here is not 100% line coverage but 100% of the behavior
should be covered.
Also, suggest you change the variable name perf to preference. Looks like
it is a typo.
Regards,
João & Matt
On Thu, May 11, 2017 at 3:08 AM, Murtuza Zabuawala <
murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
> Hi,
>
> PFA patch to fix the issue where user hides the any node from Preference
> dialog but the menu(both context/object) still appears in pgAdmin4.
> RM#2225
>
> --
> Regards,
> Murtuza Zabuawala
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
>
> --
> Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>
>
From | Date | Subject | |
---|---|---|---|
Next Message | Surinder Kumar | 2017-05-12 05:39:52 | Re: [pgAdmin4][Patch][RM2257]: Query tool - Insert row doesn't use default values |
Previous Message | Murtuza Zabuawala | 2017-05-11 14:21:34 | Re: [pgAdmin4][PATCH] To fix the issues with CSV file download |