Skip to content

#1270: Load global user settings outside project#1813

Open
ducminh02 wants to merge 4 commits intodevonfw:mainfrom
ducminh02:fix/1270-load-global-user-settings-outside-project
Open

#1270: Load global user settings outside project#1813
ducminh02 wants to merge 4 commits intodevonfw:mainfrom
ducminh02:fix/1270-load-global-user-settings-outside-project

Conversation

@ducminh02
Copy link
Copy Markdown
Contributor

@ducminh02 ducminh02 commented Apr 10, 2026

This PR fixes #1270

Implemented changes:

  • Updated setUserHome to refresh dependent paths (userHomeIde, downloadPath) and reset variables. This ensures tests using isolated user homes correctly load local configurations like ide.properties.

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
  • You have added the issue implemented by your PR in CHANGELOG.adoc unless issue is labeled
    with internal

@coveralls
Copy link
Copy Markdown
Collaborator

coveralls commented Apr 10, 2026

Coverage Report for CI Build 24239455949

Coverage increased (+0.01%) to 70.478%

Details

  • Coverage increased (+0.01%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 105 coverage regressions across 3 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

105 previously-covered lines in 3 files lost coverage.

File Lines Losing Coverage Coverage
com/devonfw/tools/ide/context/AbstractIdeContext.java 103 66.01%
com/devonfw/tools/ide/commandlet/EnvironmentCommandlet.java 1 94.44%
com/devonfw/tools/ide/tool/ide/IdeToolCommandlet.java 1 78.33%

Coverage Stats

Coverage Status
Relevant Lines: 15070
Covered Lines: 11071
Line Coverage: 73.46%
Relevant Branches: 6690
Covered Branches: 4265
Branch Coverage: 63.75%
Branches in Coverage %: Yes
Coverage Strength: 3.1 hits per line

💛 - Coveralls

@ducminh02 ducminh02 moved this from 🆕 New to 🏗 In progress in IDEasy board Apr 10, 2026
@ducminh02 ducminh02 marked this pull request as ready for review April 10, 2026 10:53
@ducminh02 ducminh02 self-assigned this Apr 10, 2026
@hohwille hohwille moved this from 🏗 In progress to Team Review in IDEasy board Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Team Review

Development

Successfully merging this pull request may close these issues.

IDEasy does not load global user settings if outside a project

4 participants