Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Fill test explorer with mutations on startup #1
Fill test explorer with mutations on startup #1
Changes from 10 commits
585a34d
0261f81
38f93ad
08a3690
bb0c337
cfed034
1340bf3
2b646a2
4da8ca0
2ebd1ce
12fe482
9675b23
f45bd20
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kan dit niet een static function zijn? En moeten we hier uberhaupt een class van maken?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ja, klopt. Ik heb het nu aangepast.
Ik verwacht dat deze factory later uitgebreid wordt met methodes om te bepalen welk Stryker-platform aangeroepen moet worden. Om die reden had ik hem al aangemaakt, maar een klasse is nu niet direct nodig. Ik kan hem eventueel weghalen als je wilt, maar het was alvast een opzet voor later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Deze naam is nu wel heel generiek. Als we in de class alle functionaliteit van StrykerJS gaan zetten lijkt het mij een groot bestand te worden.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ik wil deze klasse enkel gebruiken voor de verschillende aanroepen van de executable, en vervolgens de resultaten daaruit afhandelen in andere klassen. Ik denk dat hij daarom niet groot zal worden. Ook kan ik nog functionaliteit afsplitsen naar de base class Platform, want ik verwacht dat het gros vergelijkbaar zal zijn tussen de Stryker-platformen.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ik heb nu ook het e.e.a. verplaatst naar de base abstract class
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ik wil voorstellen om dit net zo op te zetten als het spawen van een childprocess. Een string is lastiger te manipuleren dan een array (van argumenten).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes dat is beter! Ik heb het aangepast.
Ik gebruik nu ook spawn i.p.v. exec omdat dit een stuk sneller blijkt te zijn.
This file was deleted.