> On Jan. 8, 2015, 8:03 p.m., Zameer Manji wrote: > > src/test/python/apache/aurora/client/cli/test_inspect.py, line 98 > > <https://reviews.apache.org/r/29696/diff/2/?file=813436#file813436line98> > > > > I'm confused, if we define the print function here aren't we just > > testing that our print function that we define here prints what we think it > > does?
There's more code being tested than just the print function, but this mock serves to capture the output that a user would see. (Well, sort of - this mock unfortunately includes logic.) > On Jan. 8, 2015, 8:03 p.m., Zameer Manji wrote: > > src/test/python/apache/aurora/client/cli/test_inspect.py, line 120 > > <https://reviews.apache.org/r/29696/diff/2/?file=813436#file813436line120> > > > > Do we even need to patch clusters here? AFAIK we don't act on the > > contents of the config so having valid clusters is not necessary. It's not needed, removed. - Bill ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29696/#review67279 ----------------------------------------------------------- On Jan. 8, 2015, 7:46 p.m., Bill Farner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29696/ > ----------------------------------------------------------- > > (Updated Jan. 8, 2015, 7:46 p.m.) > > > Review request for Aurora, Kevin Sweeney and Zameer Manji. > > > Bugs: AURORA-990 > https://issues.apache.org/jira/browse/AURORA-990 > > > Repository: aurora > > > Description > ------- > > In the course of this, i took a stab at cleaning up the unit test to remove > unnecessary mocking and avoid writing to a temporary file. The nice outcome > of the mock removal is that the job configuration in our code is identical to > what would appear in a configuration file. > > > Diffs > ----- > > src/main/python/apache/aurora/client/cli/__init__.py > 395819fdf24b7919b32be51060fb5b581c8e1514 > src/main/python/apache/aurora/client/cli/jobs.py > 744d53405081e75b9aedcdcb7e4a823a2e0e743f > src/test/python/apache/aurora/client/cli/test_inspect.py > 1c07d912c95eafa292d5451f4d6c72e9a711dae0 > > Diff: https://reviews.apache.org/r/29696/diff/ > > > Testing > ------- > > > Thanks, > > Bill Farner > >
