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

NaN values in i-Code results prevent SonarQube to import sonar-scanner report #8

Closed
begarco opened this issue Mar 21, 2018 · 1 comment
Closed

NaN values in i-Code results prevent SonarQube to import sonar-scanner report #8

begarco opened this issue Mar 21, 2018 · 1 comment
Assignees
@CfrancCyrille
Labels
bug
Milestone
Correction by Ate...

Comments

@begarco
Copy link
Contributor

begarco commented Mar 21, 2018

Expected behavior

The plugin must be able to import result file containing NaN for some metrics which is a nominal value given by i-Code. For example: when a function has lless than 10 lines, value for comment ratio is NaN.

Actual behavior

The sonar-scanner's report can not be imported.

Steps to reproduce behavior

  • Set following settings:
    • extension for result files: .res
    • source files' suffixes : *.f,*.f77,*.f90,*.sh,*.F,*.F77,*.F90
    • results folder : .
  • Create a quality profile containing all rules and set it as default
  • Execute sonar scanner in APSQ-003: APSQ-003.zip

Detection version

  • Plugin version: V0
  • SonarQube version: 6.7.1
@begarco begarco added the bug label Mar 21, 2018
@begarco begarco added this to the Correction by AtenoTech milestone May 25, 2018
@begarco begarco modified the milestones: Correction by AtenoTech, v1.0 May 25, 2018
CfrancCyrille added a commit to CfrancCyrille/sonar-icode-cnes-plugin that referenced this issue May 30, 2018
@CfrancCyrille
Fix cnescatlab#9 & cnescatlab#8 : bad langage analysis and cancel NaN…
3c15bfa 
… comment-ratio metrics
@CfrancCyrille CfrancCyrille mentioned this issue May 30, 2018
Merged
begarco pushed a commit that referenced this issue Jun 6, 2018
@CfrancCyrille @begarco
Fix #9 & #8 (#18)
5dfea6b 
* Fix #9 & #8 : bad langage analysis and cancel NaN comment-ratio metrics
@begarco
Copy link
Contributor Author

begarco commented Jun 6, 2018

Fix #8 in #18

@begarco begarco closed this as completed Jun 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@CfrancCyrille CfrancCyrille

Labels
bug
Projects
None yet
Milestone
Correction by AtenoTech
Development

No branches or pull requests
2 participants
@begarco @CfrancCyrille