Reporting to sonar qube about stryker-js HOT 6 CLOSED

I think we'll need to change our API to allow you to choose where you want to save your report. Perhaps we could add a property to the StrykerOptions of type any or something similar so it's generic. We'll also have to specify our SonarQube connection somewhere. grunt-sonar-runner does this in your Gruntfile. I'm not a big fan of saving credentials in a config because you may accidentally commit it. What do you think?

from stryker-js.

The StrykerOptions already has custom config of type any. We could do the same thing as stryker-html-reporter does and read options like this:

if (this.options['sonarReporter'] && this.options['sonarReporter']['username']) {
}

If we cannot think of sane defaults, we should throw an error early (before mutation testing).

We'll also have to specify our SonarQube connection somewhere. grunt-sonar-runner does this in your Gruntfile

You mean that the Sonar Qube plugin java (or scala?) application needs to do that? If i read my original issue from April 1st, that would be the case correct? Is it still up-to-date? Either way i'm not a fan of this either. We should use environment variables for it, with maybe a fallback to configuration.

from stryker-js.

Can we close this issue, now that we have the stryker-mutator/sonar-stryker-plugin?

from stryker-js.

Not sure if this is a mistake or not, but in the readme.md at step 4 it states that you have to 'move the jar to the downloads folder of your sonar qube instance (for example: /sonarqube-6.2/extensions/downloads)'.

Shouldn't downloads be plugins (I remember it to be plugins)?

from stryker-js.

I think sonar takes it from the downloads folder and puts it in the plugins folder

from stryker-js.

@philippw Yes, we can! 👍

from stryker-js.