Making job name case sensitive, and those allowing to have multiple jobs with same name that differs with upper/lower case - as it's possible to setup it like this in Jenkins

Wojciech Strzalka · pnathan · commit 0d617ddbec41 · 2016-04-16T11:28:30.000-07:00
diff --git a/src/main/java/com/offbytwo/jenkins/JenkinsServer.java b/src/main/java/com/offbytwo/jenkins/JenkinsServer.java
@@ -146,7 +146,7 @@ public Map<String, Job> getJobs(FolderJob folder, String view) throws IOExceptio
             @Override
             public String apply(Job job) {
                 job.setClient(client);
-                return job.getName().toLowerCase();
+                return job.getName();
             }
         });
     }
diff --git a/src/main/java/com/offbytwo/jenkins/model/FolderJob.java b/src/main/java/com/offbytwo/jenkins/model/FolderJob.java
@@ -50,7 +50,7 @@ public Map<String, Job> getJobs() {
             @Override
             public String apply(Job job) {
                 job.setClient(client);
-                return job.getName().toLowerCase();
+                return job.getName();
             }
         });
     }
@@ -66,7 +66,7 @@ public Job getJob(String name) {
             @Override
             public String apply(Job job) {
                 job.setClient(client);
-                return job.getName().toLowerCase();
+                return job.getName();
             }
         }).get(name);
     }
diff --git a/src/test/java/com/offbytwo/jenkins/JenkinsServerTest.java b/src/test/java/com/offbytwo/jenkins/JenkinsServerTest.java
@@ -50,7 +50,7 @@ public void setUp() throws Exception {
 
     @Test
     public void shouldReturnListOfJobs() throws Exception {
-        assertTrue(server.getJobs().containsKey("hello"));
+        assertTrue(server.getJobs().containsKey("Hello"));
     }
 
     @Test

Original file line number	Diff line number	Diff line change
`@@ -146,7 +146,7 @@ public Map<String, Job> getJobs(FolderJob folder, String view) throws IOExceptio`
`146`	`146`	`@Override`
`147`	`147`	`public String apply(Job job) {`
`148`	`148`	`job.setClient(client);`
`149`		`- return job.getName().toLowerCase();`
	`149`	`+ return job.getName();`
`150`	`150`	`}`
`151`	`151`	`});`
`152`	`152`	`}`
Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ public Map<String, Job> getJobs() {`
`50`	`50`	`@Override`
`51`	`51`	`public String apply(Job job) {`
`52`	`52`	`job.setClient(client);`
`53`		`- return job.getName().toLowerCase();`
	`53`	`+ return job.getName();`
`54`	`54`	`}`
`55`	`55`	`});`
`56`	`56`	`}`
`@@ -66,7 +66,7 @@ public Job getJob(String name) {`
`66`	`66`	`@Override`
`67`	`67`	`public String apply(Job job) {`
`68`	`68`	`job.setClient(client);`
`69`		`- return job.getName().toLowerCase();`
	`69`	`+ return job.getName();`
`70`	`70`	`}`
`71`	`71`	`}).get(name);`
`72`	`72`	`}`
Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ public void setUp() throws Exception {`
`50`	`50`
`51`	`51`	`@Test`
`52`	`52`	`public void shouldReturnListOfJobs() throws Exception {`
`53`		`- assertTrue(server.getJobs().containsKey("hello"));`
	`53`	`+ assertTrue(server.getJobs().containsKey("Hello"));`
`54`	`54`	`}`
`55`	`55`
`56`	`56`	`@Test`