Documentation codelistings need updating and tests

[jerub]

The code used in .tac files is too complicated. I'd like to be able to create an application object with a minimum of verbage.

What we have:

application = Application('myapp', uid=1, gid=1)
serviceCollection = IServiceCollection(application)
service1.setServiceParent(serviceCollection)
service2.setServiceParent(serviceCollection)

What I'd like

application = Application('myapp', uid=1, gid=1, servies=[service1, service2])

The tutorials and documentation would have to be updated to include this new form, and probably to continue to acknowledge how to do both ways.

[glyph]

You asked for my feedback, and my feedback is that I think this is a good idea (although definitely high enough priority for me to work on it...)

I think that this might apply to other twisted.application stuff as well, not just Application. Personally, I can never remember how to create a service without going and reading the code, and that's a very bad sign for other folks...

[jerub]

Branch is simple-app-2205. Tests and code have been updated, children=[] is the kwarg as per a suggestion on irc.

Still need to update the docs.

[thijs]

I can go through those docs.

[thijs]

Perhaps someone can review jerub's branch as well, putting it up for review.

[exarkun]

This conflicts horribly with trunk. The conflicts need to be resolved and the branch merged forward.

[thijs]

(In [25314]) Branching to 'simple-app-2205-2'

[thijs]

Replying to exarkun:

This conflicts horribly with trunk. The conflicts need to be resolved and the branch merged forward.

Done, see r25315

[thijs]

Forgot to add the tests, included in r25319

[therve]

Most of the tests are failing with this error:

[ERROR]: twisted.test.test_fingertutorial.Finger21TestCase.testFingerFactory

Traceback (most recent call last):
  File "/home/therve/projects/Twisted/trunk/twisted/test/test_fingertutorial.py", line 264, in setUp
    super(Finger19TestCase, self).setUp()
  File "/home/therve/projects/Twisted/trunk/twisted/test/test_fingertutorial.py", line 36, in setUp
    self.finger = __import__(self.module)
exceptions.ImportError: No module named finger21
-------------------------------------------------------------------------------
Ran 68 tests in 0.496s

FAILED (errors=47, successes=21)

[thijs]

kehander1 has been working on splitting up the finger example into libraries and the tests here should be made up to date for that.

[binjured]

kehander1 has stopped working on that (as far as I can tell) which takes this ticket to limbo with it. There seem to be a number of circular dependencies for the Finger tutorial to make any progress towards "maybe less awful".