kjkmadness 2014-03-03
fix broken tests
@43fd75963ddc68312a6693e74b4c7360e3bd192f
test/controllers/EnrollProjectAppTest.java
--- test/controllers/EnrollProjectAppTest.java
+++ test/controllers/EnrollProjectAppTest.java
@@ -57,7 +57,7 @@
                 fakeRequest(POST, routes.EnrollProjectApp.enroll(loginId, projectName).url()));
 
         // Then
-        assertThat(status(result)).isEqualTo(Http.Status.NOT_FOUND);
+        assertThat(status(result)).isEqualTo(Http.Status.FORBIDDEN);
     }
 
     @Test
@@ -125,7 +125,7 @@
                 fakeRequest(POST, routes.EnrollProjectApp.cancelEnroll(loginId, projectName).url()));
 
         // Then
-        assertThat(status(result)).isEqualTo(Http.Status.NOT_FOUND);
+        assertThat(status(result)).isEqualTo(Http.Status.FORBIDDEN);
     }
 
     @Test
test/controllers/PullRequestAppTest.java
--- test/controllers/PullRequestAppTest.java
+++ test/controllers/PullRequestAppTest.java
@@ -78,7 +78,7 @@
                 .withSession(UserApp.SESSION_USERID, currentUser.id.toString())
               );
 
-        assertThat(status(result)).isEqualTo(NOT_FOUND);
+        assertThat(status(result)).isEqualTo(FORBIDDEN);
     }
 
     @Test
@@ -239,7 +239,7 @@
                 .withSession(UserApp.SESSION_USERID, currentUser.id.toString())
               );
 
-        assertThat(status(result)).isEqualTo(NOT_FOUND);
+        assertThat(status(result)).isEqualTo(FORBIDDEN);
     }
 
     @Test
test/controllers/WatchProjectAppTest.java
--- test/controllers/WatchProjectAppTest.java
+++ test/controllers/WatchProjectAppTest.java
@@ -45,7 +45,7 @@
                 );
 
         //then
-        assertThat(status(result)).isEqualTo(Http.Status.NOT_FOUND);
+        assertThat(status(result)).isEqualTo(Http.Status.FORBIDDEN);
     }
 
     @Test
@@ -93,7 +93,7 @@
                 );
 
         //then
-        assertThat(status(result)).isEqualTo(Http.Status.NOT_FOUND);
+        assertThat(status(result)).isEqualTo(Http.Status.FORBIDDEN);
     }
 
     @Test
test/utils/ConfigTest.java
--- test/utils/ConfigTest.java
+++ test/utils/ConfigTest.java
@@ -74,10 +74,10 @@
         Helpers.stop(app);
 
         // The default is "Yobi".
-        additionalConfiguration.put("application.scheme", null);
+        additionalConfiguration.put("application.siteName", null);
         app = support.Helpers.makeTestApplication(additionalConfiguration);
         Helpers.start(app);
-        assertThat(Config.getScheme("https")).isEqualTo("Yobi");
+        assertThat(Config.getSiteName()).isEqualTo("Yobi");
         Helpers.stop(app);
     }
 }
Add a comment
List