Skip to content

#1337: add tests to simulate user input#1806

Open
laert-ll wants to merge 4 commits intodevonfw:mainfrom
laert-ll:feature/1337-add-tests-to-simulate-user-input
Open

#1337: add tests to simulate user input#1806
laert-ll wants to merge 4 commits intodevonfw:mainfrom
laert-ll:feature/1337-add-tests-to-simulate-user-input

Conversation

@laert-ll
Copy link
Copy Markdown
Contributor

@laert-ll laert-ll commented Apr 9, 2026

This PR fixes #1337

Implemented changes:

  • Added integration test simulating user input (yes/no/invalid) during IDE installation to verify proper stdin input.

Checklist for this PR

Make sure everything is checked before merging this PR. For further info please also see
our DoD.

  • When running mvn clean test locally all tests pass and build is successful
  • PR title is of the form #«issue-id»: «brief summary» (e.g. #921: fixed setup.bat). If no issue ID exists, title only.
  • PR top-level comment summarizes what has been done and contains link to addressed issue(s)
  • PR and issue(s) have suitable labels
  • Issue is set to In Progress and assigned to you or there is no issue (might happen for very small PRs)
  • You followed all coding conventions

@github-project-automation github-project-automation bot moved this to 🆕 New in IDEasy board Apr 9, 2026
@laert-ll laert-ll self-assigned this Apr 9, 2026
@laert-ll laert-ll moved this from 🆕 New to Team Review in IDEasy board Apr 9, 2026
@coveralls
Copy link
Copy Markdown
Collaborator

coveralls commented Apr 9, 2026

Coverage Report for CI Build 24241157392

Coverage decreased (-0.009%) to 70.457%

Details

  • Coverage decreased (-0.009%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 1 coverage regression across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

1 previously-covered line in 1 file lost coverage.

File Lines Losing Coverage Coverage
com/devonfw/tools/ide/tool/ide/IdeToolCommandlet.java 1 78.33%

Coverage Stats

Coverage Status
Relevant Lines: 15068
Covered Lines: 11067
Line Coverage: 73.45%
Relevant Branches: 6690
Covered Branches: 4263
Branch Coverage: 63.72%
Branches in Coverage %: Yes
Coverage Strength: 3.1 hits per line

💛 - Coveralls

Copy link
Copy Markdown
Contributor

@jakozian jakozian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work, i tried out the test and it succeeded 👍
I think this issues does not concern the user so it should be removed out of the CHANGELOG.

@jakozian jakozian moved this from Team Review to 👀 In review in IDEasy board Apr 10, 2026
@jakozian jakozian added internal Nothing to be added to CHANGELOG, only internal story integration-tests automated integration tests (see badge in README) labels Apr 10, 2026
@laert-ll laert-ll requested a review from hohwille April 10, 2026 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration-tests automated integration tests (see badge in README) internal Nothing to be added to CHANGELOG, only internal story

Projects

Status: 👀 In review

Development

Successfully merging this pull request may close these issues.

Add tests to simulate user input

3 participants