#5206 Resolve import error on allura.controllers.root

unreleased
invalid
nobody
None
General
nobody
2015-01-31
2012-10-30
Cory Johns
No

Since the changes [#3883] to separate the repo tools from the core, the list of tools available on project creation is now populated dynamically. To both populate the inputs on the add_project page and handle validation properly, the fields need to be populated in the NeighborhoodAddProject form's __init__. However, the form is instantiated on import and the necessary setup (registring app_globals with pylons) has not been done yet, causing it to fail.

This causes issues with the test suite (since allura.controllers.test_neighborhood_root gets discovered and imported, which imports allura.controllers.root) and the push process (since it runs the smoke test which tries to verify that allura.controllers.root imports without error). These have temporary work-arounds in place (commits [ade7d6] and [c11f79], and using --nosmoking during Allura push), but the underlying issue needs to be resolved.

Related

Tickets: #3883
Commit: [ade7d6]
Commit: [c11f79]

Discussion

  • Dave Brondsema
    Dave Brondsema
    2014-04-01

    • status: open --> invalid
     
  • Dave Brondsema
    Dave Brondsema
    2014-04-01

    Works for me now:

    $ python
    Python 2.7.3 (default, Apr 12 2012, 21:46:02) 
    [GCC 4.1.2 20080704 (Red Hat 4.1.2-50)] on linux2
    Type "help", "copyright", "credits" or "license" for more information.
    >>> import allura.controllers.root
    >>>