Re: Build failed in Jenkins: pgadmin4-master-python33 #507

public inbox for [email protected]  
help / color / mirror / Atom feed

From: Khushboo Vashi <[email protected]>
To: Dave Page <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: Build failed in Jenkins: pgadmin4-master-python33 #507
Date: Thu, 8 Mar 2018 15:49:44 +0530
Message-ID: <CAFOhELdHYwF2FmU7xfpv8N8FtRYjAs7bU0w=CU6g3q_wvYsnWA@mail.gmail.com> (raw)
In-Reply-To: <CA+OCxozhh9caZ-Y1BNsqsSRgY45Wa6ChiPiWajQDZjGZE_ykTg@mail.gmail.com>
References: <[email protected]>
	<CA+OCxoxrPgB_mv_-50z9d2hj7iHQyeMJGhXtEkGfFMYf4_Ep+w@mail.gmail.com>
	<CAFOhELfx_KfUmFc+McR+maHJgLfsi+z+CMfnn3VPgdv539XtuQ@mail.gmail.com>
	<CA+OCxoziegBKj165ysM86fWL3m1CFN5+hnVBDjx3D9eeFHH6=Q@mail.gmail.com>
	<CAFOhELc+=eod5gtxQvRw=G2t=BQWb=2SgHOxbLrA2wxEROutzQ@mail.gmail.com>
	<CA+OCxox9OOr-UNS3WvMLB4ayZF+xvVjBbZH7JEi-C=fWQ2tZuw@mail.gmail.com>
	<CAFOhELe+5kX67EWH7nZc8965doOLanvqORKiqGX0BUyh42Hb-A@mail.gmail.com>
	<CA+OCxoy3EQogDYScsTSKas6SX0x=L8SW2RZ1Cd5hA=J7Y7itOA@mail.gmail.com>
	<CA+OCxoyMfKvF6jxFtR7w4Zi5=aUVMtcsXtJw_0ouDRgQRQtr8Q@mail.gmail.com>
	<CAFOhELddhg2i+_Hc152ojpejFp97B=bMMLqfNMoL99AbL1tAXA@mail.gmail.com>
	<CAFOhELed+dfW1r3_hfOc1LgxHEkr_8T9423aTr=RhUuNm6FL-w@mail.gmail.com>
	<CAFOhELc577sDgVxCNYuvC9S=5+qer2So2JW9FXEtb3FLpFPcGA@mail.gmail.com>
	<CA+OCxozG5+wHq1LQao8Tj6qQ9c+TD85HG1=+pQynDkFWy86jRA@mail.gmail.com>
	<CA+OCxoz0R+UoyOEh_4__TyyZ9G_nkOi3uHofMs=HjGc3mzavtw@mail.gmail.com>
	<CAFOhELcpCQb775=M5PZL9uY+2kpBba6Z-8tpV5TAucQzU8NSCg@mail.gmail.com>
	<CA+OCxozhh9caZ-Y1BNsqsSRgY45Wa6ChiPiWajQDZjGZE_ykTg@mail.gmail.com>

On Thu, Mar 8, 2018 at 3:35 PM, Dave Page <[email protected]> wrote:

>
>
> On Thu, Mar 8, 2018 at 9:47 AM, Khushboo Vashi <
> [email protected]> wrote:
>
>>
>>
>> On Thu, Mar 8, 2018 at 3:08 PM, Dave Page <[email protected]> wrote:
>>
>>> Well, it looks like Python 3 might be happy now. 2.6 and 2.7 aren't
>>> though (which is odd, as it passed on my local 2.7).
>>>
>>> Please find the attached patch to fix the test issue in PY2. I have
tested it on python 2.7 and 3.5 with 3 servers.

> I am looking into this. Btw, which command do you use to run tests?
>>
>
> make check-python
>
>
>
>> On Thu, Mar 8, 2018 at 9:13 AM, Dave Page <[email protected]> wrote:
>>>
>>>> Thanks, applied.
>>>>
>>>> On Thu, Mar 8, 2018 at 4:26 AM, Khushboo Vashi <
>>>> [email protected]> wrote:
>>>>
>>>>>
>>>>>
>>>>> On Wed, Mar 7, 2018 at 11:13 PM, Khushboo Vashi <
>>>>> [email protected]> wrote:
>>>>>
>>>>>> Hi Dave,
>>>>>>
>>>>>> Please find the patch to fix this.
>>>>>> I have fixed this issue in the StartRunningQueryTest class, this test
>>>>>> class changes the behaviour of the methods through MagicMock and due to
>>>>>> this this test case is failing.
>>>>>>
>>>>>> Revised patch with some modification.
>>>>>
>>>>>> Thanks,
>>>>>> Khushboo
>>>>>>
>>>>>>
>>>>>> On Wed, Mar 7, 2018 at 10:34 PM, Khushboo Vashi <
>>>>>> [email protected]> wrote:
>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> On Wed, Mar 7, 2018 at 10:33 PM, Dave Page <[email protected]>
>>>>>>> wrote:
>>>>>>>
>>>>>>>> It still seems to be failing :-(
>>>>>>>>
>>>>>>>> :(
>>>>>>>
>>>>>>>> https://jenkins.pgadmin.org/job/pgadmin4-master-python26/las
>>>>>>>> tFailedBuild/consoleFull
>>>>>>>>
>>>>>>>> On Wed, Mar 7, 2018 at 4:50 PM, Dave Page <[email protected]>
>>>>>>>> wrote:
>>>>>>>>
>>>>>>>>> Thanks, applied. Lets see what happens.
>>>>>>>>>
>>>>>>>>> On Wed, Mar 7, 2018 at 4:37 PM, Khushboo Vashi <
>>>>>>>>> [email protected]> wrote:
>>>>>>>>>
>>>>>>>>>> Please find the attached updated patch.
>>>>>>>>>> This should work.
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> On Wed, Mar 7, 2018 at 9:46 PM, Dave Page <[email protected]>
>>>>>>>>>> wrote:
>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> On Wed, Mar 7, 2018 at 4:13 PM, Khushboo Vashi <
>>>>>>>>>>> [email protected]> wrote:
>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>> On Wed, Mar 7, 2018 at 9:23 PM, Dave Page <[email protected]>
>>>>>>>>>>>> wrote:
>>>>>>>>>>>>
>>>>>>>>>>>>> Hi
>>>>>>>>>>>>>
>>>>>>>>>>>>> On Wed, Mar 7, 2018 at 3:10 PM, Khushboo Vashi <
>>>>>>>>>>>>> [email protected]> wrote:
>>>>>>>>>>>>>
>>>>>>>>>>>>>> Hi Dave,
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> On Wed, Mar 7, 2018 at 7:34 PM, Dave Page <[email protected]>
>>>>>>>>>>>>>> wrote:
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> Khushboo, can you look at this ASAP please? It passed tests
>>>>>>>>>>>>>>> on my machine, but Jenkins isn't happy.
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> I have tried to reproduce the issue with many combination
>>>>>>>>>>>>>> but couldn't reproduce it.
>>>>>>>>>>>>>> With the help of logs, I have tried to fix it. Can you please
>>>>>>>>>>>>>> check this fix is helpful or not?
>>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>> On my laptop I now get:
>>>>>>>>>>>>>
>>>>>>>>>>>>> ============================================================
>>>>>>>>>>>>> ==========
>>>>>>>>>>>>> ERROR: runTest (pgadmin.tools.sqleditor.tests
>>>>>>>>>>>>> .test_poll_query_tool.TestPollQueryTool)
>>>>>>>>>>>>> When query tool polling returns messages with result data-set
>>>>>>>>>>>>> ------------------------------------------------------------
>>>>>>>>>>>>> ----------
>>>>>>>>>>>>> Traceback (most recent call last):
>>>>>>>>>>>>>   File "/Users/dpage/git/pgadmin4/web
>>>>>>>>>>>>> /pgadmin/tools/sqleditor/tests/test_poll_query_tool.py", line
>>>>>>>>>>>>> 104, in runTest
>>>>>>>>>>>>>     response_data['data']['additional_messages'])
>>>>>>>>>>>>>   File "/System/Library/Frameworks/Py
>>>>>>>>>>>>> thon.framework/Versions/2.7/lib/python2.7/unittest/case.py",
>>>>>>>>>>>>> line 802, in assertIn
>>>>>>>>>>>>>     if member not in container:
>>>>>>>>>>>>> TypeError: argument of type 'NoneType' is not iterable
>>>>>>>>>>>>>
>>>>>>>>>>>>> ------------------------------------------------------------
>>>>>>>>>>>>> ----------
>>>>>>>>>>>>>
>>>>>>>>>>>>> Please provide some more details. Is this failing for
>>>>>>>>>>>> particular server version?
>>>>>>>>>>>> I am not getting any error with this patch and without this
>>>>>>>>>>>> patch on PY2 and PY3 with a single/multiple servers.
>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> Python 2.7.10 on macOS Sierra, testing against PostgreSQL 9.4.
>>>>>>>>>>> Here are the package versions in the venv:
>>>>>>>>>>>
>>>>>>>>>>> alabaster==0.7.10
>>>>>>>>>>> alembic==0.9.6
>>>>>>>>>>> args==0.1.0
>>>>>>>>>>> Babel==2.3.4
>>>>>>>>>>> backports.csv==1.0.4
>>>>>>>>>>> beautifulsoup4==4.4.1
>>>>>>>>>>> blinker==1.3
>>>>>>>>>>> chromedriver-installer==0.0.6
>>>>>>>>>>> click==6.6
>>>>>>>>>>> clint==0.5.1
>>>>>>>>>>> dateutils==0.6.6
>>>>>>>>>>> docutils==0.13.1
>>>>>>>>>>> extras==1.0.0
>>>>>>>>>>> fixtures==3.0.0
>>>>>>>>>>> Flask==0.11.1
>>>>>>>>>>> Flask-Babel==0.11.1
>>>>>>>>>>> Flask-Gravatar==0.4.2
>>>>>>>>>>> Flask-HTMLmin==1.2
>>>>>>>>>>> Flask-Login==0.3.2
>>>>>>>>>>> Flask-Mail==0.9.1
>>>>>>>>>>> Flask-Migrate==2.0.3
>>>>>>>>>>> Flask-Paranoid==0.1.0
>>>>>>>>>>> Flask-Principal==0.4.0
>>>>>>>>>>> Flask-Script==2.0.6
>>>>>>>>>>> Flask-Security==1.7.5
>>>>>>>>>>> Flask-SQLAlchemy==2.1
>>>>>>>>>>> Flask-WTF==0.12
>>>>>>>>>>> funcsigs==1.0.2
>>>>>>>>>>> html5lib==1.0b3
>>>>>>>>>>> htmlmin==0.1.11
>>>>>>>>>>> imagesize==0.7.1
>>>>>>>>>>> importlib==1.0.3
>>>>>>>>>>> itsdangerous==0.24
>>>>>>>>>>> Jinja2==2.7.3
>>>>>>>>>>> linecache2==1.0.0
>>>>>>>>>>> Mako==1.0.7
>>>>>>>>>>> MarkupSafe==0.23
>>>>>>>>>>> minify==0.1.4
>>>>>>>>>>> mock==2.0.0
>>>>>>>>>>> passlib==1.6.2
>>>>>>>>>>> pbr==3.1.1
>>>>>>>>>>> pkginfo==1.4.1
>>>>>>>>>>> psycopg2==2.7.3.2
>>>>>>>>>>> pycodestyle==2.3.1
>>>>>>>>>>> pycrypto==2.6.1
>>>>>>>>>>> Pygments==2.2.0
>>>>>>>>>>> pyperclip==1.5.32
>>>>>>>>>>> pyrsistent==0.14.0
>>>>>>>>>>> python-dateutil==2.5.0
>>>>>>>>>>> python-editor==1.0.3
>>>>>>>>>>> python-mimeparse==1.6.0
>>>>>>>>>>> pytz==2014.10
>>>>>>>>>>> requests==2.13.0
>>>>>>>>>>> requests-toolbelt==0.7.1
>>>>>>>>>>> selenium==3.3.3
>>>>>>>>>>> simplejson==3.6.5
>>>>>>>>>>> six==1.11.0
>>>>>>>>>>> snowballstemmer==1.2.1
>>>>>>>>>>> speaklater==1.3
>>>>>>>>>>> Sphinx==1.5.3
>>>>>>>>>>> SQLAlchemy==1.0.14
>>>>>>>>>>> sqlparse==0.1.19
>>>>>>>>>>> testscenarios==0.5.0
>>>>>>>>>>> testtools==2.0.0
>>>>>>>>>>> traceback2==1.4.0
>>>>>>>>>>> unittest2==1.1.0
>>>>>>>>>>> Werkzeug==0.9.6
>>>>>>>>>>> WTForms==2.0.2
>>>>>>>>>>> yuicompressor==2.4.8
>>>>>>>>>>>
>>>>>>>>>>> --
>>>>>>>>>>> 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
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>>>
>>>>
>>>> --
>>>> 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
>


Attachments:

  [text/x-patch] testcase_fix_py2.patch (818B, 3-testcase_fix_py2.patch)
  download | inline diff:
diff --git a/web/pgadmin/tools/sqleditor/utils/tests/test_start_running_query.py b/web/pgadmin/tools/sqleditor/utils/tests/test_start_running_query.py
index bc95e5a..23a5c7f 100644
--- a/web/pgadmin/tools/sqleditor/utils/tests/test_start_running_query.py
+++ b/web/pgadmin/tools/sqleditor/utils/tests/test_start_running_query.py
@@ -529,6 +529,6 @@ class StartRunningQueryTest(BaseTestGenerator):
     def tearDown(self):
         #  Reset methods to the original state
         StartRunningQuery.is_rollback_statement_required = \
-            self.is_rollback_statement_required
+            staticmethod(self.is_rollback_statement_required)
         StartRunningQuery.is_rollback_statement_required = \
-            self.is_rollback_statement_required
+            staticmethod(self.is_rollback_statement_required)

view thread (185+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected]
  Subject: Re: Build failed in Jenkins: pgadmin4-master-python33 #507
  In-Reply-To: <CAFOhELdHYwF2FmU7xfpv8N8FtRYjAs7bU0w=CU6g3q_wvYsnWA@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox