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

tests: extend computed fields tests with list type (#87412) #1184

Merged
fpeters merged 2 commits from wip/87412-computed-field-test-lists into main 2024-02-26 11:23:14 +01:00
Conversation 5 Commits 2 Files Changed 2 +70 -12
fpeters commented 2024-02-23 23:05:03 +01:00
Owner
Copy Link

Les exemples proposés dans le ticket apparaissent fonctionner, il doit y avoir quelque chose d'autre dans le ticket client.

Les exemples proposés dans le ticket apparaissent fonctionner, il doit y avoir quelque chose d'autre dans le ticket client.
fpeters changed title from tests: extend computed fields tests with list type (#87412) to WIP: tests: extend computed fields tests with list type (#87412) 2024-02-23 23:08:20 +01:00
fpeters commented 2024-02-23 23:09:09 +01:00
Author
Owner
Copy Link

Le ticket client parle de liste mais les exemples donnés sont en fait des dictionnaires (?).

Le ticket client parle de liste mais les exemples donnés sont en fait des dictionnaires (?).
bdauvergne commented 2024-02-23 23:11:27 +01:00
Owner
Copy Link

Le ticket client parle de liste mais les exemples donnés sont en fait des dictionnaires (?).

La description du ticket parle bien de dictionnaires. Dans le ticket client en première opinion je me suis dit qu'il pouvait y avoir un souci avec une liste au final ce n'était pas le problème.

> Le ticket client parle de liste mais les exemples donnés sont en fait des dictionnaires (?). La description du ticket parle bien de dictionnaires. Dans le ticket client en première opinion je me suis dit qu'il pouvait y avoir un souci avec une liste au final ce n'était pas le problème.
fpeters force-pushed wip/87412-computed-field-test-lists from 6aaa10afed to 1f0d83fd5e 2024-02-23 23:14:16 +01:00 Compare
fpeters commented 2024-02-23 23:16:10 +01:00
Author
Owner
Copy Link

Je cite : « car la liste "data" qu'il retourne n'est pas interprété », mais quoiqu'il en soit j'ai ajouté un test pour correspondre et ça n'échoue toujours pas; il reste quelque chose de particulier dans le ticket client.

Je cite : « car la liste "data" qu'il retourne n'est pas interprété », mais quoiqu'il en soit j'ai ajouté un test pour correspondre et ça n'échoue toujours pas; il reste quelque chose de particulier dans le ticket client.
fpeters force-pushed wip/87412-computed-field-test-lists from 1f0d83fd5e to 7cbca19bb4 2024-02-23 23:36:32 +01:00 Compare
fpeters commented 2024-02-23 23:37:23 +01:00
Author
Owner
Copy Link

Et pour suivre une autre différence citée dans le ticket d'origine, les mêmes tests en POST (ils fonctionnent aussi).

Et pour suivre une autre différence citée dans le ticket d'origine, les mêmes tests en POST (ils fonctionnent aussi).
fpeters added 1 commit 2024-02-24 10:08:38 +01:00
fpeters commented 2024-02-24 10:10:42 +01:00
Author
Owner
Copy Link

La différence de l'appel qui échouait était l'envoi d'une donnée complexe (le fichier) vers le webservice, ça cassait la réception de données elles-mêmes complexes.

La différence de l'appel qui échouait était l'envoi d'une donnée complexe (le fichier) vers le webservice, ça cassait la réception de données elles-mêmes complexes.
fpeters changed title from WIP: tests: extend computed fields tests with list type (#87412) to tests: extend computed fields tests with list type (#87412) 2024-02-24 10:17:54 +01:00
bdauvergne approved these changes 2024-02-26 11:17:00 +01:00
fpeters merged commit aeb2d548af into main 2024-02-26 11:23:14 +01:00
fpeters deleted branch wip/87412-computed-field-test-lists 2024-02-26 11:23:14 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
bdauvergne
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#1184
No description provided.
Delete Branch "wip/87412-computed-field-test-lists"

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?