Fix #69, #71, #72 - Githubissues

aaaaalbert commented 8 years ago

Issue #69 suggested to replace optparse with argparse. Issue #71 suggested to make this the module test sequence: setup -- spawn subprocess -- run test cases -- tear down subprocess -- shutdown. Issue #72 requested to allow for multiple files to be speficied for the -f command-line option.

aaaaalbert commented 8 years ago

@rf1591, please review!

rayfok commented 8 years ago

Looks good to me! Multiple file testing will definitely be useful.

I will test various modules with the updated utf.py. So far, I have gotten the same pass/fails for repy_v2.

On Tuesday, July 19, 2016, aaaaalbert notifications@github.com wrote:

@rf1591 https://github.com/rf1591, please review!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/SeattleTestbed/utf/pull/73#issuecomment-233668230, or mute the thread https://github.com/notifications/unsubscribe-auth/ANcxEb_OJvsxDItMOW2sJIpaT4z7FX1Iks5qXOvxgaJpZM4JP2Jd .

Raymond Fok New York University Tandon School of Engineering BS Computer Science '19

aaaaalbert commented 8 years ago

Report whether the module's unit tests run unchanged.
Test additional cases manually. (For instance, I don't know if the utf unit tests check that -a really runs setup/subprocess/shutdown scripts for all modules. touch a few files for made-up module names and see....)

SeattleTestbed / utf

Fix #69, #71, #72 #73