Tryton - Issues

 

Issue6278

Title Test selection methods of Selections and Reference fields
Priority feature Status resolved
Superseder Nosy List pokoli, reviewbot, roundup-bot
Type feature request Components trytond
Assigned To pokoli Keywords review
Reviews 26171002
View: 26171002

Created on 2017-02-16.13:00:16 by pokoli, last changed by roundup-bot.

Messages
New changeset 488104918c39 by Sergi Almacellas Abellana in branch 'default':
Add test for selection fields methods
http://hg.tryton.org/trytond/rev/488104918c39
review26171002 updated at https://codereview.tryton.org/26171002/#ps40001
review26171002 updated at https://codereview.tryton.org/26171002/#ps20001
New review26171002 at https://codereview.tryton.org/26171002/#ps1
msg31977 (view) Author: [hidden] (pokoli) (Tryton committer) Date: 2017-02-16.14:05:31
Here is review26171002

I run it with all the current modules and it passes for all the current definitions.
msg31975 (view) Author: [hidden] (pokoli) (Tryton committer) Date: 2017-02-16.13:00:15
Following issue6069 we can add a test that ensures the definition of selection fields. For instance methods this will ensure that all the fields dependencies are defined correctly. For classmethods it will ensure that they always return a valid list.
History
Date User Action Args
2017-02-23 09:27:56roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg32138
2017-02-21 09:57:43reviewbotsetmessages: + msg32086
2017-02-17 10:02:49reviewbotsetmessages: + msg31989
2017-02-16 14:40:43reviewbotsetnosy: + reviewbot
messages: + msg31978
2017-02-16 14:40:42reviewbotsetreviews: 26171002
keyword: + review
2017-02-16 14:05:31pokolisetstatus: in-progress -> testing
messages: + msg31977
2017-02-16 13:00:16pokolicreate

Showing 10 items. Show all history (warning: this could be VERY long)