Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtp (Exim 4.84_2) (envelope-from ) id 1cbpbC-0005aq-DI for pgadmin-hackers@arkaria.postgresql.org; Thu, 09 Feb 2017 14:21:26 +0000 Received: from localhost ([127.0.0.1] helo=postgresql.org) by malur.postgresql.org with smtp (Exim 4.84_2) (envelope-from ) id 1cbpbC-0000st-0C for pgadmin-hackers@arkaria.postgresql.org; Thu, 09 Feb 2017 14:21:26 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.84_2) (envelope-from ) id 1cbpaw-0000dE-KK for pgadmin-hackers@postgresql.org; Thu, 09 Feb 2017 14:21:10 +0000 Received: from mail-qt0-x22b.google.com ([2607:f8b0:400d:c0d::22b]) by magus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.84_2) (envelope-from ) id 1cbpaq-0000ZH-Me for pgadmin-hackers@postgresql.org; Thu, 09 Feb 2017 14:21:09 +0000 Received: by mail-qt0-x22b.google.com with SMTP id w20so4902664qtb.1 for ; Thu, 09 Feb 2017 06:21:03 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=pivotal-io.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=WTRiPAJXgBD4JhH2n+IoXZWZ65gMonjTdpLTUphyNsY=; b=ZdbPG5Rmfhb/3ASY80l0nW45QUbWG7vWNydHzcapE2zeUi/BewGDGECSX6xIuu4oVr QZ+NPjaSEKnTbovWNmhRGGQtBFNpAu53GDrIKrG75W5wg7BK61lcybT61lnfqJuGLG9x beLxSebjc//KNUTlrCGEZYigOhnk/Ddej1ejNQPFv5Cpywd7MIxEoZeITCGJBa/oTZLe 8nciUIowlVQEV0IFtBoPH4yn7ZnyoDphamlRk97bumvSupTDBKB9ZAJaAbYEriVRDdqw HhTU5aFSh12RF2Mc++fV+Z6Ipp72w2fLDnR7TXIiDbIiSTBo3wF3/4dNqHSpFiSNmayz NSAw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=WTRiPAJXgBD4JhH2n+IoXZWZ65gMonjTdpLTUphyNsY=; b=hTqfsakvQP1d93Rqbz+ZINr9PF9MUrgzyaDTl4otPlqrcXjs273rFjn7r8V2DIxvGG u6dy7vbdY0PMlzGWjvn7g+RR6gC3Y2c8oKpdYs4+TvJ7rXjft404Yjuw8GehEMx51gYn Uh/2fo/P1h5Tvbyq9SmMaPOsOHGK4gnqV1JURKa8g+MkKyDgR6sv+RxsXW/qRmKdC2ba +94VXeNLzgTX1t80ue2IfBOhOgBDJTytL2tX/y+Tji8F8IU6EHcOrYlXxSKcxC8huKOu hGe9zejcH9v5j+alEMXX9hhV9egoi7oRxFFOwtj11wNhVcFEU/E8ftFD3dck3afyDqz/ F8EQ== X-Gm-Message-State: AMke39n30cRyjqgNp6wymo8WLcG3ddPVwRoRZWfSbqY/O5Rf3cZ5cxJ0toBOSggNpYpPsSSMevDiREnMN3YSgdBc X-Received: by 10.200.42.66 with SMTP id l2mr2974741qtl.33.1486650061916; Thu, 09 Feb 2017 06:21:01 -0800 (PST) MIME-Version: 1.0 References: In-Reply-To: From: Atira Odhner Date: Thu, 09 Feb 2017 14:20:51 +0000 Message-ID: Subject: Re: Acceptance Tests against a browser (WIP) To: Dave Page Cc: George Gelashvili , pgadmin-hackers Content-Type: multipart/alternative; boundary=001a1145ed7ed8a910054819b0f1 X-Pg-Spam-Score: -2.6 (--) List-Archive: List-Help: List-ID: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: X-Mailing-List: pgadmin-hackers Precedence: bulk Sender: pgadmin-hackers-owner@postgresql.org --001a1145ed7ed8a910054819b0f1 Content-Type: text/plain; charset=UTF-8 Certainly. We did mention the dependency in the email. Would it be better to mention it in the patch name? Is there a better way for us to manage these changes? On other open source projects, I've seen github mirrors set up so that changes can be pulled in like branches rather then as patch applies. That would have avoided this situation since the parent commit would be pulled in with the same SHA from either pull request branch and git would not see it as a conflict. I'm rather new to dealing with patch files like this so I would love some tips. Thanks, Tira On Thu, Feb 9, 2017, 8:27 AM Dave Page wrote: > OK, well that one was sent back with feedback as well, so please > resubmit when the relevant updates have been made to either and > they've been retested. Given the amount of work you're doing at the > moment, it would be helpful if you could note when one patch is > dependent on another. It's hard to keep track when you're this > productive! > > Thanks. > > On Thu, Feb 9, 2017 at 1:15 PM, Atira Odhner wrote: > > create_table is the change we pulled into the other patch which would > need > > to be applied first. > > > > > > On Thu, Feb 9, 2017, 7:47 AM Dave Page wrote: > >> > >> Hi > >> > >> I get the following crash when running with Python 3.4 or 3.5: > >> > >> (pgadmin4-py34) piranha:pgadmin4 dpage$ python > web/regression/runtests.py > >> pgAdmin 4 - Application Initialisation > >> ====================================== > >> > >> > >> The configuration database - '/Users/dpage/.pgadmin/test_pgadmin4.db' > >> does not exist. > >> Entering initial setup mode... > >> NOTE: Configuring authentication for DESKTOP mode. > >> > >> The configuration database has been created at > >> /Users/dpage/.pgadmin/test_pgadmin4.db > >> > >> =============Running the test cases for 'Regression - PG > 9.4'============= > >> runTest > >> > (pgadmin.acceptance.tests.connect_to_server_feature_test.ConnectsToServerFeatureTest) > >> ... Traceback (most recent call last): > >> File "web/regression/runtests.py", line 276, in > >> verbosity=2).run(suite) > >> File > >> > "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/unittest/runner.py", > >> line 168, in run > >> test(result) > >> File > >> > "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/unittest/suite.py", > >> line 84, in __call__ > >> return self.run(*args, **kwds) > >> File > >> > "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/unittest/suite.py", > >> line 122, in run > >> test(result) > >> File > >> > "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/unittest/case.py", > >> line 628, in __call__ > >> return self.run(*args, **kwds) > >> File > >> > "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/unittest/case.py", > >> line 588, in run > >> self._feedErrorsToResult(result, outcome.errors) > >> File > >> > "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/unittest/case.py", > >> line 515, in _feedErrorsToResult > >> if issubclass(exc_info[0], self.failureException): > >> TypeError: issubclass() arg 2 must be a class or tuple of classes > >> > >> With Python 2.7, it initially opens Chrome with the URL "data:," > >> (without the quotes), and then spits out: > >> > >> ====================================================================== > >> ERROR: runTest > >> > (pgadmin.acceptance.tests.connect_to_server_feature_test.ConnectsToServerFeatureTest) > >> ---------------------------------------------------------------------- > >> Traceback (most recent call last): > >> File > >> > "/Users/dpage/git/pgadmin4/web/pgadmin/acceptance/tests/connect_to_server_feature_test.py", > >> line 41, in setUp > >> test_utils.create_table(self.server, "acceptance_test_db", > >> "test_table") > >> AttributeError: 'module' object has no attribute 'create_table' > >> > >> ====================================================================== > >> ERROR: runTest > >> > (pgadmin.acceptance.tests.template_selection_feature_test.TemplateSelectionFeatureTest) > >> ---------------------------------------------------------------------- > >> Traceback (most recent call last): > >> File > >> > "/Users/dpage/git/pgadmin4/web/pgadmin/acceptance/tests/template_selection_feature_test.py", > >> line 36, in runTest > >> test_utils.create_table(self.server, "acceptance_test_db", > >> "test_table") > >> AttributeError: 'module' object has no attribute 'create_table' > >> > >> ====================================================================== > >> ERROR: runTest > >> > (pgadmin.acceptance.tests.template_selection_feature_test.TemplateSelectionFeatureTest) > >> ---------------------------------------------------------------------- > >> Traceback (most recent call last): > >> File > >> > "/Users/dpage/git/pgadmin4/web/pgadmin/acceptance/tests/template_selection_feature_test.py", > >> line 66, in tearDown > >> self.page.find_by_xpath("//button[contains(.,'Cancel')]").click() > >> File "/Users/dpage/git/pgadmin4/web/regression/utils/pgadmin_page.py", > >> line 46, in find_by_xpath > >> return self.wait_for_element(lambda: > >> self.driver.find_element_by_xpath(xpath)) > >> File "/Users/dpage/git/pgadmin4/web/regression/utils/pgadmin_page.py", > >> line 86, in wait_for_element > >> return self._wait_for("element to exist", element_if_it_exists) > >> File "/Users/dpage/git/pgadmin4/web/regression/utils/pgadmin_page.py", > >> line 120, in _wait_for > >> raise RuntimeError("timed out waiting for " + waiting_for_message) > >> RuntimeError: timed out waiting for element to exist > >> > >> ---------------------------------------------------------------------- > >> Ran 149 tests in 59.258s > >> > >> FAILED (errors=3, skipped=12) > >> > >> > >> On Wed, Feb 8, 2017 at 10:15 PM, Atira Odhner > wrote: > >> > Hey Dave, > >> > > >> > We re-used one of the test helpers for the > >> > 'fix-greenplum-show-tables.diff' > >> > patch, so here is an updated patch which does not include adding that > >> > test > >> > helper in case you apply the show-tables patch first. Also, we saw > some > >> > strange test behavior yesterday where form fields weren't being filled > >> > in > >> > correctly so we changed the way that input fields get filled to be > more > >> > reliable. > >> > > >> > In short these need to be applied in this order: > >> >> > >> >> git apply fix-greenplum-show-tables.diff > >> >> > >> >> git apply > >> >> acceptance-tests-minus-create-table-helper-with-fixed-inputs.diff > >> > > >> > > >> > We also moved the --exclude flag changes out to a separate patch. > >> > > >> > On our side we are still dealing with these as 20 separate commits. > What > >> > is > >> > the best way for us to send you these patches? Do you prefer having > them > >> > all > >> > squashed down to a single patch or to have smaller patches? > >> > > >> > > >> > > >> > On Mon, Feb 6, 2017 at 9:54 AM, Atira Odhner > wrote: > >> >> > >> >> I agree that we should rename the test. We've renamed it to > >> >> "template_selection_feature_test". > >> >> Your other suggestions are captured in our backlog as future > >> >> improvements. > >> >> We definitely can and should do those things but I think it would be > >> >> valuable to go ahead and get this suite in and give other devs a > chance > >> >> to > >> >> use and iterate on this work. > >> >> > >> >> Thanks, > >> >> > >> >> Tira & George > >> >> > >> >> On Mon, Feb 6, 2017 at 5:32 AM, Dave Page wrote: > >> >>> > >> >>> Hi > >> >>> > >> >>> On Fri, Feb 3, 2017 at 9:56 PM, Atira Odhner > >> >>> wrote: > >> >>> > Hi Dave, > >> >>> > > >> >>> > Here is a new patch which includes the following: > >> >>> > - randomized ports > >> >>> > - delete the acceptance_test_db database in setup in case a prior > >> >>> > run > >> >>> > failed > >> >>> > - fixed size browser window > >> >>> > >> >>> Definitely getting there :-). A couple of thoughts/questions: > >> >>> > >> >>> - Now there are 2 tests in there, it's clear that both the Python > >> >>> server and browser session are restarted for each test. Can this be > >> >>> avoided? It'll really slow down test execution as more and more are > >> >>> added. > >> >>> > >> >>> - We've got a new monster name: > >> >>> > >> >>> > >> >>> > pgadmin.acceptance.tests.sql_template_selection_by_postgres_version_works_feature_test.SQLTemplateSelectionByPostgresVersionWorksFeatureTest > >> >>> (which on disk is > >> >>> sql_template_select_by_postgres_version_works_feature_test.py). > Names > >> >>> like that really must be shortened to something more sane and > >> >>> manageable. > >> >>> > >> >>> - I'm a little confused by why the tests cannot be run in server > mode. > >> >>> The error says it's because the username/password is unknown - > >> >>> however, both the pgAdmin and database server usernames and > passwords > >> >>> are in test_config.json. > >> >>> > >> >>> Thanks! > >> >>> > >> >>> -- > >> >>> Dave Page > >> >>> Blog: http://pgsnake.blogspot.com > >> >>> Twitter: @pgsnake > >> >>> > >> >>> EnterpriseDB UK: http://www.enterprisedb.com > >> >>> The Enterprise PostgreSQL Company > >> >> > >> >> > >> > > >> > >> > >> > >> -- > >> Dave Page > >> Blog: http://pgsnake.blogspot.com > >> Twitter: @pgsnake > >> > >> EnterpriseDB UK: http://www.enterprisedb.com > >> The Enterprise PostgreSQL Company > > > > -- > Dave Page > Blog: http://pgsnake.blogspot.com > Twitter: @pgsnake > > EnterpriseDB UK: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > --001a1145ed7ed8a910054819b0f1 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable

Certainly.=C2=A0 We did mention the dependency in the email.= Would it be better to mention it in the patch name? Is there a better way = for us to manage these changes? On other open source projects, I've see= n github mirrors set up so that changes can be pulled in like branches rath= er then as patch applies. That would have avoided this situation since the = parent commit would be pulled in with the same SHA from either pull request= branch and git would not see it as a conflict.

I'm rather new to dealing with patch files like this so = I would love some tips.

Thanks,
Tira


On Thu, Feb 9, 2017, 8:27 A= M Dave Page <dpage@pgadmin.org&= gt; wrote:
OK, well that one was se= nt back with feedback as well, so please
resubmit when the relevant updates have been made to either and
they've been retested. Given the amount of work you're doing at the=
moment, it would be helpful if you could note when one patch is
dependent on another. It's hard to keep track when you're this
productive!

Thanks.

On Thu, Feb 9, 2017 at 1:15 PM, Atira Odhner <aodhner@pivotal.io>= ; wrote:
> create_table is the change we pulled into the other patch which would = need
> to be applied first.
>
>
> On Thu, Feb 9, 2017, 7:47 AM Dave Page <dpage@pgadmin.org> w= rote:
>>
>> Hi
>>
>> I get the following crash when running with Python 3.4 or 3.5:
>>
>> (pgadmin4-py34) piranha:pgadmin4 dpage$ python web/regression/runt= ests.py
>> pgAdmin 4 - Application Initialisation
>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>>
>>
>> The configuration database - '/Users/dpage/.pgadmin/test_pgadm= in4.db'
>> does not exist.
>> Entering initial setup mode...
>> NOTE: Configuring authentication for DESKTOP mode.
>>
>> The configuration database has been created at
>> /Users/dpage/.pgadmin/test_pgadmin4.db
>>
>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3DRunning the test cases for = 'Regression - PG 9.4'=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>> runTest
>> (pgadmin.acceptance.tests.connect_to_server_feature_test.ConnectsT= oServerFeatureTest)
>> ... Traceback (most recent call last):
>>=C2=A0 =C2=A0File "web/regression/runtests.py", line 276,= in <module>
>>=C2=A0 =C2=A0 =C2=A0verbosity=3D2).run(suite)
>>=C2=A0 =C2=A0File
>> "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/= lib/python3.4/unittest/runner.py",
>> line 168, in run
>>=C2=A0 =C2=A0 =C2=A0test(result)
>>=C2=A0 =C2=A0File
>> "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/= lib/python3.4/unittest/suite.py",
>> line 84, in __call__
>>=C2=A0 =C2=A0 =C2=A0return self.run(*args, **kwds)
>>=C2=A0 =C2=A0File
>> "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/= lib/python3.4/unittest/suite.py",
>> line 122, in run
>>=C2=A0 =C2=A0 =C2=A0test(result)
>>=C2=A0 =C2=A0File
>> "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/= lib/python3.4/unittest/case.py",
>> line 628, in __call__
>>=C2=A0 =C2=A0 =C2=A0return self.run(*args, **kwds)
>>=C2=A0 =C2=A0File
>> "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/= lib/python3.4/unittest/case.py",
>> line 588, in run
>>=C2=A0 =C2=A0 =C2=A0self._feedErrorsToResult(result, outcome.errors= )
>>=C2=A0 =C2=A0File
>> "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/= lib/python3.4/unittest/case.py",
>> line 515, in _feedErrorsToResult
>>=C2=A0 =C2=A0 =C2=A0if issubclass(exc_info[0], self.failureExceptio= n):
>> TypeError: issubclass() arg 2 must be a class or tuple of classes<= br class=3D"gmail_msg"> >>
>> With Python 2.7, it initially opens Chrome with the URL "data= :,"
>> (without the quotes), and then spits out:
>>
>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>> ERROR: runTest
>> (pgadmin.acceptance.tests.connect_to_server_feature_test.ConnectsT= oServerFeatureTest)
>> ------------------------------------------------------------------= ----
>> Traceback (most recent call last):
>>=C2=A0 =C2=A0File
>> "/Users/dpage/git/pgadmin4/web/pgadmin/acceptance/tests/conne= ct_to_server_feature_test.py",
>> line 41, in setUp
>>=C2=A0 =C2=A0 =C2=A0test_utils.create_table(self.server, "acce= ptance_test_db",
>> "test_table")
>> AttributeError: 'module' object has no attribute 'crea= te_table'
>>
>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>> ERROR: runTest
>> (pgadmin.acceptance.tests.template_selection_feature_test.Template= SelectionFeatureTest)
>> ------------------------------------------------------------------= ----
>> Traceback (most recent call last):
>>=C2=A0 =C2=A0File
>> "/Users/dpage/git/pgadmin4/web/pgadmin/acceptance/tests/templ= ate_selection_feature_test.py",
>> line 36, in runTest
>>=C2=A0 =C2=A0 =C2=A0test_utils.create_table(self.server, "acce= ptance_test_db",
>> "test_table")
>> AttributeError: 'module' object has no attribute 'crea= te_table'
>>
>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>> ERROR: runTest
>> (pgadmin.acceptance.tests.template_selection_feature_test.Template= SelectionFeatureTest)
>> ------------------------------------------------------------------= ----
>> Traceback (most recent call last):
>>=C2=A0 =C2=A0File
>> "/Users/dpage/git/pgadmin4/web/pgadmin/acceptance/tests/templ= ate_selection_feature_test.py",
>> line 66, in tearDown
>>=C2=A0 =C2=A0 =C2=A0self.page.find_by_xpath("//button[contains= (.,'Cancel')]").click()
>>=C2=A0 =C2=A0File "/Users/dpage/git/pgadmin4/web/regression/ut= ils/pgadmin_page.py",
>> line 46, in find_by_xpath
>>=C2=A0 =C2=A0 =C2=A0return self.wait_for_element(lambda:
>> self.driver.find_element_by_xpath(xpath))
>>=C2=A0 =C2=A0File "/Users/dpage/git/pgadmin4/web/regression/ut= ils/pgadmin_page.py",
>> line 86, in wait_for_element
>>=C2=A0 =C2=A0 =C2=A0return self._wait_for("element to exist&qu= ot;, element_if_it_exists)
>>=C2=A0 =C2=A0File "/Users/dpage/git/pgadmin4/web/regression/ut= ils/pgadmin_page.py",
>> line 120, in _wait_for
>>=C2=A0 =C2=A0 =C2=A0raise RuntimeError("timed out waiting for = " + waiting_for_message)
>> RuntimeError: timed out waiting for element to exist
>>
>> ------------------------------------------------------------------= ----
>> Ran 149 tests in 59.258s
>>
>> FAILED (errors=3D3, skipped=3D12)
>>
>>
>> On Wed, Feb 8, 2017 at 10:15 PM, Atira Odhner <aodhner@pivotal= .io> wrote:
>> > Hey Dave,
>> >
>> > We re-used one of the test helpers for the
>> > 'fix-greenplum-show-tables.diff'
>> > patch, so here is an updated patch which does not include add= ing that
>> > test
>> > helper in case you apply the show-tables patch first. Also, w= e saw some
>> > strange test behavior yesterday where form fields weren't= being filled
>> > in
>> > correctly so we changed the way that input fields get filled = to be more
>> > reliable.
>> >
>> > In short these need to be applied in this order:
>> >>
>> >> git apply fix-greenplum-show-tables.diff
>> >>
>> >> git apply
>> >> acceptance-tests-minus-create-table-helper-with-fixed-inp= uts.diff
>> >
>> >
>> > We also moved the --exclude flag changes out to a separate pa= tch.
>> >
>> > On our side we are still dealing with these as 20 separate co= mmits. What
>> > is
>> > the best way for us to send you these patches? Do you prefer = having them
>> > all
>> > squashed down to a single patch or to have smaller patches? >> >
>> >
>> >
>> > On Mon, Feb 6, 2017 at 9:54 AM, Atira Odhner <aodhner@piv= otal.io> wrote:
>> >>
>> >> I agree that we should rename the test. We've renamed= it to
>> >> "template_selection_feature_test".
>> >> Your other suggestions are captured in our backlog as fut= ure
>> >> improvements.
>> >> We definitely can and should do those things but I think = it would be
>> >> valuable to go ahead and get this suite in and give other= devs a chance
>> >> to
>> >> use and iterate on this work.
>> >>
>> >> Thanks,
>> >>
>> >> Tira & George
>> >>
>> >> On Mon, Feb 6, 2017 at 5:32 AM, Dave Page <dpage@pgadm= in.org> wrote:
>> >>>
>> >>> Hi
>> >>>
>> >>> On Fri, Feb 3, 2017 at 9:56 PM, Atira Odhner <aod= hner@pivotal.io>
>> >>> wrote:
>> >>> > Hi Dave,
>> >>> >
>> >>> > Here is a new patch which includes the following= :
>> >>> > - randomized ports
>> >>> > - delete the acceptance_test_db database in setu= p in case a prior
>> >>> > run
>> >>> > failed
>> >>> > - fixed size browser window
>> >>>
>> >>> Definitely getting there :-). A couple of thoughts/qu= estions:
>> >>>
>> >>> - Now there are 2 tests in there, it's clear that= both the Python
>> >>> server and browser session are restarted for each tes= t. Can this be
>> >>> avoided? It'll really slow down test execution as= more and more are
>> >>> added.
>> >>>
>> >>> - We've got a new monster name:
>> >>>
>> >>>
>> >>> pgadmin.acceptance.tests.sql_template_selection_by_po= stgres_version_works_feature_test.SQLTemplateSelectionByPostgresVersionWork= sFeatureTest
>> >>> (which on disk is
>> >>> sql_template_select_by_postgres_version_works_feature= _test.py). Names
>> >>> like that really must be shortened to something more = sane and
>> >>> manageable.
>> >>>
>> >>> - I'm a little confused by why the tests cannot b= e run in server mode.
>> >>> The error says it's because the username/password= is unknown -
>> >>> however, both the pgAdmin and database server usernam= es and passwords
>> >>> are in test_config.json.
>> >>>
>> >>> Thanks!
>> >>>
>> >>> --
>> >>> Dave Page
>> >>> Blog: http://pgsnake.blogspot.c= om
>> >>> Twitter: @pgsnake
>> >>>
>> >>> EnterpriseDB UK: http://www.ent= erprisedb.com
>> >>> The Enterprise PostgreSQL Company
>> >>
>> >>
>> >
>>
>>
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com=
>> The Enterprise PostgreSQL Company



--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
--001a1145ed7ed8a910054819b0f1--