Ask Your Question
1

What tests are useful while reviewing a patch if the patchbot report is already positive?

asked 2013-09-05 11:04:42 -0600

twch gravatar image

updated 2015-01-14 04:52:40 -0600

FrédéricC gravatar image

Hi,

the sage developer guide lists several tests that should be performed manually during the review process, like a complete doctest, building the documentation and checking coverage. Are all these test necessary and useful, if the patch bot already reported, that the patch has been positively tested against the latest pre-release? Or can some be simply skipped because they are already checked by the bot? What does the bot actually check?

edit retag flag offensive close merge delete

1 answer

Sort by » oldest newest most voted
3

answered 2013-09-05 15:01:07 -0600

kcrisman gravatar image

You can always do them again - just because the bot is happy doesn't mean a different platform/chip won't have different results. Especially ones in the directories touched by the patch.

Also, tests other than the ones in the doc are the most useful! Use your imagination.

edit flag offensive delete link more

Your Answer

Please start posting anonymously - your entry will be published after you log in or create a new account.

Add Answer

Question Tools

Stats

Asked: 2013-09-05 11:04:42 -0600

Seen: 301 times

Last updated: Sep 05 '13