Skip to content

Tests aus Entwicklerbuild herausnehmen #108

Closed

Description

@pidoubleyou

Aus einer Diskussion über die Laufzeit der Tests in #102 entstand der Wunsch, dass im normalen Entwicklerbuild keine Tests automatisch ausgeführt werden sollen.
Die Tests sollen nur laufen, wenn der Entwickler diese manuell startet.
Im Travis sollen die Tests aber weiterhin im Build mitlaufen.

Activity

  1. Nicklas2751 commented on Mar 4, 2017

    @Nicklas2751
    Member

    Ok, kann ich machen, hatte ich auch schon für meinen Performance Test gemacht.

  2. self-assigned this
    on Mar 4, 2017
  3. self-assigned this
    on Mar 5, 2017
  4. pidoubleyou commented on Mar 5, 2017

    @pidoubleyou
    ContributorAuthor

    Die "Lösung" für den Performancetest deckt aus meiner Sicht nicht die Anforderung ab.
    Durch das exclude in build.gradle wird der Test soweit ich es sehe, nie ausgeführt. Als Entwickler kann man den Test nur ausführen, wenn man das exclude des Tests explizit entfernt.

    Ich probiere gerade eine Lösung mit einem eigenen Gradle-Task aus.

  5. pidoubleyou commented on Mar 5, 2017

    @pidoubleyou
    ContributorAuthor

    Ich habe im Branch für das Feature eine Lösung mit eigenen Gradle-Tasks eingebracht.
    Der ursprüngliche Task test führt jetzt keine Tests mehr aus, so dass im "normalen" Develop-Build keine Test mehr laufen.

    Um Tests auszuführen, gibt es jetzt drei eigenständige Tasks:

    • developTest: führt die UnitTests aus
    • integrationTest: führt die IntegrationTests aus (aktuell gibt es dafür nur den ZDF-Crawler)
    • performanceTest: führt die PerformanceTests aus

    Die Lösung bedingt, dass der Entwickler der Tests diese in den entsprechenden Ordner einsortiert, aber das sehe ich nicht als Problem an.

    Im Travis werden aktuell nur developTest und integrationTest ausgeführt, da der PerformanceTest scheitert.

    @Nicklas2751 @alex1702 @derreisende77 was haltet ihr von der Lösung?

  6. Nicklas2751 commented on Mar 6, 2017

    @Nicklas2751
    Member

    Passt

  7. added 2 commits that reference this issue on Mar 27, 2017
    40470f3
    b83221b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions