Re: Feature test issues

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

In response to

Responses

Browse pgadmin-hackers by date

  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