[fix-14517][api] An error occurred while trying to access the Postgres data…

[hoey94]

Purpose of the pull request

This pull request fix #14517

Brief change log

old code:

if (dataSource.getType() == DbType.POSTGRESQL) {
    rs = connection.createStatement().executeQuery(Constants.DATABASES_QUERY_PG);
} 
rs = connection.createStatement().executeQuery(Constants.DATABASES_QUERY);

new code:

if (dataSource.getType() == DbType.POSTGRESQL) {
    rs = connection.createStatement().executeQuery(Constants.DATABASES_QUERY_PG);
} else {
    rs = connection.createStatement().executeQuery(Constants.DATABASES_QUERY);
}

Verify this pull request

This change can be verified as follows:

Manually verified the change by testing locally.

[codecov-commenter]

Codecov Report

Merging #14574 (c34430d) into dev (9d10de6) will increase coverage by 0.01%.
The diff coverage is 0.00%.

❗ Current head c34430d differs from pull request most recent head e00c467. Consider uploading reports for the commit e00c467 to get more accurate results

@@             Coverage Diff              @@
##                dev   #14574      +/-   ##
============================================
+ Coverage     38.66%   38.68%   +0.01%     
- Complexity     4575     4578       +3     
============================================
  Files          1241     1241              
  Lines         43569    43569              
  Branches       4826     4826              
============================================
+ Hits          16848    16854       +6     
+ Misses        24850    24848       -2     
+ Partials       1871     1867       -4     

Impacted Files	Coverage Δ
...eduler/api/service/impl/DataSourceServiceImpl.java	50.83% <0.00%> (ø)

... and 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[SbloodyS]

LGTM

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication