Guia Test Driven Development B3

B3: Guia TDD

Arregla cosetes de la pràctica anterior:

Refactorings Extract method: Extreieu, del mètode creaNovaObra de la classe AmalgamaOnline, un mètode helper cercaAutor que donat un id de autor retorni una referencia a l'autor o llenci l'excepció. Cridant-lo des de creaNovaObra. Tot hauria de seguir funcionant igual.

Implementa les conversions de format amb un fake

Fes fallar i passar el test testGeneratsPerDefecte_senseOriginals. Com no volem modificar el codi de test, per fer-ho fallar, creem un fitxer tonto quan afegim l'obra

Fes fallar i passar el test testGeneratsPerDefecte_ambUnOriginal de forma obvia: Creant els fitxers a partir dels noms literals que poden estar a un array. Tal com mostra el següent exemple:

	const char* fakeGenerats[] = {
		"generats/Un autor - Una obra [imprimible].pdf",
		"generats/Un autor - Una obra [marca d'aigua].pdf",
		"generats/Un autor - Una obra [multiples fitxers html].war",
		0 // Aquest zero es necessari per finalitzar el bucle
		};
	for (unsigned i = 0; fakeGenerats[i]; i++)
		std::ofstream newfile( fakeGenerats[i] ) ;

Un cop en green, ves substituint pas a pas, els camps i les parts comunes de l'string comprovant a cada pas que segueixes en green.
Refactor: Extreu aquest codi en un mètode helper privat converteixOriginal

Adapta la llibreria libOpenOffice2Html amb la classe adaptadora ConversorHtml. Ho farem guiats pels següents tests unitaris. (TestsConversorHtml.cxx, que heu de crear vosaltres)

testConverteix_generaUnFitxer:
Després de crear el fitxer original, quan fem:
```
	ConversorHtml conversor;
	creaFitxerOriginal("Original.odt", 99); // El metode helper l'heu de copiar d'un altre test
	conversor.converteix("originals/Original.odt","generats/Prefix");
	
```
es crea un fitxer anomenat "generats/Prefix [multiples fitxers html].war". Usa el LibFileSystem::llistaDirectoriOrdenadament per fer l'ASSERT. Passem a verd ràpid simplement creant el fitxer amb un std::ofstream. Després refactoritzem per usar els paràmetres que rep converteix
testConverteix_generaContingut:
El test és bàsicament el mateix que l'anterior però, en comptes de comprovar que el fitxer existeixi, fem servir LibFileSystem::contingutFitxer per comprovar que conte un contingut correcte. La nostra llibreria de mentida escriu això:
```
"War file generated from 'originals/Original.odt' with 99 pages\n"
```
Ho podeu escriure, primer insertant el literal i, després cridant a la llibreria libOpenOffice2Html.
testConverteix_ambOriginalInexistent:
Tractem el cas en que no poguem convertir perque l'original no existeix. La llibreria ens ho notifica retornant un -1, però, la nostra interfície vol rebre una excepció amb el missatge "No s'ha trobat el fitxer original".
testConverteix_cridatPolimorficament:
El ConversorHtml ha de derivar de Conversor que defineix la interfície comuna amb el ConversorPdf.
Per testejar-ho cal derivar un de l'altre i definir buit a Conversor el mètode converteix.
```
	ConversorHtml conversorHtml;
	Conversor & conversor = conversorHtml; // Variable polimorfica
	creaFitxerOriginal("Original.odt", 222);
	// cridara a un o l'altre segons si converteix esta o no definit virtual a la base i si la signatura es identica
	conversor.converteix("originals/Original.odt","generats/Prefix");
	ASSERT_EQUALS(...)
	
```
El test cascara o no depenen de si està definit converteix com a virtual o no.
Sempre que definiu una classe base amb funcions virtuals heu de definir-li un destructor virtual, encara que sigui buit!!!!
El compilador es queixa d'aixo amb un warning pero mes endavant, quan feu deletes de punters polimorfics, os podreu trobar amb segmentation faults.

Adapta la llibreria libOpenOffice2Pdf amb la classe adaptadora ConversorPdf. Els tests unitaris. (TestsConversorPdf.cxx) sòn anàlegs però és diferent la forma de fer-los fallar i passar-los. Coses que cal tenir en compte:

Els simbols d'aquesta llibreria estan dintre del namespace OpenOfficeTools
Cas de fallada: Cal interceptar l'excepció de la llibreria per llançar la nostra. Quan feu cascar el test de "fallada per original inexistent", en comptes de cascar amb Falla, MiniCppUnit cascarà amb una excepció no esperada. És una excepció propia de la llibreria. Per tal de passar aquest test, a l'adaptador, hem de fer un try/catch de l'excepció que llença la llibreria i, al codi de catch, llençar la nostra propia excepció.
Compte quan feu el catch: L'excepció de OpenOfficeTools no deriva de std::exception. Hem de fer el catch del tipus concret.
Marca d'aigua: El conversor PDF té dos modes depenent del qual es posa en mode marca d'aigua o imprimible. Tant el nom del fitxer generat (com hem vist als tests funcionals) com el contingut ha de ser diferent. Cal provar el conversor PDF en els dos modes. Per defecte està en mode 'imprimible' en que la marca d'aigua es "". A partir del moment que cridem a ConversorPdf::activaMarcaDAigua(string) les conversions es faran en l'altre mode.
Truc pel polimorfisme: Per fer fallar el test de polimorfisme sense que casqui també el de Html, podeu alterar la signatura del converteix perquè el compilador no el consideri una redefinició. Per exemple, si un dels paràmetres, en comptes de ser una referència constant, és un string per copia.

Cal crear la classe GrupDeConversors que gestioni una llista de conversors. (composició polimòrfica múltiple). Afegirem conversors al grup de la següent manera:

grupDeConversors.afegeix("html");

grupDeConversors.afegeix("pdf_imprimible"); // Sense marca d'aigua

grupDeConversors.afegeix("pdf_marcat"); // amb marca d'aigua

D'aquesta manera ocultem els tipus concrets de conversors als seus clients. El grup de conversors defineix el seu propi mètode converteix que crida recursivament als del seus agregats. Els testos unitaris (TestGrupDeConversors.cxx) que cal fer segueixen l'esquema de les altres composició múltiple que vam fer, però, en un punt cal introduir el polimorfisme.

testConverteix_ambUnConversorHtml
Composició de 1 ConversorHtml: Implementeu la delegació.
testConverteix_senseCapConversorNoFaRes
Composició de 0..1 ConversorHtml: Gestioneu l'opcionalitat.
testConverteix_ambUnConversorPdfImprimible
Composició poliformica de 0..1 Conversor: Feu-lo polimòrfic + construccio condicional al 'afegeix'.
testConverteix_ambUnConversorPdfAmbMarca
Composició poliformica de 0..1 Conversor: Afegiu un nou tipus
testConverteix_ambConversorsPdfIHtml
Composició poliformica de 0..* Conversors: Cardinalitat múltiple.
testConverteix_ambUnConversorExtrany
Si ens fan convertir 'doc', amb aquesta crida:
```
grupDeConversors.afegeix("doc");
```
que és un format que no suportem llencem una excepció amb el missatge "Format no suportat".

Un cop tenim aquesta infraestructura muntada podem fer un refactoring per substituir la implementació fake que tenim a la facade (AmalgamaOnline). Li afegim al facade un membre GrupDeConversors, i des del constructor del facade li afegim els formats que abans traduiem amb el fake, tot hauria de seguir funcionant.

Duplicar: Afegim un membre GrupDeConversors a AmalgamaOnline
Farcir: Desde el constructor, afegim tots els conversors que necessitem al grup de conversors segons la llista de sufixos que tenim a AmalgamaOnline::converteixOriginal().
Canviar l'us: En comptes de cridar a AmalgamaOnline::converteixOriginal() cridem a _grupDeConversors.converteix(original, prefix).
Netejar: Esborrem la implementacio converteixOriginal

Opcional: Passar el darrer test funcional: Fer que agafi els formats actius d'un fitxer de configuració passat com a paràmetre a un constructor alternatiu. Pista: Com les etiquetes de format son paraules (sense espais) aquest constructor alternatiu pot fer el parsing simplement amb:

	std::ifstream config(configFilename.c_str());
	std::string format;
	while (config >> format)
		// afegeix el format