chore: Replace data element no groups Java check with SQL based check

[jason-p-pickering]

• Replaces check for aggregate data elements with no groups with a Java based check
• Add unit test in legacy report

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

0.0% Coverage
0.0% Duplication

[codecov]

Codecov Report

Merging #15812 (d4b5201) into master (d0e12c6) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #15812      +/-   ##
============================================
- Coverage     66.34%   66.34%   -0.01%     
  Complexity    31440    31440              
============================================
  Files          3491     3491              
  Lines        130107   130105       -2     
  Branches      15192    15192              
============================================
- Hits          86322    86320       -2     
- Misses        36704    36705       +1     
+ Partials       7081     7080       -1     

Flag	Coverage Δ
integration	50.04% <100.00%> (+<0.01%)	⬆️
integration-h2	32.33% <0.00%> (-0.01%)	⬇️
unit	30.40% <0.00%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
...isp/dhis/dataintegrity/DataIntegrityCheckType.java	100.00% <ø> (ø)
...is/dataintegrity/FlattenedDataIntegrityReport.java	97.26% <100.00%> (-0.02%)	⬇️
...his/dataintegrity/DefaultDataIntegrityService.java	76.68% <100.00%> (ø)

... and 3 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d0e12c6...d4b5201. Read the comment docs.

[david-mackessy]

nice find, the sql type check was always there but never used?