Skip to content
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

Upgrade Stryker Dashboard to the latest Mutation Test Elements (1.7.8) #211

Merged
merged 2 commits into from Dec 31, 2021

Conversation

maks-rafalko
Copy link
Contributor

@maks-rafalko maks-rafalko commented Dec 27, 2021

Reasons:

maks-rafalko added a commit to infection/infection that referenced this pull request Dec 27, 2021
Related to #1625

Now, Stryker logger allows to send badge information or the full mutation test elements report to display HTML report on Stryker Dashboard.

Example: https://dashboard.stryker-mutator.io/reports/github.com/infection/infection/master#mutant

Note, at the time of creating this PR, Stryker Dashboard has an old version of mutation-testing-elements lib which is too slow. Made a PR to update it: stryker-mutator/stryker-dashboard#211. The locally generated `html` report (see `logger.html` in `infection.json` already generates report with the latest lib)

This is a BC break.

Before:

`infection.json`

```json
{
    "logs": {
        "badge": {
            "branch": "master"
        }
    }
}
```

After:

`infection.json` - for badge information only

```json
{
    "logs": {
        "stryker": {
            "badge": "master"
        }
    }
}
```

or for badge *and* HTML report

```json
{
    "logs": {
        "badge": {
            "report": "master"
        }
    }
}
```
@nicojs
Copy link
Member

nicojs commented Dec 27, 2021

Thanks 👍

I'll update the pipeline this week and merge it in.

@nicojs nicojs enabled auto-merge (squash) December 31, 2021 09:12
@nicojs nicojs merged commit 0203930 into stryker-mutator:master Dec 31, 2021
@maks-rafalko maks-rafalko deleted the patch-1 branch December 31, 2021 09:19
maks-rafalko added a commit to infection/infection that referenced this pull request Jan 8, 2022
Related to #1625

Now, Stryker logger allows to send badge information or the full mutation test elements report to display HTML report on Stryker Dashboard.

Example: https://dashboard.stryker-mutator.io/reports/github.com/infection/infection/master#mutant

Note, at the time of creating this PR, Stryker Dashboard has an old version of mutation-testing-elements lib which is too slow. Made a PR to update it: stryker-mutator/stryker-dashboard#211. The locally generated `html` report (see `logger.html` in `infection.json` already generates report with the latest lib)

This is a BC break.

Before:

`infection.json`

```json
{
    "logs": {
        "badge": {
            "branch": "master"
        }
    }
}
```

After:

`infection.json` - for badge information only

```json
{
    "logs": {
        "stryker": {
            "badge": "master"
        }
    }
}
```

or for badge *and* HTML report

```json
{
    "logs": {
        "badge": {
            "report": "master"
        }
    }
}
```
maks-rafalko added a commit to infection/infection that referenced this pull request Jan 8, 2022
* Replace `badge` logger with more advanced `stryker` logger

Related to #1625

Now, Stryker logger allows to send badge information or the full mutation test elements report to display HTML report on Stryker Dashboard.

Example: https://dashboard.stryker-mutator.io/reports/github.com/infection/infection/master#mutant

Note, at the time of creating this PR, Stryker Dashboard has an old version of mutation-testing-elements lib which is too slow. Made a PR to update it: stryker-mutator/stryker-dashboard#211. The locally generated `html` report (see `logger.html` in `infection.json` already generates report with the latest lib)

This is a BC break.

Before:

`infection.json`

```json
{
    "logs": {
        "badge": {
            "branch": "master"
        }
    }
}
```

After:

`infection.json` - for badge information only

```json
{
    "logs": {
        "stryker": {
            "badge": "master"
        }
    }
}
```

or for badge *and* HTML report

```json
{
    "logs": {
        "badge": {
            "report": "master"
        }
    }
}
```

* Add more tests, remove todo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants