Bugfixes demo

Description

Fix many bugs for the demo.

Closes #186 (closed)

• Trampoline asked for report too early and submissionController returned empty report. Now returns null until getReportDone() does not return null
• Fix testSmellAnalyzer creating new fileEntities by creating a new container class that contains the TestFile and the fileEntity the files belong to.
• Fix testSmellAnalyzer not adding files to class by finding first class in the test file
• Test smells script did not work due to result.size() call
• Test smells output StringMetric and StringSetMetric
• CPD exited when lexical errors were encountered
• DuplicateCodeBlocks could not be serialized in J2V8
• Lizard passing script did not work

Changes

See above

Additions

Added passing scripts for various 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

changed milestone to %Bug fixes

added priocritical + 1 deleted label

added 1 commit

• 719130f4 - Raise version to 1.4.0 and fix broken CPDChecker default passing script.

Compare with previous version

added 1 commit

• da4f7ab5 - Add latest version of dist.zip

Compare with previous version

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

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

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

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

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

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

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

changed the description

approved this merge request

mentioned in commit 70a627b9

merged

unassigned @taivanderhorst