aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGravatar Vitaly Takmazov2018-02-09 00:24:17 +0300
committerGravatar Vitaly Takmazov2018-02-09 00:24:42 +0300
commit515d5a80eb9ace41e1aa94c6f880738efc56e3b9 (patch)
tree55eb4929d42aef12fcee8c4214ccb57ce98c1245
parent7414f1034d32c249294a081f1e176a9266fc92ac (diff)
www: 404 for non existent blog
-rw-r--r--juick-www/src/main/java/com/juick/www/controllers/Messages.java2
-rw-r--r--juick-www/src/test/java/com/juick/www/WebAppTests.java4
2 files changed, 5 insertions, 1 deletions
diff --git a/juick-www/src/main/java/com/juick/www/controllers/Messages.java b/juick-www/src/main/java/com/juick/www/controllers/Messages.java
index 5a76256e..d862e979 100644
--- a/juick-www/src/main/java/com/juick/www/controllers/Messages.java
+++ b/juick-www/src/main/java/com/juick/www/controllers/Messages.java
@@ -224,7 +224,7 @@ public class Messages {
ModelMap model) throws IOException {
com.juick.User user = userService.getUserByName(uname);
com.juick.User visitor = UserUtils.getCurrentUser();
- if (user.isBanned()) {
+ if (user.isBanned() || user.isAnonymous()) {
throw new HttpNotFoundException();
}
diff --git a/juick-www/src/test/java/com/juick/www/WebAppTests.java b/juick-www/src/test/java/com/juick/www/WebAppTests.java
index dcbcb66c..01dfcdcb 100644
--- a/juick-www/src/test/java/com/juick/www/WebAppTests.java
+++ b/juick-www/src/test/java/com/juick/www/WebAppTests.java
@@ -325,4 +325,8 @@ public class WebAppTests {
.andExpect(model().attribute("visitor", hasProperty("authHash", equalTo(hash))))
.andExpect(content().string(containsString(hash)));
}
+ @Test
+ public void nonExistentBlogShouldReturn404() throws Exception {
+ mockMvc.perform(get("/ololoe/")).andExpect(status().isNotFound());
+ }
}