From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Atira Odhner <aodhner(at)pivotal(dot)io> |
Cc: | George Gelashvili <ggelashvili(at)pivotal(dot)io>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Feature test issues |
Date: | 2017-02-28 16:59:32 |
Message-ID: | CA+OCxoxvZXohFRncfrK6ia6K4_6s-YGxwicePk3RmmmCaSf73Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Tue, Feb 28, 2017 at 4:50 PM, Atira Odhner <aodhner(at)pivotal(dot)io> wrote:
>> On my top-of-the-range-last-year Mac, startup takes about 14 seconds
>> per test - that's from the start of the test, until the point at which
>> the "Loading pgAdmin..." spinner vanishes.
>
>
> That includes the time it takes to load the page, not just app startup time.
> The spinner doesn't appear until the app has started. We're working today on
> eliminating the multiple startups, but it won't have an effect on page load
> time. I think the only way to fix that test run time would be to actually
> improve the page load time. That said, we're seeing tests run much faster
> than you are-- 43 seconds for the entire suite of which 30 seconds was the
> two feature tests.
Right - when I talk about having a single startup, I'm not just
talking about the app server, but the browser and initial page load as
well. It may pay to have a function we can call that would reset the
treeview and the tabset, thus allowing us to reset the view cheaply
between groups of tests (assuming we might have a number of tests that
could all run in the same query tool instance for example).
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2017-02-28 17:02:06 | pgAdmin 4 commit: Further fixes and cleanups to the Windows build scrip |
Previous Message | Atira Odhner | 2017-02-28 16:56:23 | [patch] Use TEST_SQLITE_PATH for tests |