Properly document and implement "menu_path"

+++ b/restws.api.php
@@ -54,12 +54,15 @@ function hook_restws_resource_info() {
+  $resource_info['node']['menu_info'] = 'mypath';

should be menu_path, not menu_info. Also documentation on hook_restws_resource_info() is missing.

+++ b/restws.info
@@ -3,5 +3,5 @@ description = Provides RESTful web services.
-files[] = restws.test
+files[] = tests/restws.test

"For modules, a single [module name].test file is the general rule. It should be placed directly in the module folder." from http://drupal.org/node/325974

+++ b/restws.module
@@ -209,40 +209,43 @@ function restws_menu_alter(&$items) {
+    // Remove the slash at the end for the querying paths if menu is set, or use
+    // the resource name as query path.

That comment does not relate to any code statements here?

+++ b/tests/restws_test.module
@@ -0,0 +1,13 @@
+/**
+ * Implements hook_restws_resource_info_alter().
+ */
+function restws_test_restws_resource_info_alter(&$resource_info) {
+  $resource_info['node']['menu_path'] = 'foo';
+}

Should have an @see reference to the test method.

Did some small modifications and committed this.

Now we need to fix all the URLs that are being used in resource references and querying. And we need tests that verify that the changed menu path is printed correctly in the format output.

Committed a slightly modified version of your patch.

Now the only missing thing are tests that verify that the changed menu path is printed correctly in the format output.

Do not duplicate the httpRequest() method. The new test class should either extend the first class or you need a third class as common parent class to the actual test classes.

I guess module_enable() did not work because the menu needs to be rebuild, so that the new paths get recognized (menu_rebuild()). Please try that again.

Tagging.

Marked #2286515: Multipart menu_path's are not working as a duplicate.

The testMenuPath() method is quite broken now and does not test the right thing. Maybe we should make restws_test_restws_resource_info_alter() configurable with variable_get(), so that each test can specify what the test path should look like.

Excellent work, almost there!

1. +++ b/restws.test
   @@ -541,8 +539,21 @@ class RestWSTestCase extends DrupalWebTestCase {
   +      $this->assertResponse(200);
   

   We should also assert that the returned content in the response body is correct. You can check for the title of the node for example.

2. +++ b/restws.test
   @@ -541,8 +539,21 @@ class RestWSTestCase extends DrupalWebTestCase {
   +      $this->assertResponse(200);
   

   Same here.

Yeah no, the original assertion is wrong. Comparing the title that we generated with the node does not make sense. You need to pull out the title from the result of drupalGet().

Committed, thanks!

Oops, forgot to push that commit, done now!