testsuitegenerator: Use normal methods more instead of lambdas
testsuitegenerator: Use normal methods more instead of lambdas

I am not sure why I wrote it this way, but if I had to guess:

* Using lambdas allowed for the fancy syntax with currying and with
  passing the lambda as a block to another method
* Lambdas inherited the scope, so we did not have to pass around a few
  of the variables, they were kind of all global

Now older and wiser, I think the fancy syntax is often a bad idea
(especially in a repository where Ruby code is the exception), and
treating all variables as global is almost always a bad idea.

Change-Id: Icea57bdb8397e128048cc13e9cd2ed7f166d726b

wikimedia/oojs-ui,wikimedia/oojs-ui,wikimedia/oojs-ui,wikimedia/oojs-ui