Functional Endpoint Tests and Unit Tests for services.module

Here is patch for testing node resource with REST server. See #984426: Testing REST server node resourse. for more details.

eeek, ygerasimov, I already did this for Node Endpoint testing. Its in the dev branches for 6.x and 7.x already.

I have done review of the tests for 6.x-3.x branch and edited them a bit. Mainly replaced statements like:

if (...) {
  $this->pass(...);
else {
  $this->fail(...);
}

with simple $this->assertTrue()

Also changed curlExec to have proper call method (GET, PUT, POST, DELETE) displayed.

Is there any specific reason to use json format in calls instead of serialized PHP? I believe we can simplify preparing arguments if we use serialized PHP (we will not have functions services_build_postfields, services_flatten_fields).

Hey ygerasimov, I like the changes, but IM curious why this wasnt done to the ServicesResourceNodeTests.test.

Can they be made to that file so the patch applies cleanly.

I think your version of services might be a bit outdated. See beta2

I restructured the code a bit so that its easier to run tests one at a time. The old way would have taken like10 minutes to run through all the resource endpoints. This way now allows us to have a set of test cases per resource and we can run them one by one, saving a ton of time.

I have rerolled patch and done following:
1. Changed calls to expect serialized php.
2. Changed formatting (spaces problems).
3. Fixed definition of constructor for class ServicesArgumentException

Here is patch with tests for user resource.

Please note that in order to pass it will need to be fixed issues #1013720: User resource. Method index SQL error when parameters passed. and #967786: Rest_server passing GET arguments

Here is patch with tests for taxonomy resource.

In order to pass patches from following issues should be applied:
#967786: Rest_server passing GET arguments
#1025186: Taxonomy resource. selectNodes method default values

Here is patch with all tests including tests for comment and service resources.

Needed patches to pass tests:
#967786: Rest_server passing GET arguments
#1025186: Taxonomy resource. selectNodes method default values
#1028682: System resource. set_variable method

hm... something gone wrong with patch. Another try.

Ok These tests have been comitted for 6.x, setting this to 7.x as thees tests need to be forward ported

I have started working on tests for D7. Here is node resource test.

In order to pass following patches should be applied:
#1039144: Node resource. Update method Exceptions
#1039142: Node resource. Nid argument type missing

These have been committed, and yes yes yes all tests passed!!

Setting to needs work b/c now all we need Taxonomy, File, Comment, User and system.

Test is ready for user_resource. There a lot of changes:
1. node_resource.inc -- minor styling changes
2. user_resource.inc
a. delete method default arguments
b. update method rebuilt
c. login action small changes
d. index method small changes
3. services.runtime.inc -- add default value of $data value for constructor of ServicesArgumentException
4. Test for user resource itself.

ALso what I have noticed is in D6 in login method variables called 'name' and 'pass' but in D7 they are 'username' and 'password'. I think we should have the same names (I like full names 'username' and 'password'). Please advise your comments about this so I will prepare patch for both resource and test.

Were closing this and separating the remaining issues out to their own tickets.