Merge pull request #68 from OpenEnade/ano-private-controller

Add Ano private controller
OpenEnade · Mar 3, 2019 · be9f337 · be9f337
2 parents 0c1e8e7 + b961d96
commit be9f337
Show file tree

Hide file tree

Showing 11 changed files with 60 additions and 86 deletions.
diff --git a/pom.xml b/pom.xml
@@ -60,6 +60,7 @@
 			<artifactId>h2</artifactId>
 			<scope>runtime</scope>
 		</dependency>
+
 	</dependencies>
 
 	<build>

diff --git a/src/main/java/br/com/openenade/api/ano/Ano.java b/src/main/java/br/com/openenade/api/ano/Ano.java
@@ -9,6 +9,13 @@ public class Ano {
     @Id
     private Integer ano;
 
+    public Ano() {
+
+    }
+
+    public Ano(Integer ano) {
+        this.ano = ano;
+    }
 
     public Integer getAno() {
         return this.ano;
@@ -18,13 +25,11 @@ public void setAno(Integer ano) {
         this.ano = ano;
     }
 
-
     @Override
     public String toString() {
         return "" + this.ano;
     }
 
-
     @Override
     public int hashCode() {
         final int prime = 31;

diff --git a/src/main/java/br/com/openenade/api/ano/AnoService.java b/src/main/java/br/com/openenade/api/ano/AnoService.java
@@ -19,9 +19,9 @@ public Collection<Ano> getAllAnos() {
     }
 
     public Ano getAno(Integer ano) {
-        Optional<Ano> anoo = this.repository.findById(ano);
-        if (anoo.isPresent()) {
-            return anoo.get();
+        Optional<Ano> optAno = this.repository.findById(ano);
+        if (optAno.isPresent()) {
+            return optAno.get();
         } else {
             throw new ResourceNotFound("" + ano);
         }

diff --git a/src/main/java/br/com/openenade/api/ano/AnoUtils.java b/src/main/java/br/com/openenade/api/ano/AnoUtils.java
diff --git a/src/main/java/br/com/openenade/api/ano/PrivateAnoController.java b/src/main/java/br/com/openenade/api/ano/PrivateAnoController.java
@@ -0,0 +1,27 @@
+package br.com.openenade.api.ano;
+
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.web.bind.annotation.DeleteMapping;
+import org.springframework.web.bind.annotation.PathVariable;
+import org.springframework.web.bind.annotation.PostMapping;
+import org.springframework.web.bind.annotation.RequestBody;
+import org.springframework.web.bind.annotation.RequestMapping;
+import org.springframework.web.bind.annotation.RestController;
+
+@RestController
+@RequestMapping(path = PublicAnoController.ENDPOINT)
+public class PrivateAnoController {
+
+    @Autowired
+    private AnoService service;
+
+    @PostMapping
+    public Ano createAno(@RequestBody Ano ano) {
+        return this.service.addAno(ano);
+    }
+
+    @DeleteMapping(path = "/{" + PublicAnoController.ANO_ID + "}")
+    public void deleteAno(@PathVariable(name = PublicAnoController.ANO_ID) Integer ano) {
+        this.service.deleteAno(ano);
+    }
+}
diff --git a/.../com/openenade/api/ano/AnoController.java → ...penenade/api/ano/PublicAnoController.java b/.../com/openenade/api/ano/AnoController.java → ...penenade/api/ano/PublicAnoController.java
@@ -2,31 +2,23 @@
 
 import java.util.Collection;
 import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.http.HttpStatus;
-import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.CrossOrigin;
-import org.springframework.web.bind.annotation.DeleteMapping;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.PathVariable;
-import org.springframework.web.bind.annotation.PostMapping;
-import org.springframework.web.bind.annotation.PutMapping;
-import org.springframework.web.bind.annotation.RequestBody;
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RestController;
 
 @RestController
 @CrossOrigin(origins = "*")
-@RequestMapping(path = AnoController.ENDPOINT)
-public class AnoController {
+@RequestMapping(path = PublicAnoController.ENDPOINT)
+public class PublicAnoController {
 
     public static final String ENDPOINT = "/anos";
-
     public static final String ANO_ID = "ano";
 
     @Autowired
     private AnoService service;
 
-
     @GetMapping
     public Collection<Ano> getAnos() {
         return service.getAllAnos();
@@ -37,19 +29,4 @@ public Ano getAno(@PathVariable(name = ANO_ID) Integer ano) {
         return service.getAno(ano);
     }
 
-    @PostMapping
-    public Ano saveAno(@RequestBody Ano ano) {
-        return this.service.addAno(ano);
-    }
-
-    @PutMapping
-    public Ano updateAno(@RequestBody Ano ano) {
-        return this.service.updateAno(ano);
-    }
-
-    @DeleteMapping(path = "/{" + ANO_ID + "}")
-    public ResponseEntity<String> updateAno(@PathVariable(name = ANO_ID) Integer ano) {
-        this.service.deleteAno(ano);
-        return AnoUtils.getResponseEntity(HttpStatus.OK, "");
-    }
 }
diff --git a/src/main/java/br/com/openenade/api/stub_data/StubDataCreator.java b/src/main/java/br/com/openenade/api/stub_data/StubDataCreator.java
@@ -128,8 +128,7 @@ private Ano[] addAnos() {
 
         Ano[] anos = new Ano[anosInt.length];
         for (int i = 0; i < anos.length; i++) {
-            anos[i] = new Ano();
-            anos[i].setAno(anosInt[i]);
+            anos[i] = new Ano(anosInt[i]);
             this.anoRepository.save(anos[i]);
         }
 

diff --git a/src/test/java/br/com/openenade/api/BaseUnitTest.java b/src/test/java/br/com/openenade/api/BaseUnitTest.java
@@ -21,6 +21,8 @@
 
 public class BaseUnitTest {
 
+    public static final String BASE_URI = "/api";
+
     @Autowired
     private AnoService anoService;
 

diff --git a/src/test/java/br/com/openenade/api/ano/AnoUnityTests.java b/src/test/java/br/com/openenade/api/ano/AnoUnityTests.java
@@ -22,9 +22,7 @@ public class AnoUnityTests extends BaseUnitTest {
 
     @Test
     public void addAnoTest() {
-        Ano ano = new Ano();
-
-        ano.setAno(2016);
+        Ano ano = new Ano(2016);
 
         this.service.addAno(ano);
         assertEquals(ano, service.getAno(2016));
@@ -33,11 +31,8 @@ public void addAnoTest() {
 
     @Test
     public void addAnosTests() {
-        Ano ano1 = new Ano();
-        Ano ano2 = new Ano();
-
-        ano1.setAno(2016);
-        ano2.setAno(2017);
+        Ano ano1 = new Ano(2016);
+        Ano ano2 = new Ano(2017);
 
         List<Ano> list = new ArrayList<>();
 
@@ -52,9 +47,7 @@ public void addAnosTests() {
 
     @Test(expected = ResourceNotFound.class)
     public void deleteAnoTest() {
-        Ano ano1 = new Ano();
-
-        ano1.setAno(2016);
+        Ano ano1 = new Ano(2016);
 
         this.service.addAno(ano1);
 

diff --git a/src/test/java/br/com/openenade/api/nota/NotaControllerTests.java b/src/test/java/br/com/openenade/api/nota/NotaControllerTests.java
@@ -47,8 +47,7 @@ public class NotaControllerTests extends BaseUnitTest {
 
     @Test
     public void postTestBasic() throws Exception {
-        Ano ano = new Ano();
-        ano.setAno(2018);
+        Ano ano = new Ano(2018);
         Regiao regiao = new Regiao("NO");
         Estado estado = new Estado("XD", regiao);
         Municipio municipio = new Municipio(123L, estado, "Capoeira Grande");
@@ -79,8 +78,7 @@ public void postTestBasic() throws Exception {
 
     @Test
     public void getTestBasicPlusDelete() throws Exception {
-        Ano ano = new Ano();
-        ano.setAno(2018);
+        Ano ano = new Ano(2018);
         Regiao regiao = new Regiao("NE");
         Estado estado = new Estado("GO", regiao);
         Municipio municipio = new Municipio(123L, estado, "Poeira Grande");

diff --git a/src/test/java/br/com/openenade/api/nota/NotaServiceTests.java b/src/test/java/br/com/openenade/api/nota/NotaServiceTests.java
@@ -40,8 +40,7 @@ public void setUp() {
     }
 
     public List<Nota> addAnos() {
-        Ano ano1 = new Ano();
-        ano1.setAno(2017);
+        Ano ano1 = new Ano(2017);
         Regiao regiao1 = new Regiao("NE");
         Estado estado1 = new Estado("PE", regiao1);
         Municipio municipio1 = new Municipio(333L, estado1, "Recife");
@@ -64,8 +63,7 @@ public List<Nota> addAnos() {
         nota1.getAvaliacao().setEnadeFaixa(3);
 
 
-        Ano ano2 = new Ano();
-        ano2.setAno(2018);
+        Ano ano2 = new Ano(2018);
         Regiao regiao2 = new Regiao("CO");
         Estado estado2 = new Estado("MG", regiao2);
         Municipio municipio2 = new Municipio(222L, estado2, "Minas Gerais");
@@ -86,8 +84,7 @@ public List<Nota> addAnos() {
         nota2.getAvaliacao().setEnadeContinuo(3.551);
         nota2.getAvaliacao().setEnadeFaixa(4);
 
-        Ano ano = new Ano();
-        ano.setAno(2019);
+        Ano ano = new Ano(2019);
         Regiao regiao = new Regiao("NE");
         Estado estado = new Estado("PB", regiao);
         Municipio municipio = new Municipio(123L, estado, "Campina Grande");
@@ -123,8 +120,7 @@ public List<Nota> addAnos() {
 
     @Test
     public void save() {
-        Ano ano = new Ano();
-        ano.setAno(2018);
+        Ano ano = new Ano(2018);
         Regiao regiao = new Regiao("NO");
         Estado estado = new Estado("XD", regiao);
         Municipio municipio = new Municipio(123L, estado, "Capoeira Grande");
@@ -153,8 +149,7 @@ public void save() {
     }
 
     public void saveUnivDuplicated() {
-        Ano ano = new Ano();
-        ano.setAno(2018);
+        Ano ano = new Ano(2018);
         Regiao regiao = new Regiao("NO");
         Estado estado = new Estado("XD", regiao);
         Municipio municipio = new Municipio(123L, estado, "Capoeira Grande");
@@ -194,8 +189,7 @@ public void saveUnivDuplicated() {
 
     @Test
     public void getAll() {
-        Ano ano = new Ano();
-        ano.setAno(2017);
+        Ano ano = new Ano(2017);
         Regiao regiao = new Regiao("NE");
         Estado estado = new Estado("PB", regiao);
         Municipio municipio = new Municipio(123L, estado, "Campina Grande");
@@ -210,8 +204,7 @@ public void getAll() {
 
         nota3 = this.notaService.save(nota3);
 
-        ano = new Ano();
-        ano.setAno(2017);
+        ano = new Ano(2017);
         regiao = new Regiao("N");
         estado = new Estado("AM", regiao);
         municipio = new Municipio(333L, estado, "Leruado");
@@ -232,8 +225,7 @@ public void getAll() {
 
     @Test
     public void getById() {
-        Ano ano = new Ano();
-        ano.setAno(2049);
+        Ano ano = new Ano(2049);
         Regiao regiao = new Regiao("C");
         Estado estado = new Estado("Ancapistão", regiao);
         Municipio municipio = new Municipio(123L, estado, "Paulo Kogos");
@@ -264,8 +256,7 @@ public void getById() {
 
     @Test
     public void deleteById() {
-        Ano ano = new Ano();
-        ano.setAno(2019);
+        Ano ano = new Ano(2019);
         Regiao regiao = new Regiao("NE");
         Estado estado = new Estado("PE", regiao);
         Municipio municipio = new Municipio(123L, estado, "Campina Grande");
@@ -390,9 +381,7 @@ public void testFilterByAnoInterval() {
 
     @Test
     public void caso1() {
-
-        Ano ano = new Ano();
-        ano.setAno(2018);
+        Ano ano = new Ano(2018);
         Regiao regiao = new Regiao("NO");
         Estado estado = new Estado("XD", regiao);
         Municipio municipio = new Municipio(123L, estado, "Capoeira Grande");
@@ -419,9 +408,7 @@ public void caso1() {
 
     @Test
     public void caso2() {
-
-        Ano ano1 = new Ano();
-        ano1.setAno(2017);
+        Ano ano1 = new Ano(2017);
         Regiao regiao1 = new Regiao("NE");
         Estado estado1 = new Estado("PE", regiao1);
         Municipio municipio1 = new Municipio(333L, estado1, "Recife");