changeset 21:43360bf09b1a

unificando criterio de parĂ¡metros en Agenda
author nelo@MTEySS.neluz.int
date Mon, 14 Mar 2011 21:05:19 -0300
parents c5a99dde072f
children d6e124e5c9c4
files Agendas/trunk/src/Agendas.Domain/Agenda.cs Agendas/trunk/src/Agendas.Tests/PropuestasTests.cs Agendas/trunk/src/Agendas.Tests/PulicarTests.cs
diffstat 3 files changed, 12 insertions(+), 18 deletions(-) [+]
line wrap: on
line diff
--- a/Agendas/trunk/src/Agendas.Domain/Agenda.cs	Mon Mar 14 20:59:28 2011 -0300
+++ b/Agendas/trunk/src/Agendas.Domain/Agenda.cs	Mon Mar 14 21:05:19 2011 -0300
@@ -38,14 +38,17 @@
 			_eventosRepository.Save(evento);
 		}
 
-		public void Recordar(Evento evento)
+		public void Recordar(Guid eventoId)
 		{
+			var evento = _eventosRepository.Get(eventoId);
 			if (_recordador != null)
 				_recordador.Recordar(evento);
 		}
 
-		public void Proponer(Evento evento)
+		public void Proponer(string titulo)
 		{
+			var evento = new Evento {Titulo = titulo};
+
 			if (string.IsNullOrWhiteSpace(evento.Titulo))
 				throw new ValidationException();
             if (NoEstaAutenticado(_seguridad))
--- a/Agendas/trunk/src/Agendas.Tests/PropuestasTests.cs	Mon Mar 14 20:59:28 2011 -0300
+++ b/Agendas/trunk/src/Agendas.Tests/PropuestasTests.cs	Mon Mar 14 21:05:19 2011 -0300
@@ -18,13 +18,11 @@
 		{
 			var repository = new EventoRepository();
 
-			var van = new Evento { Titulo = "Van propuesta" };
-
             var seguridad = new Mock<ISeguridad>();
             var agenda = new Agenda(null, null, seguridad.Object, repository);
             
             seguridad.Setup(s => s.GetPrincipal()).Returns(SeguridadObjectMother.GetGenericPrincipalAutenticadoSinRoles());
-			agenda.Proponer(van);
+			agenda.Proponer("Van propuesta");
 
 			IList<Evento> eventosPropuestos = agenda.GetEventosPropuestos();
 			Assert.IsNotNull(eventosPropuestos);
@@ -35,14 +33,11 @@
         [Test]
         public void Verificar_propuesta_separada_de_publicacion()
         {
-			var seguridad = new Mock<ISeguridad>();
-            var agenda = new Agenda(null, null, seguridad.Object, new EventoRepository());
+            var agenda = new Agenda(null, null, SeguridadServiceDefault, new EventoRepository());
 
-            var vanPropuesta = new Evento { Titulo = "Van propuesta" };
             var vanPublicada = EventoObjectMother.GetVanValidaParaPublicar();
-            seguridad.Setup(s => s.GetPrincipal()).Returns(SeguridadObjectMother.GetGenericPrincipalAutenticadoSinRoles());
 
-            agenda.Proponer(vanPropuesta);
+			agenda.Proponer("Van propuesta");
             agenda.Publicar(vanPublicada.Titulo, vanPublicada.Ponente, vanPublicada.Fecha);
 
             IList<Evento> eventosPropuestos = agenda.GetEventosPropuestos();
@@ -56,13 +51,11 @@
 		public void Propuesta_de_van_sin_titulo()
 		{
 			var repository = new Mock<IEventoRepository>();
-			var van = new Evento();
 
             var seguridad = new Mock<ISeguridad>();
             var agenda = new Agenda(null, null, seguridad.Object, repository.Object);
             
-            Assert.Throws<ValidationException>(() => agenda.Proponer(van));
-			Assert.AreEqual(Guid.Empty, van.Id);
+            Assert.Throws<ValidationException>(() => agenda.Proponer(string.Empty));
 
 			repository.Verify(p => p.Save(It.IsAny<Evento>()), Times.Exactly(0));
 		}
@@ -78,8 +71,7 @@
             seguridad.Setup(s => s.GetPrincipal()).Returns(SeguridadObjectMother.GetGenericPrincipalAutenticadoSinRoles());
 
 			{
-				var van = new Evento {Titulo = "Van propuesta"};
-                agenda.Proponer(van);
+                agenda.Proponer("Van propuesta");
 			}
 
 			{
@@ -100,8 +92,7 @@
             seguridad.Setup(s => s.GetPrincipal()).Returns(SeguridadObjectMother.GetGenericPrincipalAutenticadoSinRoles());
 
 			{
-				var van = new Evento { Titulo = "Van propuesta" };
-                agenda.Proponer(van);
+				agenda.Proponer("Van propuesta");
 			}
 
 			{
--- a/Agendas/trunk/src/Agendas.Tests/PulicarTests.cs	Mon Mar 14 20:59:28 2011 -0300
+++ b/Agendas/trunk/src/Agendas.Tests/PulicarTests.cs	Mon Mar 14 21:05:19 2011 -0300
@@ -109,7 +109,7 @@
 			var van = EventoObjectMother.GetVanValidaParaPublicar();
 			agenda.Publicar(van.Titulo, van.Ponente, van.Fecha);
 
-			agenda.Recordar(van);
+			agenda.Recordar(van.Id);
 
 			publicador1.Verify(p => p.Publicar(It.IsAny<Evento>()), Times.Exactly(1));
 			publicador2.Verify(p => p.Publicar(It.IsAny<Evento>()), Times.Exactly(1));