Kåre Fiedler Christiansen

Make new database connection for each statement
Make new database connection for each statement
Okay, de hedder heller ikke coldfolder i koden alle steder. coldfolder hedder faktisk "processedFiles" i den metode der kaldes. Jeg tror det ville være en fordel at bruge én terminologi, men i hver...

Okay, de hedder heller ikke coldfolder i koden alle steder. coldfolder hedder faktisk "processedFiles" i den metode der kaldes. Jeg tror det ville være en fordel at bruge én terminologi, men i hvert fald at benytte den samme i testen som i koden

Perhaps it should be made explicit that this is not a test-dependency? Wither by a comment or setting the scope explicitly?

Perhaps it should be made explicit that this is not a test-dependency? Wither by a comment or setting the scope explicitly?

Hvorfor ikke kalde dem "hotfolder" og "coldfolder" så de svarer til resten af koden?

Hvorfor ikke kalde dem "hotfolder" og "coldfolder" så de svarer til resten af koden?

Doc?

Doc?

Det ville give mere mening at logge diff'en på info hvis der er ændringer. Det her ligner noget der højst er på debug-niveau.

Det ville give mere mening at logge diff'en på info hvis der er ændringer. Det her ligner noget der højst er på debug-niveau.

Jeg forklarer gerne hvad tingene gør, men jeg tror det kræver at jeg står ved en computer

Jeg forklarer gerne hvad tingene gør, men jeg tror det kræver at jeg står ved en computer

Der er en ekstra ting: Collection skal for alle objekter være DPA_Collection, ikke Newspaper_Collectiion

Der er en ekstra ting: Collection skal for alle objekter være DPA_Collection, ikke Newspaper_Collectiion

Undskyld. Det var ikke min mening at frigive reviewet før jeg havde snakket med jer. Mads Analyse er korrekt. For at det virker skal DOMS-vagrant'en selvfølgelig afhænge af et snapshot der inkluder...

Undskyld. Det var ikke min mening at frigive reviewet før jeg havde snakket med jer.
Mads Analyse er korrekt.
For at det virker skal DOMS-vagrant'en selvfølgelig afhænge af et snapshot der inkluderer dette kode

Make DPA Datamodel
Make DPA Datamodel
Jeg savner i hele denne klasse en kort beskrivelse af hvilke dele af forretningslogikken der bliver dækket af hvilke tests

Jeg savner i hele denne klasse en kort beskrivelse af hvilke dele af forretningslogikken der bliver dækket af hvilke tests

Jeg savner unit-tests på egentligt unit-niveau. Ideelt set har en unit test egenskaber, så den tester netop én metode, mocker alt andet, og ikke tester effekter af ting der sker i andre metoder. Je...

Jeg savner unit-tests på egentligt unit-niveau.
Ideelt set har en unit test egenskaber, så den tester netop én metode, mocker alt andet, og ikke tester effekter af ting der sker i andre metoder.
Jeg vil gerne anbefale at du læser
https://sbprojects.statsbiblioteket.dk/stash/projects/BEVIS/repos/unittesting/browse/unittesting.pdf

Undgå _

Undgå _

Jeg går ud fra at det er checket at denne version svarer til den version af DOMS der er i drift med reklamefilmene

Jeg går ud fra at det er checket at denne version svarer til den version af DOMS der er i drift med reklamefilmene

Jeg går ud fra at det er checket at denne version svarer til den version af DOMS der er i drift med reklamefilmene

Jeg går ud fra at det er checket at denne version svarer til den version af DOMS der er i drift med reklamefilmene

Det virker ikke som en god idé at denne fil er inde i jar-filen på classpath

Det virker ikke som en god idé at denne fil er inde i jar-filen på classpath

Hov, der ser ud til at være noget galt med logikken her. Hvad hvis der er flere publisher-nodes, og det ikke er den første der er tv2d? Enten skal det check lægges ind i xpath-udtrykket, eller også...

Hov, der ser ud til at være noget galt med logikken her. Hvad hvis der er flere publisher-nodes, og det ikke er den første der er tv2d? Enten skal det check lægges ind i xpath-udtrykket, eller også skal alle nodes løbes igennem og checkes

Og igen, hvad hvis denne node ikke findes?

Og igen, hvad hvis denne node ikke findes?

Whoa!!!! Det overskriver en eventuel description der er der i forvejen, det er vist ikke forventet opførsel. Hvis der er tekst i i forvejen skal vi vel tilføje teksten, evt. efter et linjeskift

Whoa!!!! Det overskriver en eventuel description der er der i forvejen, det er vist ikke forventet opførsel. Hvis der er tekst i i forvejen skal vi vel tilføje teksten, evt. efter et linjeskift

Er dette godt nok? Alternativt kunne man jo fjerne hele denne nodes forælder?

Er dette godt nok? Alternativt kunne man jo fjerne hele denne nodes forælder?

Hvad hvis denne node slet ikke findes?

Hvad hvis denne node slet ikke findes?

Igen: Klassekommentar

Igen: Klassekommentar

Er denne fil kontrolleret nok til at det er okay ikke at lave input-validering?

Er denne fil kontrolleret nok til at det er okay ikke at lave input-validering?

Det virker som en dårlig løsning at bage uuid'erne ind i jar-filen. Det kræver et rerelease at køre programmet igen på andre input. Jeg foreslår at filen enten placeres et sted i filsystemet pr. ko...

Det virker som en dårlig løsning at bage uuid'erne ind i jar-filen. Det kræver et rerelease at køre programmet igen på andre input.
Jeg foreslår at filen enten placeres et sted i filsystemet pr. konvention (ligesom properties-filen) eller det angives i properties hvor den ligger

Her kommer jeg lige til kort i forhold til den konkrete opgave: Er det helt sikkert at AlternativeTitle også skulle flyttes på tv2-reklamefilmene?

Her kommer jeg lige til kort i forhold til den konkrete opgave: Er det helt sikkert at AlternativeTitle også skulle flyttes på tv2-reklamefilmene?