entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

statistics: always consider workflow of same formdef (#84553) #909

Merged
fpeters merged 1 commits from wip/84553-test_statistics_multiple_forms_count into main 2023-12-11 13:45:59 +01:00
Conversation 2 Commits 1 Files Changed 1 +1 -1
fpeters commented 2023-12-11 13:17:06 +01:00
Owner
Copy Link
No description provided.
fpeters added 1 commit 2023-12-11 13:17:07 +01:00
gitea/wcs/pipeline/head There was a failure building this commit Details
e710716782 statistics: always consider workflow of same formdef (#84553)
fpeters changed title from WIP: statistics: always consider workflow of same formdef (#84553) to statistics: always consider workflow of same formdef (#84553) 2023-12-11 13:25:16 +01:00
fpeters commented 2023-12-11 13:26:51 +01:00
Author
Owner
Copy Link

À noter que je me suis embrouillé sur une fausse piste autour de :

workflow.add_status(name='Just submitted', id='just_sumbitted')

(noter la typo dans just_suMBitted) (mais je ne corrige pas ça là parce que des tests dépendent de l'erreur)

À noter que je me suis embrouillé sur une fausse piste autour de : ``` workflow.add_status(name='Just submitted', id='just_sumbitted') ``` (noter la typo dans just_suMBitted) (mais je ne corrige pas ça là parce que des tests dépendent de l'erreur)
vdeniaud approved these changes 2023-12-11 13:40:23 +01:00
vdeniaud left a comment
Owner
Copy Link

OK pas facile à comprendre en effet, il y a dans le lot un formdef avec le workflow par défaut créé implicitement et donc avec un statut just_submitted bien orthographié.

Et le commentaire dans les tests # filter on status exclusive to one formdef is ignored est faux, c'est un comportement aléatoire qui dépend de l'ordre.

OK pas facile à comprendre en effet, il y a dans le lot un formdef avec le workflow par défaut créé implicitement et donc avec un statut just_submitted bien orthographié. Et le commentaire dans les tests `# filter on status exclusive to one formdef is ignored` est faux, c'est un comportement aléatoire qui dépend de l'ordre.
fpeters commented 2023-12-11 13:45:55 +01:00
Author
Owner
Copy Link

Je passe ça ainsi pour avoir un build clean, on pourra voir pour nettoyer un peu commentaire/typo après.

Je passe ça ainsi pour avoir un build clean, on pourra voir pour nettoyer un peu commentaire/typo après.
fpeters merged commit e710716782 into main 2023-12-11 13:45:59 +01:00
fpeters deleted branch wip/84553-test_statistics_multiple_forms_count 2023-12-11 13:45:59 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
vdeniaud
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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#909
No description provided.
Delete Branch "wip/84553-test_statistics_multiple_forms_count"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?