Add Curso private controller and Curso refactor

pom.xml

@@ -61,6 +61,12 @@
 			<scope>runtime</scope>
 		</dependency>
 
+		<dependency>
+			<groupId>com.google.code.gson</groupId>
+			<artifactId>gson</artifactId>
+		</dependency>
+
+
 	</dependencies>
 
 	<build>

src/main/java/br/com/openenade/api/categoriaadmin/CategoriaAdmin.java

@@ -1,22 +1,21 @@
 package br.com.openenade.api.categoriaadmin;
 
-
 import com.fasterxml.jackson.annotation.JsonValue;
 
 public enum CategoriaAdmin {
-	
-	PUBLICO("Publico"),PRIVADO("Privado");
-    
+
+    PUBLICO("Publico"), PRIVADO("Privado");
+
     private String value;
-    
+
     private CategoriaAdmin() {
-        
+
     }
-    
+
     private CategoriaAdmin(String value) {
         this.value = value;
     }
- 
+
     @JsonValue
     public String getValue() {
         return this.value;

src/main/java/br/com/openenade/api/categoriaadmin/CategoriaAdminController.java

@@ -12,13 +12,13 @@
 @CrossOrigin(origins = "*")
 @RequestMapping(path = CategoriaAdminController.ENDPOINT)
 public class CategoriaAdminController {
-    
+
     public static final String ENDPOINT = "/categorias-admin";
-    
+
     @RequestMapping(method = RequestMethod.GET)
     public @ResponseBody ResponseEntity<CategoriaAdmin[]> index() {
-        
+
         return new ResponseEntity<>(CategoriaAdmin.values(), HttpStatus.OK);
     }
-   
+
 }

src/main/java/br/com/openenade/api/curso/Curso.java

@@ -4,27 +4,24 @@
 import javax.persistence.Entity;
 import javax.persistence.EnumType;
 import javax.persistence.Enumerated;
-import javax.validation.constraints.NotNull;
 import javax.validation.constraints.Positive;
 import javax.validation.constraints.NotBlank;
 import br.com.openenade.api.modalidade.Modalidade;
 import javax.persistence.IdClass;
 
-
 @Entity
 @IdClass(CursoId.class)
 public class Curso {
 
-    @NotBlank(message = "'nome' não pode ser vazio.")
+    @NotBlank(message = "Curso 'nome' nao pode ser vazio")
     private String nome;
 
     @Id
-    @Positive(message = "'codigoArea' precisa ser positivo.")
+    @Positive(message = "Curso 'codigoArea' precisa ser positivo")
     private Long codigoArea;
 
     @Id
-    @Enumerated(EnumType.STRING)
-    @NotNull(message = "Você precisa especificar uma 'modalidade' de ensino.")
+    @Enumerated(EnumType.ORDINAL)
     private Modalidade modalidade;
 
     public Curso() {

src/main/java/br/com/openenade/api/curso/CursoService.java

@@ -4,32 +4,37 @@
 import java.util.Optional;
 import org.springframework.stereotype.Service;
 import org.springframework.beans.factory.annotation.Autowired;
+import br.com.openenade.api.exceptions.ResourceNotFound;
 import br.com.openenade.api.modalidade.Modalidade;
 
-
 @Service
 public class CursoService {
 
     @Autowired
     private CursoRepository repository;
 
-    public Curso save(Curso curso) {
+    public Curso addCurso(Curso curso) {
         return this.repository.save(curso);
     }
 
     public List<Curso> getAll() {
         return this.repository.findAll();
     }
 
-    public Optional<Curso> getByCodigo(Long codigo, Modalidade modalidade) {
+    public Curso getByCodigo(Long codigo, Modalidade modalidade) {
         CursoId cursoId = new CursoId(codigo, modalidade);
-        return this.repository.findById(cursoId);
-
+        Optional<Curso> optCurso = this.repository.findById(cursoId);
+        if (optCurso.isPresent()) {
+            return optCurso.get();
+        } else {
+            throw new ResourceNotFound("Cannot find Curso with Codigo [" + Long.toString(codigo)
+                    + "] and Modalidade [" + modalidade.getValue() + "]");
+        }
     }
-    
-    // unutilized
-    public void deleteCursoById(CursoId id) {
-        this.repository.deleteById(id);
+
+    public void deleteById(CursoId cursoId) {
+        this.getByCodigo(cursoId.getCodigoArea(), cursoId.getModalidade());
+        this.repository.deleteById(cursoId);
     }
 
 }

src/main/java/br/com/openenade/api/curso/PrivateCursoController.java

@@ -0,0 +1,34 @@
+package br.com.openenade.api.curso;
+
+import javax.validation.Valid;
+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.ResponseBody;
+import org.springframework.web.bind.annotation.RestController;
+import br.com.openenade.api.modalidade.ModalidadeService;
+
+@RestController
+@RequestMapping(path = PublicCursoController.ENDPOINT)
+public class PrivateCursoController {
+
+    @Autowired
+    private CursoService service;
+
+    @ResponseBody
+    @PostMapping
+    public Curso addCurso(@Valid @RequestBody Curso newCurso) {
+        return this.service.addCurso(newCurso);
+    }
+
+    @DeleteMapping(path = "/{codigo}/{modalidade}")
+    public void deleteCursoByCodigo(@PathVariable(name = "codigo") Long codigo,
+            @PathVariable(name = "modalidade") Integer modalidadeId) {
+        this.service
+                .deleteById(new CursoId(codigo, ModalidadeService.getModalidadeById(modalidadeId)));
+    }
+
+}

src/main/java/br/com/openenade/api/curso/PublicCursoController.java

@@ -0,0 +1,36 @@
+package br.com.openenade.api.curso;
+
+import java.util.List;
+import org.springframework.web.bind.annotation.GetMapping;
+import org.springframework.web.bind.annotation.CrossOrigin;
+import org.springframework.web.bind.annotation.PathVariable;
+import org.springframework.web.bind.annotation.ResponseBody;
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.web.bind.annotation.RequestMapping;
+import org.springframework.web.bind.annotation.RestController;
+import br.com.openenade.api.modalidade.ModalidadeService;
+
+@RestController
+@CrossOrigin("*")
+@RequestMapping(path = PublicCursoController.ENDPOINT)
+public class PublicCursoController {
+
+    public static final String ENDPOINT = "cursos";
+
+    @Autowired
+    private CursoService service;
+
+    @GetMapping
+    @ResponseBody
+    public List<Curso> getAll() {
+        return this.service.getAll();
+    }
+
+    @ResponseBody
+    @GetMapping(path = "/{codigo}/{modalidade}")
+    public Curso getCursoByCodigo(@PathVariable(name = "codigo") Long codigo,
+            @PathVariable(name = "modalidade") Integer modalidadeId) {
+        return this.service.getByCodigo(codigo, ModalidadeService.getModalidadeById(modalidadeId));
+    }
+
+}

src/main/java/br/com/openenade/api/modalidade/Modalidade.java

@@ -7,20 +7,23 @@
 public enum Modalidade {
 
     @Enumerated(EnumType.ORDINAL)
-    EDUCACAO_PRESENCIAL("Educação Presencial"), EDUCACAO_A_DISTANCIA("Educação à Distância");
+    EDUCACAO_PRESENCIAL("Educação Presencial", 0), EDUCACAO_A_DISTANCIA("Educação à Distância", 1);
 
     private String value;
+    private Integer id;
 
     private Modalidade() {
 
     }
 
-    private Modalidade(String value) {
+    private Modalidade(String value, Integer id) {
         this.value = value;
+        this.id = id;
     }
 
     @JsonValue
     public String getValue() {
-        return this.value;
+        return this.value + " - " + this.id;
     }
+
 }

src/main/java/br/com/openenade/api/modalidade/ModalidadeController.java

@@ -16,8 +16,8 @@ public class ModalidadeController {
     public static final String ENDPOINT = "/modalidades";
 
     @RequestMapping(method = RequestMethod.GET)
-    public @ResponseBody ResponseEntity<Modalidade[]> index() {
+    public @ResponseBody ResponseEntity<Modalidade[]> getAllModalidades() {
         return new ResponseEntity<>(Modalidade.values(), HttpStatus.OK);
     }
-
+    
 }

src/main/java/br/com/openenade/api/modalidade/ModalidadeService.java

@@ -0,0 +1,17 @@
+package br.com.openenade.api.modalidade;
+
+import br.com.openenade.api.exceptions.ResourceNotFound;
+
+public class ModalidadeService {
+
+    public static Modalidade getModalidadeById(Integer id) {
+        Modalidade[] modalidades = Modalidade.values();
+        if (id < 0 || id >= modalidades.length) {
+            throw new ResourceNotFound(
+                    "Cannot find Modalidade with Id [" + Integer.toString(id) + "]");
+        } else {
+            return modalidades[id];
+        }
+    }
+
+}

src/main/java/br/com/openenade/api/universidade/UniversidadeService.java

@@ -39,14 +39,9 @@ public Universidade save(Universidade universidade) {
     }
 
     public Universidade addCurso2Universidade(Universidade universidade, Curso curso) {
-        Optional<Curso> optCurso =
-                this.cursoService.getByCodigo(curso.getCodigoArea(), curso.getModalidade());
-        if (optCurso.isPresent()) {
-            universidade.addCurso(optCurso.get());
-            return this.repository.saveAndFlush(universidade);
-        } else {
-            throw new ResourceNotFound(curso.toString());
-        }
+        curso = this.cursoService.getByCodigo(curso.getCodigoArea(), curso.getModalidade());
+        universidade.addCurso(curso);
+        return this.repository.saveAndFlush(universidade);
     }
 
     public Collection<Universidade> getAll() {

src/test/java/br/com/openenade/api/BaseUnitTest.java

@@ -61,7 +61,7 @@ public void clean() {
         for (Curso curso : this.cursoService.getAll()) {
 
             CursoId id = new CursoId(curso.getCodigoArea(), curso.getModalidade());
-            this.cursoService.deleteCursoById(id);
+            this.cursoService.deleteById(id);
         }
         for (Ano ano : this.anoService.getAllAnos()) {
             this.anoService.deleteAno(ano.getAno());

src/test/java/br/com/openenade/api/curso/CursoServiceTests.java

@@ -23,7 +23,7 @@ public class CursoServiceTests extends BaseUnitTest {
 
     @Test
     public void testSaveBasic() {
-        this.cursoService.save(new Curso("CC", 13, Modalidade.EDUCACAO_PRESENCIAL));
+        this.cursoService.addCurso(new Curso("CC", 13, Modalidade.EDUCACAO_PRESENCIAL));
     }
 
     @Test
@@ -35,10 +35,10 @@ public void testGetAllBasic() {
 
         Curso e = new Curso("História", 13, Modalidade.EDUCACAO_A_DISTANCIA);
 
-        this.cursoService.save(a);
-        this.cursoService.save(b);
-        this.cursoService.save(c);
-        this.cursoService.save(d);
+        this.cursoService.addCurso(a);
+        this.cursoService.addCurso(b);
+        this.cursoService.addCurso(c);
+        this.cursoService.addCurso(d);
 
         List<Curso> cursos = this.cursoService.getAll();
 
@@ -63,13 +63,13 @@ public void testGetByCodigo() {
         Curso b = new Curso(cursos[1], 5, modalidade);
         Curso c = new Curso(cursos[2], 7, modalidade);
 
-        this.cursoService.save(a);
-        this.cursoService.save(b);
-        this.cursoService.save(c);
+        this.cursoService.addCurso(a);
+        this.cursoService.addCurso(b);
+        this.cursoService.addCurso(c);
 
-        Curso aa = this.cursoService.getByCodigo(3L, modalidade).get();
-        Curso bb = this.cursoService.getByCodigo(5L, modalidade).get();
-        Curso cc = this.cursoService.getByCodigo(7L, modalidade).get();
+        Curso aa = this.cursoService.getByCodigo(3L, modalidade);
+        Curso bb = this.cursoService.getByCodigo(5L, modalidade);
+        Curso cc = this.cursoService.getByCodigo(7L, modalidade);
 
         assertEquals(a, aa);
         assertEquals(b, bb);