testrocket icon indicating copy to clipboard operation
testrocket copied to clipboard

remove old implements

Open maito1201 opened this issue 4 years ago • 1 comments

I think testrocket is really interesting idea. But there is some legacy code snippets. It seems to be left behind in the flow of improvement.

1. remove useless retun value

def _show(r); (TestRocket.out || STDERR) << r + "\n"; r end That variable "r" was used when testrocket had been started to invent. But it`s logic is changed, return value is not used now.

2. remove Proc.send

Proc.send :include, TestRocket now refine and using works enough. We don't have to load TestRocket module using send method.

maito1201 avatar Mar 31 '20 11:03 maito1201

did you run the tests after removing that "useless" return value?

zenspider avatar Dec 17 '23 22:12 zenspider