From: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
---|---|
To: | Yosry Muhammad <yosrym93(at)gmail(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, Dave Page <dpage(at)pgadmin(dot)org> |
Subject: | Re: Transaction Status Python Tests |
Date: | 2019-08-19 12:29:59 |
Message-ID: | CANxoLDd0Vhoj4+hsUFO30j4yXdBRAVb=z-jHNWe8R3ZcgZLzDg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Yosry
Test cases with auto commit disabled gets failed for following scenario
only:
- Run all test cases on more than one server "*--pkg all* *--exclude
feature_tests"*. For first sever all the tests passed and for second
onwards fail for all.
- If you run only sqleditor test cases on multiple server then no
issues. Strange behaviour.
- We have tried manually on pgAdmin4 and seems the test cases should
pass logic seems correct to me.
def runTest(self):
self._create_test_table()
self._set_auto_commit(self.is_auto_commit_enabled)
self._execute_select_sql()
self._check_transaction_status(self.transaction_status) *--
transection status after executing the query is 0 and expected is 2*
self._save_changed_data()
self._check_transaction_status(self.transaction_status)
On Mon, Aug 19, 2019 at 5:50 PM Yosry Muhammad <yosrym93(at)gmail(dot)com> wrote:
> Can you send me the error message? Unfortunately there is a problem with
> my machine right now I am unable to investigate it myself, I am very sorry
> for the inconvenience.
>
> On Mon, Aug 19, 2019, 2:08 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
> wrote:
>
>> Hi Yosry
>>
>> We have tried a lot to figure out the issue but didn't find the actual
>> cause. Can you please figure out and send the patch asap, I'll have to
>> create candidate build tomorrow.
>>
>> On Mon, Aug 19, 2019 at 1:22 PM Yosry Muhammad <yosrym93(at)gmail(dot)com>
>> wrote:
>>
>>> Hi Akshay,
>>>
>>> I am sorry I do not have access to my machine right now. I will try to
>>> work on it (and the JS patch) once I get the chance to.
>>>
>>> On Mon, Aug 19, 2019, 9:49 AM Akshay Joshi <
>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Yosry
>>>>
>>>> Test cases are failing on our build machines. When I reviewed your
>>>> patch I have run test cases with paramaster "*--pkg
>>>> tools.sqleditor.tests*" on multiple database servers and every time
>>>> test case has been passed.
>>>>
>>>> But on build machines we run test cases with parameter "*--pkg all* *--exclude
>>>> feature_tests" *on multiple database server and it fails for second
>>>> server onwards. I have also tried the same and it is failing on my machine
>>>> too.
>>>>
>>>> I am debugging the code why it is getting failed, but if you found any
>>>> solution please send the patch asap.
>>>>
>>>> On Mon, Aug 19, 2019 at 11:48 AM Akshay Joshi <
>>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>>
>>>>> Thanks patch applied.
>>>>>
>>>>> On Sat, Aug 17, 2019 at 6:22 PM Yosry Muhammad <yosrym93(at)gmail(dot)com>
>>>>> wrote:
>>>>>
>>>>>> Hi Hackers,
>>>>>>
>>>>>> Please find attached a patch including new python test that checks
>>>>>> the transaction status after executing queries and saving data when
>>>>>> auto-commit is on/off. It also includes some refactoring of previously
>>>>>> written python tests.
>>>>>>
>>>>>> Please review !
>>>>>> Thanks. Regards.
>>>>>>
>>>>>> --
>>>>>> *Yosry Muhammad Yosry*
>>>>>>
>>>>>> Computer Engineering student,
>>>>>> The Faculty of Engineering,
>>>>>> Cairo University (2021).
>>>>>> Class representative of CMP 2021.
>>>>>> https://www.linkedin.com/in/yosrym93/
>>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> *Thanks & Regards*
>>>>> *Akshay Joshi*
>>>>>
>>>>> *Sr. Software Architect*
>>>>> *EnterpriseDB Software India Private Limited*
>>>>> *Mobile: +91 976-788-8246*
>>>>>
>>>>
>>>>
>>>> --
>>>> *Thanks & Regards*
>>>> *Akshay Joshi*
>>>>
>>>> *Sr. Software Architect*
>>>> *EnterpriseDB Software India Private Limited*
>>>> *Mobile: +91 976-788-8246*
>>>>
>>>
>>
>> --
>> *Thanks & Regards*
>> *Akshay Joshi*
>>
>> *Sr. Software Architect*
>> *EnterpriseDB Software India Private Limited*
>> *Mobile: +91 976-788-8246*
>>
>
--
*Thanks & Regards*
*Akshay Joshi*
*Sr. Software Architect*
*EnterpriseDB Software India Private Limited*
*Mobile: +91 976-788-8246*
From | Date | Subject | |
---|---|---|---|
Next Message | Yosry Muhammad | 2019-08-19 12:34:49 | Re: Transaction Status Python Tests |
Previous Message | Yosry Muhammad | 2019-08-19 12:20:08 | Re: Transaction Status Python Tests |