zeitgeist team mailing list archive

[Markus Korn <thekorn@xxxxxx>]

The proposal to merge lp:~thekorn/zeitgeist/fix.test.failures.20100826 into lp:zeitgeist has been updated.

Description changed to:

This branch fixes the failures in our test suite we have right now.

* Event.new_for_values() does also have some optional arguments prefixed
  "subject_"
* added convenient method to split Event.new_for_values() arguments from
  arguments which are handed over to another method

To verify if this brnach does the right thing run our testsuite in lp:zeitgeist and compare it to the result you get when running the testsuite in this branch, there shouldn't be failures anymore.

Once this branch landed and we have a working testsuite I would like to discuss ways to simplify the whole bunch of methods we have with anonymous/optional arguments (e.g. **kwargs)


-- 
https://code.launchpad.net/~thekorn/zeitgeist/fix.test.failures.20100826/+merge/33816
Your team Zeitgeist Framework Team is requested to review the proposed merge of lp:~thekorn/zeitgeist/fix.test.failures.20100826 into lp:zeitgeist.