tests: add missing fixture to test_fields.test_page (#89318) #1386

yweber · 2024-04-09T16:22:26+02:00

yweber commented

2024-04-09 16:22:26 +02:00

No description provided.

yweber added 1 commit 2024-04-09 16:22:27 +02:00

gitea/wcs/pipeline/head This commit looks good Details

cebebe0693 tests: add missing fixture to test_fields.test_page (#89318 )

fpeters requested changes 2024-04-09 16:24:43 +02:00

Dismissed

fpeters left a comment

Les tests sont prévus pour être lancés dans leur intégralité par fichier; ça peut changer, ça serait même bien, mais il faudrait traiter tout le fichier, pas juste un test.

yweber commented

2024-04-09 16:31:15 +02:00

Les tests sont prévus pour être lancés dans leur intégralité par fichier; ça peut changer, ça serait même bien,

Ok ! Donc il vaut mieux que j'évite de les lancer avec plusieurs processus ? (c'est comme ça que je suis tombé sur l'erreur)

mais il faudrait traiter tout le fichier, pas juste un test.

Je n'avais pas idée que le problème était aussi présent ailleur, je n'ai pas eu d'autres tests qui partaient en erreur (sûrement vu que j'ai toujours lancé les tests avec un même nombre de processus, l'ordre des tests devaient être toujours le même)

> Les tests sont prévus pour être lancés dans leur intégralité par fichier; ça peut changer, ça serait même bien, Ok ! Donc il vaut mieux que j'évite de les lancer avec plusieurs processus ? (c'est comme ça que je suis tombé sur l'erreur) > mais il faudrait traiter tout le fichier, pas juste un test. Je n'avais pas idée que le problème était aussi présent ailleur, je n'ai pas eu d'autres tests qui partaient en erreur (sûrement vu que j'ai toujours lancé les tests avec un même nombre de processus, l'ordre des tests devaient être toujours le même)

yweber force-pushed wip/89318-fix-test-fields-test-page from cebebe0693 to eb862fc7f2

2024-04-09 16:38:03 +02:00

Compare

fpeters commented

2024-04-09 16:40:18 +02:00

Donc il vaut mieux que j'évite de les lancer avec plusieurs processus

On peut mais c'est pour ça qu'il y a --dist loadfile, que les processus soient distribués selon les fichiers.

> Donc il vaut mieux que j'évite de les lancer avec plusieurs processus On peut mais c'est pour ça qu'il y a --dist loadfile, que les processus soient distribués selon les fichiers.

yweber commented

2024-04-09 16:45:31 +02:00

On peut mais c'est pour ça qu'il y a --dist loadfile, que les processus soient distribués selon les fichiers.

Du coup ça serait pas une bonne idée que j'ajoute --dist loadfile dans tox.ini dans un second commit ?

> On peut mais c'est pour ça qu'il y a --dist loadfile, que les processus soient distribués selon les fichiers. Du coup ça serait pas une bonne idée que j'ajoute `--dist loadfile` dans tox.ini dans un second commit ?

fpeters commented

2024-04-09 16:51:35 +02:00

Du coup ça serait pas une bonne idée que j'ajoute --dist loadfile dans tox.ini dans un second commit ?

Il y est, sinon ça échouerait quotidiennement dans jenkins.

(je n'utilise pas tox)

> Du coup ça serait pas une bonne idée que j'ajoute --dist loadfile dans tox.ini dans un second commit ? Il y est, sinon ça échouerait quotidiennement dans jenkins. (je n'utilise pas tox)

yweber changed title from ~~WIP: tests: add missing fixture to test_fields.test_page (#89318)~~ to tests: add missing fixture to test_fields.test_page (#89318)

2024-04-09 17:11:43 +02:00

fpeters approved these changes 2024-04-09 18:44:53 +02:00

fpeters merged commit eb862fc7f2 into main

2024-04-12 07:12:40 +02:00

fpeters deleted branch wip/89318-fix-test-fields-test-page

2024-04-12 07:12:40 +02:00

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#1386