question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Refactor the `stryker-cli` class

See original GitHub issue

The stryker-cli is untestable at the moment. See https://github.com/stryker-mutator/stryker/blob/master/packages/stryker/src/stryker-cli.ts. Simply require-ing the file is enough to start mutation testing. I would rather see an implementation with a class.

So instead of require('../src/stryker-cli) in the stryker binary i would like to see something like const StrykerCli = require('../src/StrykerCli'); new StrykerCli(process.argv).run().

Next we should create some unit tests for the cli.

Issue Analytics

  • State:closed
  • Created 6 years ago
  • Comments:5 (5 by maintainers)

github_iconTop GitHub Comments

1reaction
simondelcommented, Oct 2, 2017

Hi! Thanks for wanting to contribute 😃 I think you’re trying to push to our repository. That is the output of the command git remote get-url origin is it this repository?

To contribute, you can fork this repository (top right of the github page). That creates a copy of stryker on your profile at https://github.com/bicep/stryker. You can then push to that repository can create a PR from there. You can use the command git remote set-url origin https://github.com/bicep/stryker to change the url of the remote named origin.

If you need any help. let us know! Also, have you signed up for hacktoberfest?

0reactions
simondelcommented, Oct 18, 2017

Fixed with #426 🎉

Read more comments on GitHub >

github_iconTop Results From Across the Web

Refactor main stryker classes · Issue #397
We now have the following data structure in the main stryker package: | ConfigReader.ts | ConfigValidator.ts | FileStatements.ts | globals.ts ...
Read more >
Interactive Refactoring Course: Dive Into ...
The course covers 21 smells of bad code and 66 refactoring techniques to fix them. Each chapter includes examples in Java, C# and...
Read more >
@stryker-mutator/core | Yarn - Package Manager
It exports 2 classes for you to use: Stryker and StrykerCli . import { Stryker, StrykerCli } from '@stryker-mutator/core';. Both classes can be...
Read more >
stryker-cli
Command-line interface (CLI) for Stryker, the JavaScript mutation testing framework. Latest version: 1.0.2, last published: 2 years ago.
Read more >
Fun with Refactoring
Let's look at the method called in the constructor. We have a dynamic property on the class - which is being dropped in...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found