Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

connect_graphs_avoid_random_success überdenken #1623

Open
AMatutat opened this issue Aug 16, 2024 · 0 comments
Open

connect_graphs_avoid_random_success überdenken #1623

AMatutat opened this issue Aug 16, 2024 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@AMatutat
Copy link
Contributor

Der Test connect_graphs_avoid_random_success() prüft, ob keine ungültigen Graphen vom Graphgenerator generiert werden. Der Test baut einen Graphen mit 10.000 Knoten und überprüft diesen. Das dauert zeitlich lange und wurde deshalb in der automatischen Test Suite deaktiviert. Es gibt einen analogen Test, der weniger Knoten erstellt und durchgeführt wird.

Der "große" Test entstand damals aus der "Angst", dass der erzeugte Graph nicht zufällig korrekt ist, sondern der Algorithmus dahinter. Bei den vielen Zufallswerten, die in der Generierung eine Rolle spielen, wollte ich mit einem besonders großen Graphen sicherstellen, dass der Algorithmus korrekt arbeitet.

In der Praxis wird niemand diesen Test je aktivieren, um Änderungen zu überprüfen. Ich selbst habe die Existenz des Tests auch vergessen.

@AMatutat AMatutat self-assigned this Aug 16, 2024
@AMatutat AMatutat added the bug Something isn't working label Aug 16, 2024
@AMatutat AMatutat changed the title ?connect_graphs_avoid_random_success` überdenken connect_graphs_avoid_random_success überdenken Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant