turbogears/Lobby

The web framework that scales with you. TurboGears 2.4.3 released https://github.com/TurboGears/tg2/releases/tag/tg2.4.3

Alessandro Molina

@amol-

As tests fixtures are not within a controller action called within a request, they are one of those cases where you need to manage on your own the transaction life.

@jknapka hope I addressed your doubts

Joe Knapka

@jknapka

@amol- Thank you... sorry I didn't reply sooner, it's 9AM here :-) I suspected it was something along these lines but I don't know sqlalchemy or the transaction module well enough to get to the bottom of it.

Yes, that totally works! Thank you!

Alessandro Molina

@amol-

:+1:

Joe Knapka

@jknapka

Hmm, still confused actually. If I create any DB entities inside my test methods, the same error occurs (unless I manually commit or abort the transaction). But test methods should be run inside a TG request and therefore with automatic transaction management, correct?

Alessandro Molina

@amol-

the test method is still outside of a TG reuqest

you would be running within a request if you did self.app.get(....) then that triggers request

otherwise in a test method you are not within an HTTP request

it's just a plain normal python function

so you need to commit the things youself

Joe Knapka

@jknapka

Ah! OK, thank you.

Joe Knapka

@jknapka

Hi! I'm having another testing problem. My test fixture creates a user in tg_user, then tries to load a page as that user in a test method, but authentication forging does not appear to work. The test method is annotated with @require(predicated.not_anonymous()) and the response from self.app.get("/the_url",extra_environ={'REMOTE_USER':'test_user'},response=200) is always a 401.

Also my code depends on being able to fetch the current tg_user object in order to generate the result page, but response.identity is None. How to handle this properly?

Nils Philippsen

@nphilipp

@jknapka how are you going at faking authenticated access?

Joe Knapka

@jknapka

According to the example code, it seems the extra_environ={"REMOTE_USER":"test_user"} argument to app.get() is supposed to do that, but maybe I misunderstand how this is intended to work.

(In other words by TG magic, I thought, but if I'm wrong I need to figure out how to do it right.)

Nils Philippsen

@nphilipp

Is the class in which your doing this derived from <project>.tests.TestController?

Joe Knapka

@jknapka

According to the quickstarted tests/init.py file, test suites should be run "with authentication disabled so that developers can test protected areas independently [of repoze.who]". Maybe that doesn't mean what I think it means.

Nils Philippsen

@nphilipp

I.e. you're modelling this after <project>.tests.functional.test_root?

Joe Knapka

@jknapka

Yes, exactly: I've copied test_root and am deriving from TestController.

Nils Philippsen

@nphilipp

Do the test_root tests succeed?

Joe Knapka

@jknapka

Yes.

Nils Philippsen

@nphilipp

Where are your tests located?

Joe Knapka

@jknapka

My other tests (that do not depend on particular user logins) also work fine.

project/tests/functional/test_blah.py

Nils Philippsen

@nphilipp

Hmm.

Where is your test_user created?

Ahh you mentioned it.

(Thinking aloud, I should stop that.)

Joe Knapka

@jknapka

In MyTestClass.setUp(). I can just paste the code here, although of course it won't run, but you could see the structure. It's only fifty lines or so. Shall I do that?

Nils Philippsen

@nphilipp

You could pastebin it somewhere.

E.g. as a github gist.

Joe Knapka

@jknapka

Hmm, actually it's a bit more complicated than it needs to be for clarity. I'll find a minimal reproducible case and post that somewhere in a little while.

Nils Philippsen

@nphilipp

I have a hunch that the tested method doesn't see the user because it's not committed to the DB or something similar.

Can you just pastebin the method that creates the user and adds it to the DB?

Joe Knapka

@jknapka

My "minimal reproduction" succeeds (unexpectedly), so I'm sure I'm missing something simple. I'll try to figure it out myself and post here again if I hit a brick wall. (You may very well be right about the DB issue, but I actually call the same utility method to create users in other test cases and that works. So I think it's something about how I've structured this specific test.)

Nils Philippsen

@nphilipp

Do the other test cases work against a test app, like the functional ones do?

Joe Knapka

@jknapka

Some do and some don't. I already figured out (from @amol_) that test fixtures and test cases that don't use a test app have to do manual transaction management, and I'm (probably!) handling that correctly :-D

Joe Knapka

@jknapka

Yep, my test fixture was raising an exception that was rolling back the txn! Thanks for the nudge in the right direction.

Nils Philippsen

@nphilipp

You're welcome!

Alessandro Molina

@amol-

@jknapka I suggest you verify the controller from which you are inheriting. There is a class attribute application_under_test (or something similar) that specified which configuration should be used to create the application instance used for the tests.

By default there are two applications configured in test.ini the main and mainnoauth (or something similar).

The noauth one supports faking authentication with the REMOTE_USER, while the other one requires you do to a self.app.post('/login_handler') to actually log before your can call authenticated endpoints

(It's main_without_authn the app name that accepts REMOTE_USER, just checked the name)

When you quickstart, you can see that there are two examples. test_root which uses main_without_authn (actually doesn't specify anything and main_without_authn is the default) and test_authentication which has application_under_test = 'main' at the begin of the controller.

You can see that tests in test_root can use REMOTE_USER

while the one under test_authentication do use real authentication and cookies

Joe Knapka

@jknapka

Thanks, @amol_ . I found the problem, my test fixture was causing an exception that rolled back its transaction. All the TG machinery around auth forging is working as expected.

Joe Knapka

@jknapka

Hello all, I am trying to add i18n to my TG2.4 application. This seems to have broken many of my test cases with the following exception while calling ugettext():

Traceback (most recent call last):
  File "/Users/jk/Software/py3-venv-2/lib/python3.7/site-packages/nose/case.py", line 198, in runTest
    self.test(*self.arg)
  File "/Users/jk/Software/unter/src/tg/unter/unter/tests/functional/test_multi_alerts.py", line 86, in test_0_oneVolunteerEvent
    self.sendAlert(ev_id=1)
  File "/Users/jk/Software/unter/src/tg/unter/unter/tests/functional/test_multi_alerts.py", line 82, in sendAlert
    need.checkOneEvent(model.DBSession,ev_id=ev_id,honorLastAlertTime=False)
  File "/Users/jk/Software/unter/src/tg/unter/unter/controllers/need.py", line 27, in checkOneEvent
    debugTest(_("Checking need event {} {}").format(ev_id,notes))
  File "/Users/jk/Software/py3-venv-2/lib/python3.7/site-packages/TurboGears2-2.4.0a1-py3.7.egg/tg/i18n.py", line 87, in ugettext
    return tg.translator.gettext(value)
  File "/Users/jk/Software/py3-venv-2/lib/python3.7/site-packages/TurboGears2-2.4.0a1-py3.7.egg/tg/support/objectproxy.py", line 19, in __getattr__
    return getattr(self._current_obj(), attr)
  File "/Users/jk/Software/py3-venv-2/lib/python3.7/site-packages/TurboGears2-2.4.0a1-py3.7.egg/tg/request_local.py", line 233, in _current_obj
    return getattr(context, self.name)
  File "/Users/jk/Software/py3-venv-2/lib/python3.7/site-packages/TurboGears2-2.4.0a1-py3.7.egg/tg/support/objectproxy.py", line 19, in __getattr__
    return getattr(self._current_obj(), attr)
  File "/Users/jk/Software/py3-venv-2/lib/python3.7/site-packages/TurboGears2-2.4.0a1-py3.7.egg/tg/support/registry.py", line 72, in _current_obj
    'thread' % self.____name__)
TypeError: No object (name: context) has been registered for this thread

I don't know where to begin fixing this. Advice would be appreciated. Thanks!

Where communities thrive

People

Repo info

Activity