~landscape/landscape-client/trunk

Viewing all changes in revision 674.

  • Committer: Tarmac
  • Author(s): Free Ekanayaka
  • Date: 2013-05-14 12:23:22 UTC
  • mfrom: (673.1.1 amp-test-readability)
  • Revision ID: tarmac-20130514122322-dzdzjo2mxj4h1er3
Merge amp-test-readability [f=1170669] [r=tribaal,radix] [a=Free Ekanayaka]
This is the last branch I have around cleaning up the AMP-based machinery in the client. It essentially just change the tests:

- Drop the Words class, which had a bunch of ad-hoc methods used to exercise the behavior of the MethodCall protocol. Essentially each method was associated with a test, but the context was not clear by reading the test itself (you'd have to go and read the implementation in the Words class).

- Modify most methods to define the remotely-exposed method being tested, so the context is clear.

- Partially re-wrote RemoteObjectTest, which had a bunch of tests which were only duplicating tests from MethodCallTest. Now this test class has only tests that are really specific to RemoteObject.

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: