CvanderStoep / Genetic_Swarm

1 stars 0 forks source link

Change main methods to unit tests #1

Closed lexvanderstoep closed 5 years ago

lexvanderstoep commented 5 years ago

Some of your main methods are used to test functionality of the code. It might be worthwhile to change these to unit tests.

For example, the main method in the Individual class tests whether fitness works. This should be placed in a unit test

CvanderStoep commented 5 years ago

Deed jij een voorzet?

CvanderStoep commented 5 years ago

de test doet het; dankjewel. hoe run ik alle tests in de folder test? ik kan wel mazetest runnen, maar stel je voor dat ik meer Unit tests heb als ik op de folder test ga staan, zie ik wel het commando run test in test; maar dat doet niet veel de class Mazetest geeft een melding dat hij een package mist?

lexvanderstoep commented 5 years ago

Right-click op je module, en dan komt er een actie tevoorschijn genaamd “Run All Tests”

lexvanderstoep commented 5 years ago

Als je naar de MazeTest class gaat, naar de regel met de melding dat je een pakket mist, dan komt er zo’n lampje tevoorschijn. Als je daar op klikt kan je op import klikken volgens mij

CvanderStoep commented 5 years ago

ik kreeg 2 meldingen: bij junit.jupiter de melding dat ik iets moest doen; gedaan, gelukt bij class Mazetest een melding "missing package statement", volgens mij is dat omdat het in de folder test staat. Als je nu boven de class zet: package test, dan worden vervolgens als onderliggen classes weer niet herkent.

lexvanderstoep commented 5 years ago
image

Zo ziet mijn project er uit. De "test" folder is gemarkeerd als test folder (right-click op de folder > Mark As > Test root)

CvanderStoep commented 5 years ago

Die optie heb ik niet. Alleen mark directory as excluded.

Carlo

Op 14 mei 2019 om 12:49 heeft Lex van der Stoep notifications@github.com het volgende geschreven:

Zo ziet mijn project er uit. De "test" folder is gemarkeerd als test folder (right-click op de folder > Mark As > Test root)

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub, or mute the thread.

lexvanderstoep commented 5 years ago

Genetic_Swarm > right_click > Module Settings >:

image

Klik dan op de 'test' folder, and selecteer de test mark as

CvanderStoep commented 5 years ago

Hoera!

Carlo

Op 14 mei 2019 om 13:03 heeft Lex van der Stoep notifications@github.com het volgende geschreven:

Genetic_Swarm > right_click > Module Settings >:

Klik dan op de 'test' folder, and selecteer de test mark as

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub, or mute the thread.