search

rinfo / rdl

Rättsdatalagret (dokumentation se http://dev.lagrummet.se/dokumentation/)
BSD 2-Clause "Simplified" License
4 stars 2 forks source link

Stabilisera regressionstester #125

Closed kamidev closed 10 years ago

kamidev commented 10 years ago

Ta bort onödiga konstruktioner från "inspelade" tester. Gör testernas funktion mer explicit. Gör gärna även output till terminal tydligare.

kamidev commented 10 years ago

Se även issue #123

cheijken commented 10 years ago

Fixat regressionstester för checker. Finns några anmärkningar: samma id på flera ställen, samt " som omger text på ett obra sätt. Det är lite svårt att testa på ett riktigt bra sätt. Skulle vara bra att det som genererar koden/html:en skulle ha lite tydligare testmöjligheter.

cheijken commented 10 years ago

Uppenbarligen fungerar testerna. HIttade ett fel på checker.regression.lagrummet.se Den godkänner alla "checker". Förmodligen är det för att den inte har en korrekt admin feed kopplad till sig. Fixar.

cheijken commented 10 years ago

Fixat regressionstesterna. Nu ser allt ok ut.

Förmodligen behövs det en genomgång hur vissa atom feeds buggs upp. Det finns en diskrepans mellan angivna url och hur de slås upp av de olika systemen. I vissa fall stod det bara .../rdf men det korrekta var .../rdf.rdf eller .../xhtml -> korrekt .../xhtml.html. Jag har justerat "v1.regression.testfeed.lagrummet.se" admin feed. Känns som en teknisk skuld just nu...

kamidev commented 10 years ago

Jag går igenom ändringarna nu. Ser bra ut. Ja, låt oss tala mer om det här med vilka datakällor vi använder och hur/var det anges. Vi har en issue för detta, #123.

kamidev commented 10 years ago

Se även kommentar för issue #120.