public inbox for [email protected]
help / color / mirror / Atom feedFrom: Dave Page <[email protected]>
To: Navnath Gadakh <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Cc: Kanchan Mohitey <[email protected]>
Subject: Re: pgAdmin IV : Unittest modular patch
Date: Fri, 15 Jul 2016 13:53:53 +0100
Message-ID: <CA+OCxoxqahNgLmvQqqErQZL2JS0EaQwyXkgkzvkFtGNh0OKV6w@mail.gmail.com> (raw)
In-Reply-To: <CAOAJCYr1JwYK4x2wDYw2UVAZoXC5a0FQT7Yz1Y_Wn46kzPMmQg@mail.gmail.com>
References: <CAOAJCYrcxBN2J_3dWq0Cwc82fkugzUi3NKD4jKoTKRBot=ev8w@mail.gmail.com>
<CAOAJCYpSBXcfEC5d4Sv_=jV86m1=rKUK0tMkZFhdSHkPb=XT3A@mail.gmail.com>
<CA+OCxowW0wjpNT-o0YF1Bg9rtD6Z7sqp35ZRUgoEguVKO4OtSw@mail.gmail.com>
<CAOAJCYr1JwYK4x2wDYw2UVAZoXC5a0FQT7Yz1Y_Wn46kzPMmQg@mail.gmail.com>
List-Unsubscribe: <mailto:[email protected]?body=unsub%20pgadmin-hackers>
Hi
On Mon, Jul 11, 2016 at 2:16 PM, Navnath Gadakh <
[email protected]> wrote:
> Hi Dave,
> PFA updated patch for modular API test cases.
>
> On Mon, Jul 11, 2016 at 5:11 PM, Dave Page <[email protected]>
> wrote:
>
>>
>>
>> On Fri, Jul 8, 2016 at 2:30 PM, Navnath Gadakh <
>> [email protected]> wrote:
>>
>>>
>>> Hi Dave,
>>>
>>> PFA patch for modular API test cases.
>>>
>>> Kindly, review the same and let me know for any modification.
>>>
>>
>> Please re-create the patch so that it doesn't revert various changes made
>> to config.py etc. over the last couple of weeks.
>>
>
> config.py file was not updated at my end, I have updated it in the
> current patch. I have also added code related to advance test data
> configuration which was created by Priyanka (Seprated the test data
> configuration files).
>
>
>> What is the patch for? Is it fixing issues in Priyanka's last patch, or
>> something else?
>> This patch is for -
>> 1. Now we can run testsuite node/module wise(like for browser only, for
>> databases only) OR for all nodes/modules
>> 2. I have modified the existing functionality for testsuite and all test
>> case classes (Made all test cases independently runnable, removed
>> the priority logic).
>>
>
>
For more details please refer the file 'regression/README'.
>
When running all tests, I get:
...
...
<Rule '//static/<filename>' (HEAD, OPTIONS, GET) -> redirects.static>,
<Rule '/backup/<filename>' (HEAD, OPTIONS, GET) -> backup.static>,
<Rule '/reset/<token>' (HEAD, POST, OPTIONS, GET) ->
security.reset_password>,
<Rule '/about/<filename>' (HEAD, OPTIONS, GET) -> about.static>])
Traceback (most recent call last):
File "regression/testsuite.py", line 151, in <module>
suite = get_suite(args, test_client)
File "regression/testsuite.py", line 75, in get_suite
TestsGeneratorRegistry.load_generators('pgadmin')
File "/Users/dpage/git/pgadmin4/web/pgadmin/utils/route.py", line 56, in
load_generators
module = import_module(module_name)
File
"/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/importlib/__init__.py",
line 37, in import_module
__import__(name)
File
"/Users/dpage/git/pgadmin4/web/pgadmin/browser/server_groups/servers/databases/tests/test_db_add.py",
line 14, in <module>
from regression import test_utils as utils
File "/Users/dpage/git/pgadmin4/web/regression/test_utils.py", line 23,
in <module>
def get_ids(url=config.APP_ROOT + '/regression/parent_id.pkl'):
AttributeError: 'module' object has no attribute 'APP_ROOT'
That's not overly surprising, as we don't have an APP_ROOT setting in the
config (nor should we - it should be calculated dynamically). Please fix,
and remember to test your patches against a clean copy of the git tree in
the future!
Thanks!
--
Dave Page
VP, Chief Architect, Tools & Installers
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
view thread (29+ 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], [email protected]
Subject: Re: pgAdmin IV : Unittest modular patch
In-Reply-To: <CA+OCxoxqahNgLmvQqqErQZL2JS0EaQwyXkgkzvkFtGNh0OKV6w@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