From 1837f126a37d48501dcaed68eae21b28761651a6 Mon Sep 17 00:00:00 2001
From: Robert Marshall <rob@robware.uk>
Date: Fri, 10 Apr 2020 10:11:55 +0100
Subject: [PATCH] Test for invalid id

---
 src/Robware.Api.Blog.Tests/BlogControllerTests.cs  | 10 ++++++++++
 src/Robware.Api.Blog/Controllers/BlogController.cs |  6 ------
 2 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/src/Robware.Api.Blog.Tests/BlogControllerTests.cs b/src/Robware.Api.Blog.Tests/BlogControllerTests.cs
index 37eb192..7e034d1 100644
--- a/src/Robware.Api.Blog.Tests/BlogControllerTests.cs
+++ b/src/Robware.Api.Blog.Tests/BlogControllerTests.cs
@@ -46,7 +46,17 @@ namespace Robware.Api.Blog.Tests {
 
 			var controller = new BlogController(logger, repo);
 			(await controller.Get("url")).Result.Should().BeOfType<NotFoundObjectResult>();
+		}
 
+		[Fact]
+		public async Task Get_WithInvalidId_Returns404() {
+			var logger = Substitute.For<ILogger<BlogController>>();
+			var repo = Substitute.For<IBlogRepository>();
+
+			repo.GetPostByIdAsync(1).Throws(new ItemNotFoundException("", null));
+
+			var controller = new BlogController(logger, repo);
+			(await controller.Get("1")).Result.Should().BeOfType<NotFoundObjectResult>();
 		}
 	}
 }
diff --git a/src/Robware.Api.Blog/Controllers/BlogController.cs b/src/Robware.Api.Blog/Controllers/BlogController.cs
index 6a2f1ca..fe7b521 100644
--- a/src/Robware.Api.Blog/Controllers/BlogController.cs
+++ b/src/Robware.Api.Blog/Controllers/BlogController.cs
@@ -33,9 +33,6 @@ namespace Robware.Api.Blog.Controllers {
 			}
 		}
 
-		//[HttpGet]
-		//Task<BlogPost> GetPostByUrl();
-
 		//[HttpGet]
 		//Task<IEnumerable<BlogPost>> GetLatestPosts(int limit, int offset = 0);
 
@@ -45,9 +42,6 @@ namespace Robware.Api.Blog.Controllers {
 		//[HttpGet]
 		//Task<int> GetCount();
 
-		//[HttpGet]
-		//Task<BlogPost> GetPostById(int id);
-
 		//[HttpPost]
 		//Task<BlogPost> SavePost(BlogPost post);