Skip to content

laravel scrutinizer #36

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
joki20 opened this issue May 9, 2021 · 11 comments
Open

laravel scrutinizer #36

joki20 opened this issue May 9, 2021 · 11 comments
Assignees
Labels
question Further information is requested

Comments

@joki20
Copy link

joki20 commented May 9, 2021

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
Copy link
Member

mosbth commented May 10, 2021

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

@mosbth mosbth self-assigned this May 10, 2021
@mosbth mosbth added the question Further information is requested label May 10, 2021
@joki20
Copy link
Author

joki20 commented May 10, 2021

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

@mosbth
Copy link
Member

mosbth commented May 10, 2021

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
Copy link
Author

joki20 commented May 10, 2021

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
Copy link
Member

mosbth commented May 10, 2021

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
Copy link
Author

joki20 commented May 10, 2021

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
Copy link

aigr20 commented May 10, 2021

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
Copy link
Author

joki20 commented May 10, 2021

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
Copy link

beha20 commented May 12, 2021

Löste du problemet? joki20

@beha20
Copy link

beha20 commented May 12, 2021

image
Jag har likadant problem :/

@mosbth
Copy link
Member

mosbth commented May 17, 2021

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

4 participants