[BACKLOG-42356] Check if the file already exists before saving

[SreejaSridhar]

No description provided.

[hitachivantarasonarqube]

Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube

[NJtwentyone]

Cleaner implementation and simpler than previous PR - #5812

Mostly javadoc comments and unit testing not all comments are included below:

• https://github.com/pentaho/pentaho-platform/pull/5815/files#r1888984637

• https://github.com/pentaho/pentaho-platform/pull/5815/files#r1888985652

[hitachivantarasonarqube]

Analysis Details

0 Issues

• 0 Bugs
• 0 Vulnerabilities
• 0 Code Smells

Coverage and Duplications

• 81.80% Coverage (33.50% Estimated after merge)
• 0.00% Duplicated Code (3.30% Estimated after merge)

Project ID: pentaho:pentaho-platform-ce-parent

View in SonarQube

[buildguy]

Note:

---

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

---

🐸 JFrog Frogbot

[buildguy]

✅ Build finished in 23m 10s

Build command:

mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl extensions

---

👌 All tests passed!

Tests run: 1565, Failures: 0, Skipped: 5    Test Results

---

ℹ️ This is an automatic message

[NJtwentyone]

There are 2 high security scans issues on CodeQL PR checks, these need to addressed

[NJtwentyone]

CodeQL PR check fails:
"2 new alerts including 2 high severity security vulnerabilities"
#5815 (comment)

[SreejaSridhar]

codeQL check is failing for multiple unchanged files which are not pertained to just these PR files, "Unchanged files with check annotations"

Not making changes to this PR as it might affect multiple files.

[NJtwentyone]

All items were addressed.
As noted, codeql findings are unrelated to code changes in this PR - #5815 (comment)