Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
Mikael Roos
@mosbth
Ja, jag fick höra det. Jobbigt. Vi gjorde en issue för att komma ihåg det... dbwebb-se/ramverk1#4
Emil Magnusson
@mangepong
@mosbth Hej! Har lite problem med min modul och testerna. Om jag kör make test så säger den att den inte "The service 'cache' is not loaded in the DI-container." Vet inte riktigt varför den inte laddas in som det ska.
Mikael Roos
@mosbth

@mangepong Normalt så laddas inte $di i testerna, man behöver göra det. Eller så missar man global $di;.

Jag gjorde några exempelprogram som förtydligade detta, hur man laddade tjänsterna i test-programmen, typ:
https://github.com/dbwebb-se/ramverk1/blob/master/example/sample/c/test/Controller/CControllerTest.php#L18-L37

Emil Magnusson
@mangepong
@mosbth hm okej kollade på det du skickade och testade en utav exempel testerna som hade samma kod men får samma fel på rad 32.
https://gyazo.com/d0029d2e35aafafc0b8c52292959b955
Mikael Roos
@mosbth
Kanske säger exceptionet något mer?
Kikar vi på sökvägen på rad 32, finns den katalogen hos dig?
Emil Magnusson
@mangepong
Aha okej nej det har jag inte. Hur får jag med cache mappen till modulen?
Mikael Roos
@mosbth

Du menar "hur skapar jag den katalogen?"

# Stå i roten av repot
mkdir test/cache

Det motsvarar cache/ katalogen som används när du accessar htdocs/ via webbservern (ägs av webbservern). Pga krockande rättigheter så väljer vi att ha ett separat cache-dir för enhetstesterna (ägs av dig).

Emil Magnusson
@mangepong
Hmm, testade att göra en cache folder i test så som det står men felet kvarstår https://gyazo.com/ac32f777a71304e8d47ec0d8433cc35f
Mikael Roos
@mosbth
Felsök.
Skriv ut alla tjänster i $di innan du kör testet, i metoden för ditt testfall (finns cahche med?)
Om ja, gå in i kontrollerns metod, gör samma sak (borde vara samma tjänster som visas).
Emil Magnusson
@mangepong
@mosbth Lyckades att få testerna att funka osv men när jag kör make test så får jag detta istället nu: https://gyazo.com/3ff776972ef623f11a74bbf99b253ae6
Mikael Roos
@mosbth
@mangepong Om du enbart vill leka med enhetstesterna så kan du köra make phpunit och slippa felet.
Annars behöver vi felsöka, börja med make install och se om det löser saken.
Annars behöver vi en version från phploc, ditt os och din version av php för att klura ut vad som händer, kanske leta lite bland github issues för phploc till att börja med.
Emil Magnusson
@mangepong
Testade att installera om phploc men det verkar inte funka. Har nu phploc version 7.0.1, php version 7.2.10 och jag kör windows 10. Får även detta felet på travis när jag försöker bygga testerna.
Make install gjorde heller inget.
Mikael Roos
@mosbth
2020-08-31_11:47:54_448x125.png
Jag gjorde en ren installation och works for me. Jag kör Debian och PHP 7.4.
Emil Magnusson
@mangepong
Gjorde du en ren installation av php då?
Emil Magnusson
@mangepong
@mosbth Lyckades få det att funka lokalt på datorn nu men när jag pushar det till github och gör en build på travis får jag fortfarande samma problem med phploc.
Verkar som att om jag har php 7.3 lokalt så funkar det men när jag ändrar det i composer.json att den ska ha version 7.3 så klagar travis.
Mikael Roos
@mosbth
Länk till misslyckat build på travis?
den andra är det phploc som strular och den första är när jag testade php 7.3
Mikael Roos
@mosbth
Ge mig din studakronym så hittar jag felet (last famous words)
Emil Magnusson
@mangepong
hahaha det är emmu18
Mikael Roos
@mosbth
Nu börjar nedräkningen...
Mikael Roos
@mosbth
Hmmm. Okey, när jag kör i php72 så kan jag återskapa ditt fel. Dock hittar jag ingen issue om den på phplocs github.
Emil Magnusson
@mangepong
Nej precis hittar inte heller det.
Mikael Roos
@mosbth
När jag kör i PHP 7.3 så funkar phploc. Om du vill testa i php7.3 så behöver du uppdatera travis till att köra i den versionen (.travis.yml)
Emil Magnusson
@mangepong
Yes ska testa det!
Emil Magnusson
@mangepong
Det löste det tack för hjälpen!
Mikael Roos
@mosbth
Jag höll precis på att skriva en issue om det, men jag vill gärna undvika det onda ögat från Sebastian så jag trippelkollade och den versionen som du använder (18 dagar gammal -ish) kräver php7.3. Så det är relativt nytt.
https://github.com/sebastianbergmann/phploc/blob/7.0.1/composer.json#L24
Emil Magnusson
@mangepong
Aa precis därför jag testade att köra 7.3
Mikael Roos
@mosbth
Gott!
Jag kunde inte hitta kraven på php-versionen i readme-filen, men composer.json är en bra plats att kolla.