UriBuilder.fromPath(String path= null)

[glassfishrobot]

It does not throw IllegalArgumentException as specified in API javadoc

Environment

Operating System: All
Platform: All

[glassfishrobot]

Reported by djiao@java.net

[glassfishrobot]

Was assigned to jersey-issues

[glassfishrobot]

sandoz@java.net said:
Accept.

[glassfishrobot]

djiao@java.net said:
Same problem with UriBuilder.fromUri(String uri) when uri is null.

[glassfishrobot]

sandoz@java.net said:
Fixed in the trunk.

[glassfishrobot]

djiao@java.net said:
Tried again with July 14th jersey. Same problem still shows up. Please look at
it again.

[glassfishrobot]

sandoz@java.net said:
The following Jersey unit test passes:

public void testIllegalArgumentException() {
boolean caught = false;
try

{ UriBuilder.fromPath(null); }

catch(IllegalArgumentException e)

{ caught = true; }
assertTrue(caught);

caught = false;
try { UriBuilder.fromUri((URI)null); } catch(IllegalArgumentException e) { caught = true; }

assertTrue(caught);

caught = false;
try

{ UriBuilder.fromUri((String)null); }

catch(IllegalArgumentException e)

{ caught = true; }

assertTrue(caught);
}

Suggest that you check the version of Jersey and your test code as i have not
made any changes.

[glassfishrobot]

djiao@java.net said:
The failing case is:

caught = false;
try

{ //UriBuilder.fromUri((String)null); //this one passes for me as well UriBuilder.fromUri((""); }

catch(IllegalArgumentException e)

{ caught = true; }

assertTrue(caught);

[glassfishrobot]

djiao@java.net said:
String "" is a valid uri.

[glassfishrobot]

Marked as fixed on Wednesday, July 23rd 2008, 6:31:05 am

[glassfishrobot]

This issue was imported from java.net JIRA JERSEY-48