[comixed-dev] Re: SonarCloud re-enabled

  • From: "Darryl L. Pierce" <mcpierce@xxxxxxxxx>
  • To: comixed-dev@xxxxxxxxxxxxx
  • Date: Wed, 18 Aug 2021 16:24:17 -0400

These changes are now done and I've closed out #836. If we see any new
issues around SonarCloud we can open a new issue to resolve the specific
issue.

On Wed, Aug 18, 2021 at 1:52 PM Darryl L. Pierce <mcpierce@xxxxxxxxx> wrote:

Will do. And thanks again for fixing the SonarCloud issues! :D

On Wed, Aug 18, 2021 at 11:06 AM Frédéric Fouret <fouretfrederic@xxxxxxxxx>
wrote:

Today I will be short on time so if you could apply the other changes you
can do it :)

Le mer. 18 août 2021 à 16:57, Darryl L. Pierce <mcpierce@xxxxxxxxx> a
écrit :

Okay, I've merged that commit onto master. Do you want me to apply those
other changes or do you have some time to do them today?

On Wed, Aug 18, 2021 at 9:59 AM Frédéric Fouret <
fouretfrederic@xxxxxxxxx> wrote:

I saw that coverage report popup, but then it went back to 0% in a
follow up push.


The PR 842 is ok with no coverage information because there is just pom
modification.. I think you can merge it in master, rebase the PR 841 onto
master and see the coverage if it's ok.

The things you found seem indeed interesting, especially if you move
the build logic into the build control files


Le mer. 18 août 2021 à 15:45, Darryl L. Pierce <mcpierce@xxxxxxxxx> a
écrit :

I saw that coverage report popup, but then it went back to 0% in a
follow up push.

I did some reading on SonarCloud and found a few things that seem like
they would make it easier to control the coverage:

1) Typescript needs different settings from Java, so the following
properties should be in the comixed-webui/pom.xml file:
    <sonar.sources>src</sonar.sources>
    <sonar.javascript.lcov.reportPaths>coverage/lcov.info
</sonar.javascript.lcov.reportPaths>
2) With those settings moved, then we should be able to put into the
root pom.xml the following properties:
    <sonar.organization>comixed</sonar.organization>
    <sonar.host.url>https://sonarcloud.io</sonar.host.url>
    <sonar.sources>src/main/java</son.sources>

<sonar.coverage.jacoco.xmlReportPaths>${project.basedir}/comixed/release/target/site/jacoco-aggregate/jacoco.xml</sonar.coverage.jacoco.xmlReportPaths>
3) And we can change .github/workflows/sonarcloud-analysis.yml could
use the following build command line;
    mvn -B clear verify
org.sonarsource.scanner.maven:sonar-maven-plugin:sonar
-Dsonar.projectKey=comixed -Pci

That should simplify the Github action to just running things and move
the build logic into the build control files where I think they should be.

What do you think?

On Wed, Aug 18, 2021 at 8:55 AM Frédéric Fouret <
fouretfrederic@xxxxxxxxx> wrote:

After taking a look at PR 841 (
https://github.com/comixed/comixed/pull/841), I found that the
coverage was not taken into account 0% when it should be 100% on the
modifications.
I tried some more corrections and finally came up with something that
seems to work (at least in this case). After modifying the pom and 
redoing
a build and an analysis by sonarcloud we get 100% coverage.

So the PR should be able to finally solve the SonarCloud problem I
think.
[image: before_modification.png]
[image: after_modification.png]

Le mar. 17 août 2021 à 16:58, Darryl L. Pierce <mcpierce@xxxxxxxxx>
a écrit :

Okay, after a few more tweaks things are now actually working (and I
checked this time <g>). I opened fixed a group of issues found by
SonarCloud and have closed #836 as resolved.

Frédéric, merci pour ton aide. :D

On Mon, Aug 16, 2021 at 5:47 PM Darryl L. Pierce <mcpierce@xxxxxxxxx>
wrote:

No, spoke too soon. Or else something else is the issue.

I just pushed up a PR for #706 to add viewing the details for a
duplicate page. I added a new service on the Java side, fully coverage 
by
unit tests. But SonarCloud is saying there are no unit tests for the 
code...

On Mon, Aug 16, 2021 at 2:56 PM Darryl L. Pierce <
mcpierce@xxxxxxxxx> wrote:

Boom! It appears to be working correctly now! I had to change the
root POM's name from "comixed-parent" to just "comixed" to get the
SonarCloud display right, but it was your changes, Frédéric, plus 
fixing
the project key that did the trick. Thank you!

On Mon, Aug 16, 2021 at 2:09 PM Darryl L. Pierce <
mcpierce@xxxxxxxxx> wrote:

I tried changing it to just "comixed" (it was "comixed_comixed")
and pushed that change now.

On Sun, Aug 15, 2021 at 5:13 AM Frédéric Fouret <
fouretfrederic@xxxxxxxxx> wrote:

I made a new PR that should correct the path to the jacoco.xml
file.
Regarding the analysis of the branch master, I saw that we have
the following error when the analysis starts:
errorMessage": "Compute Engine task main component key is null."

After searching the internet, I came across the following thread:

https://community.sonarsource.com/t/branch-analysis-fails-for-some-branches-succeeds-for-others-using-azure-devops/11932/2
It seems that updating the project key in sonarcloud can fix the
problem... maybe you can try to see that.

Le sam. 14 août 2021 à 23:10, Darryl L. Pierce <
mcpierce@xxxxxxxxx> a écrit :

Excellent work, Frederic! Thanks for looking into this. I'm
squashing your commits together on master as you tackle them so we 
can keep
the commits together. I have a build running now to see if 
SonarCloud is
happy again.

On Sat, Aug 14, 2021 at 7:48 AM Frédéric Fouret <
fouretfrederic@xxxxxxxxx> wrote:

Hello,

I am looking at the problem and trying to find a solution. I
have made a PR in development. It has been merged but does not 
solve the
problem.

From what I have seen, it looks for the jacoco.exec file in
the wrong place. After fixing that, I have another error, it 
tries to parse
the file as if it is xml when we have a binary I think.

Work in progress... i hope to find the solution for the first
issue.


Le sam. 14 août 2021 à 00:08, Darryl L. Pierce <
mcpierce@xxxxxxxxx> a écrit :

I've re-enabled SonarCloud and disabled the GitHub code
coverage tests since the latter weren't doing a very good job 
IMO. But for
some reason SonarCloud 1) won't check the Java code, it's only 
doing the
Typescript code, and 2) won't process our master branch.

Can someone (Frederic?) take a look at our
sonarcloud-analysis.yml on master and maybe give us a PR to fix 
this?

Thanks!

--
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap
together." - Gord Downie



--
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap
together." - Gord Downie



--
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap
together." - Gord Downie



--
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap
together." - Gord Downie



--
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together."
- Gord Downie



--
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together."
- Gord Downie



--
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together." -
Gord Downie



--
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together." -
Gord Downie



--
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together." - Gord
Downie



-- 
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together." - Gord
Downie

PNG image

PNG image

Other related posts:

  1. Home
  2. comixed-dev
  3. Archive
  4. 08-2021