Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-50087

Integration Test for Table Model (in LabelProvider)

        [JENKINS-50087] Integration Test for Table Model (in LabelProvider)

        Ulli Hafner created issue -
        Ulli Hafner made changes -
        Epic Link New: JENKINS-50065 [ 189073 ]
        Ulli Hafner made changes -
        Rank New: Ranked higher
        Stephan Plöderl made changes -
        Assignee Original: Ulli Hafner [ drulli ] New: Stephan Plöderl [ stephan_p ]
        Stephan Plöderl made changes -
        Status Original: Open [ 1 ] New: In Progress [ 3 ]

        Ulli Hafner added a comment -

        Sieht schlecht aus mit der JS Unterstützung in HTML Unit. Ich bin auf einige Compile Fehler gestoßen bei der Aktualisierung... Anfrage in der Mailing Liste läuft.

        Ulli Hafner added a comment - Sieht schlecht aus mit der JS Unterstützung in HTML Unit. Ich bin auf einige Compile Fehler gestoßen bei der Aktualisierung... Anfrage in der Mailing Liste läuft.

        Ulli Hafner added a comment -

        So, jetzt ging es doch schneller als gedacht. Ich habe eine neue Version der Test Bibliothek hochgeladen. Damit Sie diese bekommen müssen Sie im pom.xml des warnings plugins die properties so setzen:

          <properties>
            <pmd.version>6.1.0</pmd.version>
            <jenkins-test-harness.version>2.39-20180503.223048-1</jenkins-test-harness.version>
          </properties>
        

        Im Testfall kann dann das Setzen von JS auf false unterbleiben.

        Ulli Hafner added a comment - So, jetzt ging es doch schneller als gedacht. Ich habe eine neue Version der Test Bibliothek hochgeladen. Damit Sie diese bekommen müssen Sie im pom.xml des warnings plugins die properties so setzen: <properties> <pmd.version>6.1.0</pmd.version> <jenkins-test-harness.version>2.39-20180503.223048-1</jenkins-test-harness.version> </properties> Im Testfall kann dann das Setzen von JS auf false unterbleiben.

        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.

        Stephan Plöderl added a comment - 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.

        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.

        Stephan Plöderl added a comment - 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.

        Ulli Hafner added a comment -

        Zum ersten Punkt: das habe ich vergessen zu sagen. Da habe ich auch schon mal den Debugger angeworfen und nichts gefunden. (Siehe JENKINS-50355)

        Ulli Hafner added a comment - Zum ersten Punkt: das habe ich vergessen zu sagen. Da habe ich auch schon mal den Debugger angeworfen und nichts gefunden. (Siehe  JENKINS-50355 )

          stephan_p Stephan Plöderl
          drulli Ulli Hafner
          Votes:
          0 Vote for this issue
          Watchers:
          2 Start watching this issue

            Created:
            Updated:
            Resolved: