[ https://issues.apache.org/jira/brow...ls:all-tabpanel ]

Graham Dumpleton closed MODPYTHON-119.
--------------------------------------


> DBM Session test shouldn't use default database.
> ------------------------------------------------
>
>                 Key: MODPYTHON-119
>                 URL: https://issues.apache.org/jira/browse/MODPYTHON-119
>             Project: mod_python
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 3.2.7
>            Reporter: Graham Dumpleton
>         Assigned To: Jim Gallacher
>             Fix For: 3.3, 3.2.10
>
>
> The test_Session_Session_conf() of test/test.py doesn't override the default DBM s
ession database name that would be used for prefork/worker MPM variants. This means 
that if there is already an instance of Apache running as its designated user which 
has
already created the session database and the tests are run as a different us
er, the test will fail. Further, if mod_python has never previously been use
d and no session database exists, the test will succeed, but when someone go
es to use sessions in the
installed mod_python, it will fail in the case where the session database created by the tes
t was as a different user to that which Apache runs as.
> The test suite should ensure that it overrides any defaults and stores suc
h databases in test suite document tree. It should possibly also ensure that
 such transient files are removed so as not to create a problem on subsequen
t tests.
> Test might therefore be changed to:
>     def test_Session_Session_conf(self):
>         database = os.path.join(TESTHOME,"mp_sess_test.dbm")
>         c = VirtualHost("*",
>                         ServerName("test_Session_Session"),
>                         DocumentRoot(DOCUMENT_ROOT),
>                         Directory(DOCUMENT_ROOT,
>                                   PythonOption('session_dbm "%s"' % databa
se),
>                                   SetHandler("mod_python"),
>                                   PythonHandler("tests::Session_Session"),
>                                   PythonDebug("On")))
>         return str(c)

[ Post a follow-up to this message ]