You may have an opportunity here to refactor the tests for readability. I found the assertEquals() statements hard to read - a custom method like assertInQueryURL() may be easier to read, and also reduce the dependence on the entire query data structure:
Hi Curtis,
This looks like an excellent change, r=mars.
You may have an opportunity here to refactor the tests for readability. I found the assertEquals() statements hard to read - a custom method like assertInQueryURL() may be easier to read, and also reduce the dependence on the entire query data structure:
self. assertInQueryUR L('field. homepageurl' , 'http:// eg.dom/ bonkers')
Maris