Skip to content
Snippets Groups Projects

Aggregated analyzer

Merged
Erik Oudsenrequested to merge
aggregated-analyzer into development
All threads resolved!

Description

Somebody should probably try to run this before merging, just saying :P

  • Fancy new aggregated analyzer that makes use of the job analyzer to be run.
  • A new analyzer, STREW metrics, that makes use of this new technology.

This MR is depending on !92 (merged) Which itself is depending on !88 (merged)
Lots of merging to do before this one goes through.
Resolves #198 (closed)
Resolves #169 (closed)
Resolves #152 (closed)

Changes

Entity.getMetricList() changed to Entity.getMetrics() as it returned a Set.
Windowsified the security agent checks, they have a high track record on Windows now.

Additions

  • Aggregated Analyzer
  • STREW metrics

Test and Review

To be filled in by the reviewers

  • All of the methods are commented to expectation

  • The methods are tested to satisfaction

  • There are no unnecessary files present in the MR

  • The continuous integration has no problems with the MR

  • The MR is filled in as requested (including labels, milestones, and reviewers)

  • The documentation is up-to-date

  • All nullable parameters are marked as such

Edited by Tim van der Horst

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Does not work when analyzing code-measures (output with debug statements)

    [2019-06-05 09:55:03,625] [StrewAnalyzer] [DEBUG] Adding relative number of assertions: (0.0/5258.0)
    [2019-06-05 09:55:03,640] [StrewAnalyzer] [DEBUG] Addding relative number of test case: (267.0/5258.0)
    [2019-06-05 09:55:03,640] [StrewAnalyzer] [DEBUG] Addding assertions per test case: (0.0/981.0)
    [2019-06-05 09:55:03,647] [StrewAnalyzer] [DEBUG] Addding test production ration: (4544.0/5258.0)/(0/163.0)
    [2019-06-05 09:55:03,677] [StrewAnalyzer] [DEBUG] Addding CC_RATIO: (343.0/737.0)
    [2019-06-05 09:55:03,705] [StrewAnalyzer] [DEBUG] Addding CBO_RATIO: (0.0/0.0)
    [2019-06-05 09:55:03,730] [StrewAnalyzer] [DEBUG] Addding DIT_RATIO: (0.0/0.0)
    [2019-06-05 09:55:03,755] [StrewAnalyzer] [DEBUG] Addding WMC_RATIO: (322.0/707.0)
  • Erik Oudsen marked as a Work In Progress

    marked as a Work In Progress

  • Erik Oudsen changed the description

    changed the description

  • Erik Oudsen added 3 commits

    added 3 commits

    • 66a61c9b - 1 commit from branch development
    • 5e89907b - Merge branch 'development' into aggregated-analyzer
    • 8c7c321e - Fix checker integration test

    Compare with previous version

  • Erik Oudsen added 3 commits

    added 3 commits

    • 79332821 - Fix STREW analyzers
    • fd4d8e95 - Reduce the number of beautiful streams
    • 5d3fc0a9 - Don't default to 1 but return nothing, just a debug statement

    Compare with previous version

  • Erik Oudsen unmarked as a Work In Progress

    unmarked as a Work In Progress

  • Erik Oudsen resolved all discussions

    resolved all discussions

  • Tim van der Horst
  • Erik Oudsen added 1 commit

    added 1 commit

    • 0fdb7add - Change metricList to metrics for the UI

    Compare with previous version

  • Erik Oudsen added 1 commit

    added 1 commit

    Compare with previous version

  • Erik Oudsen resolved all discussions

    resolved all discussions

  • Erik Oudsen added 46 commits

    added 46 commits

    Compare with previous version

  • Tim van der Horst approved this merge request

    approved this merge request

  • Tim van der Horst marked the checklist item All of the methods are commented to expectation as completed

    marked the checklist item All of the methods are commented to expectation as completed

  • Tim van der Horst marked the checklist item The methods are tested to satisfaction as completed

    marked the checklist item The methods are tested to satisfaction as completed

  • Tim van der Horst marked the checklist item There are no unnecessary files present in the MR as completed

    marked the checklist item There are no unnecessary files present in the MR as completed

  • Tim van der Horst marked the checklist item The continuous integration has no problems with the MR as completed

    marked the checklist item The continuous integration has no problems with the MR as completed

  • Tim van der Horst marked the checklist item The MR is filled in as requested (including labels, milestones, and reviewers) as completed

    marked the checklist item The MR is filled in as requested (including labels, milestones, and reviewers) as completed

  • Tim van der Horst marked the checklist item The documentation is up-to-date as completed

    marked the checklist item The documentation is up-to-date as completed

  • Tim van der Horst marked the checklist item All nullable parameters are marked as such as completed

    marked the checklist item All nullable parameters are marked as such as completed

  • Ewoud Ruighaver approved this merge request

    approved this merge request

  • Ewoud Ruighaver mentioned in commit 8de86774

    mentioned in commit 8de86774

  • unassigned @eoudsen

  • Please register or sign in to reply
    Loading