Fixed mishandling of negative oid refs

Author: krmorse

src/com/rallydev/rest/util/Ref.java

@@ -20,7 +20,7 @@ public class Ref {
             Pattern.compile(".*?/(\\w{2,}/\\w+)/(\\d+)(?:\\.js\\??.*)*$"),
 
             //collection ref (/defect/1234/tasks)
-            Pattern.compile(".*?/(\\w+/\\d+)/(\\w+)(?:\\.js\\??.*)*$"),
+            Pattern.compile(".*?/(\\w+/-?\\d+)/(\\w+)(?:\\.js\\??.*)*$"),
 
             //basic ref (/defect/1234)
             Pattern.compile(".*?/(\\w+)/(\\d+)(?:\\.js\\??.*)*$"),

test/com/rallydev/rest/util/RefTest.java

@@ -118,5 +118,6 @@ public void shouldSupportCollectionRefs() {
         Assert.assertEquals(Ref.getRelativeRef("/defect/1234/tasks"), "/defect/1234/tasks", "Valid relative collection ref");
         Assert.assertEquals(Ref.getRelativeRef("https://rally1.rallydev.com/slm/webservice/1.38/portfolioitem/feature/1234/children"), "/portfolioitem/feature/1234/children", "Valid dynatype collection ref");
         Assert.assertEquals(Ref.getRelativeRef("/portfolioitem/feature/1234/children"), "/portfolioitem/feature/1234/children", "Valid dynatype relative collection ref");
+        Assert.assertEquals(Ref.getRelativeRef("/typedefinition/-12345/attributes"), "/typedefinition/-12345/attributes", "Valid built-in type def attributes relative collection ref");
     }
 }