Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 363344 - Test suite does not handle service URIs with query parameters
Summary: Test suite does not handle service URIs with query parameters
Status: CLOSED FIXED
Alias: None
Product: Lyo
Classification: Technology
Component: TestSuite (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Samuel Padgett CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-09 11:08 EST by Samuel Padgett CLA
Modified: 2013-05-23 09:13 EDT (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Samuel Padgett CLA 2011-11-09 11:08:54 EST 
The test suite adds query parameters to service URIs (like query capabilities) by appending "?name=value" to the URI. This doesn't work if the service URI already has query parameters, however.

For instance, the Bugzilla adapter has a query URI like this.

http://localhost:8282/bugz/changerequests?productId=2

The test suite will try to run this query

http://localhost:8282/bugz/changerequests?productId=2?oslc.where=oslc_cm%3Astatus%3D%22NEW%22&oslc.select=oslc_cm%3Astatus

which is not correct.
Comment 1 Samuel Padgett CLA 2011-11-09 11:28:32 EST 
I have a fix. See this pull request from github.

https://github.com/eclipse/lyo.testsuite/pull/1
Comment 3 Michael Fiedler CLA 2011-11-11 13:18:53 EST 
Resolved by comment 2
Comment 5 Samuel Padgett CLA 2013-05-23 09:13:23 EDT 
Closing...