Project

General

Profile

Enhancement #1679

Selenium tests til P2

Added by Kasper Hagel Madsen about 3 years ago. Updated 21 days ago.

Status:
Resolved (tag version)
Priority:
High
Assignee:
Estimated time:
URL med eksempel:
Kategorier:
Integration - Openlist (Personalisering)

Description

Der er lavet selenium tests i Behat til P2 (Personalisering 2) af Reload. Der skal laves code review af dem inden de skal med i release 02.2016.

History

#1 Updated by Rolf Madsen about 3 years ago

  • Status changed from New to Need more info
  • Assignee set to Kasper Hagel Madsen
  • Target version set to DDB CMS 2016 2. opgradering

Har vi et pull request?

#2 Updated by Gitte Barlach about 3 years ago

  • Kategorier Integration - Openlist (Personalisering) added

Fra Fini (ti 12-04-2016 11:32)

"Her: https://github.com/ding2/ding2/pull/216
Men jeg ved ikke hvor brugbart det er. Github syntes det er for stort.
Og der kommer nogen clashes med phpunit testene."

I følge Jesper K. giver det dog bedst mening at vente med review af testene til selve P2 koden er review'et og kommet med ind.

#3 Updated by Gitte Barlach about 3 years ago

  • Assignee changed from Kasper Hagel Madsen to Jesper Kristensen
  • Priority changed from Normal to High

#4 Updated by Rolf Madsen almost 3 years ago

  • Target version changed from DDB CMS 2016 2. opgradering to DDB CMS 2017 1. opgradering (7.x-4.0.2)

#5 Updated by Gitte Barlach almost 3 years ago

  • Status changed from Need more info to Ready for development
  • Assignee changed from Jesper Kristensen to Thomas Hansen
  • Priority changed from High to Urgent

Hej Thomas

Da P2 nu er kommet med i core, kan du så rebase denne: https://github.com/ding2/ding2/pull/216, så den kun indeholder tests og ikke hele p2 + tests?

På forhånd tak !

#6 Updated by Thomas Hansen almost 3 years ago

Kan ikke rebase, der kommer for mange conflicts, og jeg går på ferie på fredag og har andre ting jeg skal nå.

Jeg har lavet et PR hvor jeg har merget master ind istedet.

#7 Updated by Jesper Kristensen almost 3 years ago

  • Status changed from Ready for development to Reviewed - Needs info/rework

Har skrevet et par mindre kommentar, som lige skal ses på.

#8 Updated by Jesper Kristensen almost 3 years ago

Link til det nye PR https://github.com/ding2/ding2/pull/268 fra Reload.

@Thomas har du nogle ide omkring grunden til at stor set alle behat test fejler (https://circleci.com/gh/reload/ding2/277) og en del bliver "skipped".

Kan ikke merege det ind da, det vil få Circle CI til at fejle for alle fremtidige PR's!

#9 Updated by Rolf Madsen over 2 years ago

  • Target version changed from DDB CMS 2017 1. opgradering (7.x-4.0.2) to DDB CMS 2017 1. opgradering (Reload sprintbacklog)

#10 Updated by Thomas Hansen over 2 years ago

@Jesper
Lader til at det er fordi de danske oversættelser til P2 ikke er bliver importeret i en default install. Testene leder efter de danske tekster.

Om der er andet kan jeg ikke lige sige, for det drukner i føromtalte problem.

Nogen skal sørge for at P2 er oversat.

#11 Updated by Rolf Madsen over 2 years ago

Vi afventer en udredning omkring oversættelserne fra B14, da det vi hidtil har fået er en sammenblanding af built-in interface og fields, som gør det til et større udredningsabrejde for os, med den manuelle håndtering af oversættelser vi benytter pt. indtil vi får etableret en Localization servier, som vi har bestilt etableret af DBC:

#12 Updated by Thomas Hansen over 2 years ago

Nå, men jeg prøver lige om jeg kan trylle noget...

#13 Updated by Thomas Hansen over 2 years ago

Ah, der lader til at være noget rod med oversættelsesfilen. Graver i det.

#14 Updated by Thomas Hansen over 2 years ago

Bare lige et heads up:
msgcat, msguniqe og de andre standard .po file tools vil ikke arbejde med den da.po der ligger på master lige nu...

Det kan godt være at Drupal importeren er lidt mere lax, men der er ingen garantier for at den faktisk forstår den rigtigt.

Jeg prøver at rette op på det, men det bliver så på test branchen, for den er merged og rebased for mange gange til at jeg tør at fikse det i en PR og merge det ind.

#15 Updated by Thomas Hansen over 2 years ago

https://github.com/ding2/ding2/pull/268 opdateret.

Scrutinizer kan ikke lide at jeg bruger dpm() til debugging i Connie.

Der er en failure tilbage i behat testene, og det er fordi der er en reel fejl. Når man tilføjer et materiale til en liste, skulle der komme en popup som bekræftelse. Det gør der ikke.

#16 Updated by Thomas Hansen over 2 years ago

Den sidste fejl vil blive fikset hvis den her bliver merged: https://github.com/ding2/ding2/pull/350

#17 Updated by Gitte Barlach over 2 years ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Thomas Hansen to Jørgen Nielsen

#18 Updated by Jørgen Nielsen over 2 years ago

Den sidste fejl skulle også være fikset ifm. http://platform.dandigbib.org/issues/1782 (er merget ind) - vil det være en idé, at merge den ind hér også?.
Hvordan vil det spille sammen med http://platform.dandigbib.org/issues/1751, og det seneste PR? - det kunne være rart, hvis vi havde grønne tests før vi mergede tests ind...

#19 Updated by Jørgen Nielsen over 2 years ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Jørgen Nielsen to Thomas Hansen

#20 Updated by Thomas Hansen over 2 years ago

  • Assignee changed from Thomas Hansen to Jørgen Nielsen

Nå, men der var så en hel del der var blevet ændret i som havde konsekvenser for testene.

Jeg har bl.a. opdateret den til at bruge Opensearch 4.2 fordi test versionen af 4.0.1 er forsvundet.

Men den er grøn nu.. MERGE! MERGE! MERGE!

#21 Updated by Jesper Kristensen over 2 years ago

  • Assignee changed from Jørgen Nielsen to Thomas Hansen

Der mangle issue nr. på de commite beskeder der er lavet, hvilket osse fremgår at GitCop.

Koden ser fin ud lige med undtagelse af (https://github.com/ding2/ding2/pull/268/commits/52cd23f4acf3ed6a086ccb0359ced9da2cd79200) som har vist har en debug statement.

#22 Updated by Gitte Barlach over 2 years ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Thomas Hansen to Jørgen Nielsen

Jørgen, har du tid at se på denne?

#23 Updated by Thomas Hansen over 2 years ago

Jeg har fjernet debug statement'en.

#24 Updated by Jørgen Nielsen over 2 years ago

  • Status changed from Needs code review to Reviewed
  • Assignee changed from Jørgen Nielsen to Gitte Barlach

lad os få den ind mens tests er grønne!
Jeg sætter til reviewet og godkendt

#25 Updated by Gitte Barlach over 2 years ago

  • Target version changed from DDB CMS 2017 1. opgradering (Reload sprintbacklog) to DDB CMS 2016 2. opgradering (Reload sprintbacklog)

Da vi alligevel skal have et nyt byg, kan vi nå at få denne med ind i 2016-2.

#26 Updated by Rolf Madsen over 1 year ago

  • Description updated (diff)
  • Assignee changed from Gitte Barlach to Carsten Feilberg
  • Priority changed from Urgent to None

@Carsten, har jeg ret i at dette issue kan lukkes efter oprettelsen af https://platform.dandigbib.org/issues/3147?

#27 Updated by Carsten Feilberg over 1 year ago

Rolf Madsen skrev:

@Carsten, har jeg ret i at dette issue kan lukkes efter oprettelsen af https://platform.dandigbib.org/issues/3147?

Så vidt jeg kan se så handler denne sag om et PR der er lukket og et der er merged. Så sagen bør vel også lukkes?

 

#28 Updated by Rolf Madsen over 1 year ago

  • Status changed from Reviewed to Resolved

#29 Updated by Rolf Madsen 8 months ago

  • Status changed from Resolved to Ready for development
  • Assignee deleted (Carsten Feilberg)
  • Priority changed from None to Normal
  • Target version changed from DDB CMS 2016 2. opgradering (Reload sprintbacklog) to Release 30 - BPI, Kampagneplus og Sektioner - (Reload)

#30 Updated by Rolf Madsen 7 months ago

  • Status changed from Ready for development to Needs code review
  • Assignee set to Gitte Barlach

Mangler afklaring af merge status for dette issue.

#31 Updated by Jesper Kristensen 4 months ago

  • Status changed from Needs code review to Reviewed

Skal lukkes...

#32 Updated by Rolf Madsen 4 months ago

  • Status changed from Reviewed to Closed

Hermed lukket.

#33 Updated by Rolf Madsen 3 months ago

  • Status changed from Closed to Ready for development
  • Assignee changed from Gitte Barlach to Christel Krabbenhøft

#34 Updated by Christel Krabbenhøft 3 months ago

  • Assignee changed from Christel Krabbenhøft to Rolf Madsen

#35 Updated by Rolf Madsen 3 months ago

  • Assignee changed from Rolf Madsen to Christel Krabbenhøft

#36 Updated by Christel Krabbenhøft 3 months ago

  • Assignee changed from Christel Krabbenhøft to Thomas Hansen
  • Target version changed from Release 30 - BPI, Kampagneplus og Sektioner - (Reload) to Release 31 - bugfixes

#37 Updated by Christel Krabbenhøft 3 months ago

Og nu genåbnet, så Fini kan fikse det sidste i PR'et!

#38 Updated by Thomas Hansen 3 months ago

  • Status changed from Ready for development to Needs code review

OK, det skal bare merges nu, for den branch er døden at arbejde med. P2 teste'ne kører ikke fordi de Behat tests der i mellemtiden er blevet merget ligger et andet sted, men jeg magter ikke at risikere at branchen endnu engang skal merges fordi master flytter sig igen mens jeg får det op at køre og fikser de tests der med største sansynlighed nu er broken.

 

#39 Updated by Thomas Hansen 3 months ago

OK, efter at have snakket med Kasper, så dropper vi mergen af det her og jeg får lavet et nyt PR hvor vi smider historikken fra det nuværende væk ag får testene til at køre og fixed.

#41 Updated by Gitte Barlach about 2 months ago

  • Assignee changed from Thomas Hansen to Jørgen Nielsen

#42 Updated by Gitte Barlach about 1 month ago

  • Priority changed from Normal to High

Relateret til #3931

#43 Updated by Gitte Barlach about 1 month ago

Hej Jørgen

Kan du code review´e denne? Det er vigtigt vi får den indhurtigst muligt. 

#44 Updated by Jørgen Nielsen about 1 month ago

  • Status changed from Needs code review to Reviewed
  • Assignee changed from Jørgen Nielsen to Gitte Barlach

Bemærk, at nogle tests bruger 'https://oss-services.dbc.dk/opensearch/4.5/' som er "deprecated" (men stadigvæk tilgængelig).
Nyeste version er: 'https://oss-services.dbc.dk/opensearch/5.0/'
 

Men ellers: reviewet og godkendt

#45 Updated by Gitte Barlach about 1 month ago

men - skal det så ikke rettes ? det giver vel ikke mening at bruge en forældet version af Open Search?

#46 Updated by Jørgen Nielsen about 1 month ago

Du bestemmer :-)
Vil du ha' det ind i en fart, så er der ikke noget der knækker. Vil du teste mod version 5.0, så send den retur til Thomas

#47 Updated by Thomas Hansen about 1 month ago

Jeg har rettet de sidste referencer til pre-5.0 versioner af opensearch, så hvis circle stadig går i grøn, så er den good to merge.

#48 Updated by Gitte Barlach about 1 month ago

tusind tak, Thomas.

#49 Updated by Gitte Barlach 30 days ago

  • Target version changed from Release 31 - bugfixes to Release 30-1 - Place2Book

vi fremskynder denne, så vi ikke kommer ud i yderlligere rebases mv. 

#50 Updated by Kasper Garnæs 24 days ago

  • Status changed from Reviewed to Technical test

Merged.

#51 Updated by Kasper Garnæs 24 days ago

  • Status changed from Technical test to Development
  • Assignee changed from Gitte Barlach to Kasper Garnæs

Tests med login som biblioteksbruger fejler af urandsagelige årsager. Derfor er ændringen på kort sigt rullet tilbage da fejlende tests stopper releaseprocessen: https://github.com/ding2/ding2/pull/1416.

Jeg undersøger problematikken nærmere efterfølgende.

#52 Updated by Kasper Garnæs 24 days ago

  • Status changed from Development to Technical test
  • Assignee changed from Kasper Garnæs to Gitte Barlach

Ændringen er blevet genindført og fejler ikke længere i efterfølgende builds: https://circleci.com/workflow-run/315bf999-ff9b-4be5-aac3-935dcc441371.

#53 Updated by Gitte Barlach 21 days ago

  • Status changed from Technical test to Resolved (tag version)

Godkendt jvf. code review

Also available in: Atom PDF