search

dbwebb-se / mvc

Course repo for the mvc and object orientation in frameworks course - mvc.
Other
1 stars 4 forks source link

laravel scrutinizer #36

Open joki20 opened 3 years ago

joki20 commented 3 years ago

Jag får inte till badges coverage och build. Får felmeddelanden rörande databasen. Min scrutinizer-fil är som i exemplet. Vad bör jag göra?

https://scrutinizer-ci.com/g/joki20/dbwebb-mvc-ci/inspections/9c8b8236-fd7a-4f98-a7c5-9f4beae6d6e2

image

image

image

mosbth commented 3 years ago

Undvik tester mot databasen i Scrutinizer och i Travis, det gör det mer avancerat.

joki20 commented 3 years ago

Jag ska alltså exkludera Yatzy-klassen i testet, eller iaf den del som innehåller anropet till databasen?

mosbth commented 3 years ago

Delet du får är från kommandot php artisan migrate och det har du lagt i din scrutinizer.yaml fil. Det kommandot är relaterat till databasen.

Att köra enhetstester mot databasen är överkurs och något jag föreslår att man låter vara. Att köra databaser på scrutnizer och travis är överkurs. Vill du ändå göra detta så går det bra men du behöver lösa det på egen hand. Jag har ingen setup som visar hur det fungerar för de databas-alternativ vi har i kursen.

Annars får du ta bort de sakerna som har med databasen att göra, så dina tester går igenom på travis/scrutnizer.

joki20 commented 3 years ago

Min scrutinizer.yml-fil ser ut såhär, finns inget php artisan migrate där

image

Provar att kommentera bort anropen till databasen

mosbth commented 3 years ago

Bygget du länkade till visar en annan konfigurationsfil. https://scrutinizer-ci.com/g/joki20/dbwebb-mvc-ci/inspections/9c8b8236-fd7a-4f98-a7c5-9f4beae6d6e2/configs

Så har du en annan lokalt så verkar något vara udda.

joki20 commented 3 years ago

scrutinizer-filen för ovan var för senaste (jag har försökt gå tillbaka till scrutinizer-exemplet https://scrutinizer-ci.com/g/joki20/dbwebb-mvc-ci/inspections/7f8cc933-df0d-449d-8c72-14e72ebf750c

Nu försökte jag med att kommentera bort databas-anropet i Yatzy-klassen men får samma problem https://scrutinizer-ci.com/g/joki20/dbwebb-mvc-ci/inspections/b654d73b-961d-4a87-84dc-993eebbecc8a

Det fungerade inte heller att ta bort include för config-mappen i phpunit.xml https://scrutinizer-ci.com/g/joki20/dbwebb-mvc-ci/inspections/2cb226d6-255f-4708-8ae4-c761f9454dc9

aigr20 commented 3 years ago

Jag behövde lägga till följande under build i .scrutinizer.yml för att få den att hoppa över det steget

project_setup:
        override:
            - true
joki20 commented 3 years ago

Jag gjorde så med nu, och får då ett annat fel. Scrutinizer kollar i vendor-katalogen. Men jag har specifikt sagt att den INTE ska göra det i scrutinizer.yml-filen

image

image

beha20 commented 3 years ago

Löste du problemet? joki20

beha20 commented 3 years ago

image Jag har likadant problem :/

mosbth commented 3 years ago

I #39 lyckades lösningen komma utifrån följande checklista.

Generellt gäller att man får felsöka steg för steg.

  1. Kör testerna lokalt make phpunit. kolla kodtäckningen. kolla om clover-filen genereras.
  2. Dubbelkolla att scrutinizer.yaml innehåller samma setup.
  3. Bygg om på scrutniizer.
  4. Övervaka när scrutinizer bygger och se om allt ser bra ut. Eventuellt kolla loggen från bygget.
  5. Avancerat, kolla om man kan koppla sig med ssh till scrutinizer och köra bygget för hand, i dess riktiga miljö.

Försök tills ni ger upp. Ger ni upp så skriv i redovisningstexten och berätta vad ni gjorde, hur ni gick tillväga och hur långt ni kom.