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

Update jest #4421

Draft
wants to merge 5 commits into
base: develop
Choose a base branch
from
Draft

Update jest #4421

wants to merge 5 commits into from

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Sep 19, 2024

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
@casualbot/jest-sonar-reporter 2.2.7 -> 2.4.0 age adoption passing confidence
@types/jest (source) 29.5.12 -> 29.5.13 age adoption passing confidence

Warning

Some dependencies could not be looked up. Check the Dependency Dashboard for more information.


Release Notes

CasualBot/jest-sonar-reporter (@​casualbot/jest-sonar-reporter)

v2.4.0

Compare Source

  • Fix package name in README (scanner -> reporter) #​23
  • Add files to solve NPM 10 lib/src strip problem #​22
  • Add projectRoot option for relativePaths #​20

v2.3.1

Compare Source

What's Changed

New Contributors

Full Changelog: CasualBot/jest-sonar-reporter@v2.2.7...v2.3.1

v2.3.0

Compare Source

What's Changed

New Contributors

Full Changelog: CasualBot/jest-sonar-reporter@v2.2.7...v2.3.1


Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Enabled.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate renovate bot added Dependencies Pull requests that update a dependency file T-Task Tasks for the team like planning labels Sep 19, 2024
Copy link
Contributor Author

renovate bot commented Sep 23, 2024

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@florianduros
Copy link
Contributor

florianduros commented Sep 24, 2024

Error in workflow:

INFO: ------------------------------------------------------------------------
INFO: EXECUTION FAILURE
INFO: ------------------------------------------------------------------------
INFO: Total time: 35.672s
ERROR: Error during SonarScanner execution
INFO: Final Memory: 52M/168M
ERROR: Error during parsing of generic test execution report '/github/workspace/coverage/jest-sonar-reporter-691e45f0-79c3-11ef-92de-2d87ce8c40b3.xml'. Look at the SonarQube documentation to know the expected XML format.
ERROR: Caused by: Missing attribute "message" in element "skipped" at line 243
ERROR: 
ERROR: Re-run SonarScanner using the -X switch to enable full debug logging.

SonarQube xml report doc https://docs.sonarsource.com/sonarqube/latest/analyzing-source-code/test-coverage/generic-test-data/
The report https://github.com/matrix-org/matrix-js-sdk/actions/runs/10997820582/artifacts/1967098135
SonarScanner version 5.0.1.3006

The 243 line :

<testCase name="MatrixClient emitted sync events should transition ERROR -&gt; CATCHUP after /sync if prev failed" duration="0">
 <skipped/>
</testCase>

The report format is correct...

@florianduros
Copy link
Contributor

https://docs.sonarsource.com/sonarqube/9.6/analyzing-source-code/test-coverage/generic-test-data/
Until the 9.6 (included), the message attribute is needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dependencies Pull requests that update a dependency file T-Task Tasks for the team like planning
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants