GH-30 Add tests to CI

[frank-connolly]

The tests run here use the H2 in-memory DB rather than the PostgresDB to speed it up.
We can switch to Postgres later if we feel a closer reflection of production would be better

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

[ozkanpakdil]

LGTM

[frank-connolly]

LGTM

@ozkanpakdil Any reason why I'm not seeing the test coverage output to the PR here? Did I break something?

[madzimai]

@frank-connolly ,Please resolve the conflicts on
.github/workflows/CI.yml

[ozkanpakdil]

LGTM

@ozkanpakdil Any reason why I'm not seeing the test coverage output to the PR here? Did I break something?

@frank-connolly
the xml is not generated. if you check other build artifacts there is jacoco xml which is not generated with your PR, not sure why.

you can test manually in your local, by running the commands and check under build folder

[frank-connolly]

LGTM

@ozkanpakdil Any reason why I'm not seeing the test coverage output to the PR here? Did I break something?

@frank-connolly the xml is not generated. if you check other build artifacts there is jacoco xml which is not generated with your PR, not sure why.

you can test manually in your local, by running the commands and check under build folder

I'll investigate it when I get a chance - thanks!

[ArmanRuhit]

Isn't setup-java@v1 an old version?

[frank-connolly]

Isn't setup-java@v1 an old version?

Could be yes! This isn't blocking anything and I'll check it out when I get a chance. Thanks!

[frank-connolly]

Revisiting this soon.