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.

Sign up for GitHub

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

Jump to bottom

Code cleanup to correct issues reported by Sonar (Severity: high - part 3) #2981

Merged
merged 170 commits into from
Apr 26, 2024
Merged

Code cleanup to correct issues reported by Sonar (Severity: high - part 3) #2981

merged 170 commits into from
Apr 26, 2024

Conversation

rolnico
Copy link
Member

@rolnico rolnico commented Apr 16, 2024 •
edited
Loading

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Does this PR already have an issue describing the problem?

Follows #2967, #2971 and #2974 for issues identified as high severity in the rest of the modules modules

What kind of change does this PR introduce?

What is the current behavior?

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change or deprecate an API?

  • Yes
  • No

If yes, please check if the following requirements are fulfilled

  • The Breaking Change or Deprecated label has been added
  • The migration steps are described in the following section

What changes might users need to make in their application due to this PR? (migration steps)

Other information:

rolnico added 30 commits April 5, 2024 15:42
@rolnico
add test to avoid NullPointerException
625fd76 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
fix return statement in MergedBus
5c8d233 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
add test to avoid potential ArrayIndexOutOfBoundsException
6f77f25 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
BREAKING CHANGE - Change JsonTypeInfo to avoid unsafe Jackson deseria…
624c337 
…lization https://rules.sonarsource.com/java/RSPEC-4544/

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
fix GeneratorModificationActionTest
aa523f1 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
remove peek usage
489d5ff 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
remove duplicated tests
c5d232c 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
remove exceptions not thrown
05de80b 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
if simplifications
7f23910 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
removed unused methods
bcd721f 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
use Supplier from java instead if guava
21da610 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
use Supplier from java instead if guava
b3ce26c 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
ParsingContext declared final
9867b6d 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
XmlStaxContext declared final
6792363 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
replaced deprecated method for ZipFile with SeekableByteChannel
266e337 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
swapped from deprecated org.apache.commons.compress.utils.IOUtils to …
1a38ad6 
…org.apache.commons.io.IOUtils

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
removed deprecated .newInstance method call
0afcfbc 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
EmptyModuleConfigRepository declared final
e81dc90 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
removed old deprecated methods + replaced guava supplier by java supp…
8541b77 
…lier

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
replaced deprecated Zipfile constructor by builder
d836935 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
replaced supplier
59fd06e 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
fix
d7a6662 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
fix
06cbf2c 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
fix StandardCharsets.UTF_8
0e3d1b7 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
fix
4e1cf4a 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
fix
7a8c0eb 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
replace supplier
af70c7f 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
replace ZipFile by builder
05235c0 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
catch Exception instead of Throwable
afe04ce 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
group logic in createComputationManager for shortTime and longTime co…
f5ef925 
…mputationManager

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
rolnico added 4 commits April 15, 2024 17:18
@rolnico
reduce method complexity
911fadd 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
reduce method complexity
5cf07a6 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
reduce method complexity
b0eb409 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
Define a constant instead of duplicating this literal
b137a0a 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico rolnico changed the base branch from main to nro/cleanup_severity_high_2 April 16, 2024 06:37
@rolnico
rename test class to match expected regex
636ef72 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico rolnico self-assigned this Apr 23, 2024
jeandemanged
jeandemanged requested changes Apr 24, 2024
View reviewed changes
@olperr1 olperr1 mentioned this pull request Apr 25, 2024
Merged
7 tasks
rolnico added 14 commits April 25, 2024 09:42
@rolnico
Merge branch 'refs/heads/main' into nro/cleanup_severity_high_2
3dd6772 
# Conflicts:
#	action-api/src/test/java/com/powsybl/action/json/JsonActionTest.java
#	cgmes/cgmes-conformity/src/main/java/com/powsybl/cgmes/conformity/CgmesConformity1NetworkCatalog.java
#	cgmes/cgmes-conversion/src/main/java/com/powsybl/cgmes/conversion/export/StateVariablesExport.java
#	cgmes/cgmes-conversion/src/test/java/com/powsybl/cgmes/conversion/test/TransformerConversionTest.java
#	commons/src/main/java/com/powsybl/commons/config/ComponentDefaultConfig.java
#	commons/src/main/java/com/powsybl/commons/parameters/Parameter.java
#	computation/src/main/java/com/powsybl/computation/DefaultComputationManagerConfig.java
#	iidm/iidm-serde/src/main/java/com/powsybl/iidm/serde/XMLImporter.java
#	loadflow/loadflow-validation/src/main/java/com/powsybl/loadflow/validation/ValidationUtils.java
#	loadflow/loadflow-validation/src/main/java/com/powsybl/loadflow/validation/io/AbstractValidationFormatterWriter.java
#	time-series/time-series-api/src/main/java/com/powsybl/timeseries/TimeSeriesMetadata.java
@rolnico
fix import
b9bb3fc 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
fix class names
209b1e3 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
fix LocalComputationManager
0a22c28 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
rename compareYyy methods to assertYyyEquals
7e92d99 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
fix for review
1f254f7 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
remove unused import
0d610e4 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
fix checkstyle
d46a8eb 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
new PsseIoConstants class
b5c3ab3 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
fix exceptions in SensitivityAnalysisTool
ca25c1c 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
remove empty constructor (default is the same)
1c5f0b7 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
rename method + extract another
8a7a3a7 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
fix issues
415d9d4 
Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>
@rolnico
Merge branch 'refs/heads/nro/cleanup_severity_high_2' into nro/cleanu…
337a0ec 
…p_severity_high_3

# Conflicts:
#	loadflow/loadflow-api/src/test/java/com/powsybl/loadflow/tools/RunLoadFlowToolTest.java
#	security-analysis/security-analysis-api/src/test/java/com/powsybl/security/json/JsonActionAndOperatorStrategyTest.java
Base automatically changed from nro/cleanup_severity_high_2 to main April 25, 2024 09:28
@rolnico
Merge branch 'refs/heads/main' into nro/cleanup_severity_high_3
70b3e76 
# Conflicts:
#	iidm/iidm-impl/src/test/java/com/powsybl/iidm/network/impl/util/GraphvizConnectivityTest.java
#	iidm/iidm-serde/src/test/java/com/powsybl/iidm/serde/SkipExtensionTest.java
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
87.7% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@olperr1 olperr1 requested a review from jeandemanged April 26, 2024 07:37
@olperr1 olperr1 merged commit af6271c into main Apr 26, 2024
6 checks passed
@olperr1 olperr1 deleted the nro/cleanup_severity_high_3 branch April 26, 2024 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jeandemanged jeandemanged jeandemanged approved these changes

Assignees

@rolnico rolnico

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rolnico @jeandemanged @olperr1