[JIRA] (JENKINS-50087) Integration Test for Table Model (in LabelProvider)
Title: Message Title Stephan Plöderl commented on JENKINS-50087 Re: Integration Test for Table Model (in LabelProvider) fix ist in Pull-Request #66 in analysis-model Add Comment This message was sent by Atlassian JIRA (v7.3.0#73011-sha1:3c73d0e) -- You received this message because you are subscribed to the Google Groups "Jenkins Issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
[JIRA] (JENKINS-50087) Integration Test for Table Model (in LabelProvider)
Title: Message Title Stephan Plöderl commented on JENKINS-50087 Re: Integration Test for Table Model (in LabelProvider) oder besser gesagt schöner wäre es die endline berechnung in Cpd (vllt. auch simian, weiß ich noch nicht) zu fixen. Diese besteht nämlich aus startline + numberOfDuplicateLines. Add Comment This message was sent by Atlassian JIRA (v7.3.0#73011-sha1:3c73d0e) -- You received this message because you are subscribed to the Google Groups "Jenkins Issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
[JIRA] (JENKINS-50087) Integration Test for Table Model (in LabelProvider)
Title: Message Title
Stephan Plöderl commented on JENKINS-50087
Re: Integration Test for Table Model (in LabelProvider)
Hab das Problem gefunden. Es liegt an der Json-Generierung von den Duplicate-code warnings. File: DuplicateCodeScanner.java Dort wird in der toJson-Methode die Zeilenanzahl falsch berechnet:
@Override
protected JSONArray toJson(final Issue issue, final AgeBuilder ageBuilder) {
JSONArray columns = new JSONArray();
columns.add(formatDetails(issue));
columns.add(formatFileName(issue));
columns.add(formatPriority(issue.getPriority()));
columns.add(issue.getLineEnd() - issue.getLineStart() + 1);
columns.add(formatTargets(issue));
columns.add(formatAge(issue, ageBuilder));
return columns;
}
Das +1 ist zu viel, weshalb die Zeilenanzahl falsch angezeigt wird. Und da die Zeilenanzahl im Codeparser die wirkliche Zeilenanzahl, welche z.B. Cpd ausgibt verwendet, wird für die Priority zum Beispiel 3 doppelte Lines verwendet und in der Weboberfläche dann 4 angezeigt. Was zu einem Widerspruch zu den High- und NormalThresholds führt. Ich schau noch, ob die änderung irgendwelche auswirkungen auf die bisherigen tests hat. Überprüfe noch die Berechnung der doppelten Code Zeilen für Simian und stelle dann einen gesonderten Pull-Request.
Add Comment
This message was sent by Atlassian JIRA (v7.3.0#73011-sha1:3c73d0e)
[JIRA] (JENKINS-51436) Hardcoded id's for Html-Components within loops
Title: Message Title Stephan Plöderl created an issue Jenkins / JENKINS-51436 Hardcoded id's for Html-Components within loops Issue Type: Bug Assignee: Ulli Hafner Components: analysis-core-plugin Created: 2018-05-19 12:00 Environment: hm-edu-testing Labels: plugins Priority: Minor Reporter: Stephan Plöderl Within the hm-edu-testing branch, there are multiple jelly files which have hardcoded strings as id's for HTML-Components created within a loop. This will create multiple HTML-Components with the same Id and will therefore create problems for tests, _javascript_-logic and/or the usage of css. The mentioned files are the following: src/main/resources/result/fixed.jelly src/main/resources/io/jenkins/plugins/analysis/core/views/FixedWarningsDetail/index.jelly src/main/resources/hudson/plugins/analysis/views/TabDetail/details.jelly Where as "src/main/resources/hudson/plugins/analysis/views/TabDetail/details.jelly" may be connected to PR #123
[JIRA] (JENKINS-50087) Integration Test for Table Model (in LabelProvider)
Title: Message Title Stephan Plöderl commented on JENKINS-50087 Re: Integration Test for Table Model (in LabelProvider) Ok, hab das Problem gefunden und sowohl für CPD, als auch für Simian behoben. Jetzt ist mir gerade aufgefallen, dass der Text für HighThreshold besagt, dass es die minimale Anzahl an Zeilen ist um ein High-Issue zu erzeugen. Allerdings ist der Code gerade so geschrieben, dass erst ab einem wert höher ein High issue erzeugt wird. (Das selbe gilt für normal) Jetzt die Frage, was soll jetzt genau richtig sein? ist es die minimale anzahl für ein high-issue oder die maximale anzahl um noch ein Normal-Issue zu erhalten? Add Comment This message was sent by Atlassian JIRA (v7.3.0#73011-sha1:3c73d0e) -- You received this message because you are subscribed to the Google Groups "Jenkins Issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
[JIRA] (JENKINS-50087) Integration Test for Table Model (in LabelProvider)
Title: Message Title Stephan Plöderl commented on JENKINS-50087 Re: Integration Test for Table Model (in LabelProvider) Habe in analysis-core-plugin im Pull-Request #122 ein paar Probleme behoben, die diesen Test behindert haben. Zum Beispiel das Problem, dass die Tabelle nicht geladen wird, wenn man die Seite aufruft. Dies passierte, weil die Ajax-Funktion nicht aufgerufen wurde, wenn man die Homepage besucht hatte, ohne einmal den Tab gewechselt zu haben. Sprich, wenn der activeTab im localStorage noch nicht gesetzt war. Ich versuche auch noch den Bug zu finden, welcher verhindert, dass die richtigen Thresholds genutzt werden. Add Comment This message was sent by Atlassian JIRA (v7.3.0#73011-sha1:3c73d0e) -- You received this message because you are subscribed to the Google Groups "Jenkins Issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
[JIRA] (JENKINS-50087) Integration Test for Table Model (in LabelProvider)
Title: Message Title Stephan Plöderl commented on JENKINS-50087 Re: Integration Test for Table Model (in LabelProvider) Außerdem scheint auch die Eingabe verschiedener Werte für die Thresholds nicht beachtet zu werden. Es werden scheinbar immer die Default-Werte von 50 und 25 genommen, egal was man auch eingibt. Bin mir zwar gerade nicht sicher, ob die beiden Probleme zusammen hängen, aber es scheint mir nicht so. Da die Werte ja gespeichert werden, auch wenn die Ajax-Validierung von Jenkins fehl schlägt. Add Comment This message was sent by Atlassian JIRA (v7.3.0#73011-sha1:3c73d0e) -- You received this message because you are subscribed to the Google Groups "Jenkins Issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
[JIRA] (JENKINS-50087) Integration Test for Table Model (in LabelProvider)
Title: Message Title Stephan Plöderl commented on JENKINS-50087 Re: Integration Test for Table Model (in LabelProvider) Vielen Dank, läuft wunderbar. Mir ist gerade noch ein Bug aufgefallen, allerdings finde ich die Stelle nicht, wo dieser genau auftritt, weshalb ich ihn nicht fixen kann. Die Validation bei CPD und Simian zeigt immer die Fehlermeldungen an, das die Thresholds nicht stimmen. Ich konnte es soweit runter tracken, dass jenkins hier den Wert des jeweiligen Feldes immer nur als 0 schickt. Die Referenz der QueryParameter scheint er in diesem Fall also irgendwie nicht zu erkennen, jedoch konnte ich bisher noch nicht herausfinden, warum dies so ist. Der Wert checkdependson ist auf jeden Fall bei den beiden Werten ein leerer String. In den check-Funktionen sind deshalb die beiden Werte immer 0 und deshalb schlägt die Validation fehl. Da deshalb der normalThreshold gleich dem highThreshold ist. Ich hoffe Sie können mir an dieser Stelle weiter helfen. Add Comment This message was sent by Atlassian JIRA (v7.3.0#73011-sha1:3c73d0e) -- You received this message because you are subscribed to the Google Groups "Jenkins Issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
[JIRA] (JENKINS-50087) Integration Test for Table Model (in LabelProvider)
Title: Message Title Stephan Plöderl assigned an issue to Stephan Plöderl Jenkins / JENKINS-50087 Integration Test for Table Model (in LabelProvider) Change By: Stephan Plöderl Assignee: Ulli Hafner Stephan Plöderl Add Comment This message was sent by Atlassian JIRA (v7.3.0#73011-sha1:3c73d0e) -- You received this message because you are subscribed to the Google Groups "Jenkins Issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
[JIRA] (JENKINS-50073) Mockito Test for StablePluginReference
Title: Message Title Stephan Plöderl updated JENKINS-50073 Jenkins / JENKINS-50073 Mockito Test for StablePluginReference Change By: Stephan Plöderl Status: In Progress Review Add Comment This message was sent by Atlassian JIRA (v7.3.0#73011-sha1:3c73d0e) -- You received this message because you are subscribed to the Google Groups "Jenkins Issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
[JIRA] (JENKINS-50087) Integration Test for Table Model (in LabelProvider)
Title: Message Title Stephan Plöderl started work on JENKINS-50087 Change By: Stephan Plöderl Status: Open In Progress Add Comment This message was sent by Atlassian JIRA (v7.3.0#73011-sha1:3c73d0e) -- You received this message because you are subscribed to the Google Groups "Jenkins Issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
[JIRA] (JENKINS-50073) Mockito Test for StablePluginReference
Title: Message Title Stephan Plöderl commented on JENKINS-50073 Re: Mockito Test for StablePluginReference Ich würde das Ticket auch bearbeiten Add Comment This message was sent by Atlassian JIRA (v7.3.0#73011-sha1:3c73d0e) -- You received this message because you are subscribed to the Google Groups "Jenkins Issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
[JIRA] (JENKINS-50069) Unit Test for EnvironmentResolver
Title: Message Title Stephan Plöderl assigned an issue to Stephan Plöderl Jenkins / JENKINS-50069 Unit Test for EnvironmentResolver Change By: Stephan Plöderl Assignee: Ulli Hafner Stephan Plöderl Add Comment This message was sent by Atlassian JIRA (v7.3.0#73011-sha1:3c73d0e) -- You received this message because you are subscribed to the Google Groups "Jenkins Issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
