GoogleCloudPlatform / GoogleCloudPlatform/google-cloud-eclipse

Refactor UI-based tests to avoid duplication in pom.xml settings

Offen
#710 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

cleanup priority: p4 testing
Vorherrschende Sprache
Java
Sterne
90
Forks
52
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

com.google.cloud.tools.eclipse.integration.appengine and com.google.cloud.tools.eclipse.appengine.newproject.test both run tests that require the Eclipse workbench to be created, therefore both module's pom.xml contains the same configuration for tycho-surefire-plugin and almost identical target platform definition.

We should clean this up. Possible solutions:

  • define an eclipse feature that contains the TP and just refer to that from the bundles that require it
  • create a parent maven module, whose pom.xml contains these settings and any module that needs it would be a child of it thus inheriting the configuration

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Vergleiche die pom.xml-Dateien für com.google.cloud.tools.eclipse.integration.appengine und com.google.cloud.tools.eclipse.appengine.newproject.test und konzentriere dich dabei auf deren tycho-surefire-plugin-Einstellungen und Target-Platform-Definitionen. Prüfe die beiden vorgeschlagenen Maven- oder Eclipse-Feature-Ansätze und bestätige anschließend, dass die gemeinsame workbench-test-Konfiguration einmal definiert ist und beide Module ihre Tests weiterhin erfolgreich ausführen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java
Bereich
build-system
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.