[m-rev.] for review: rewrite rest of options_file.m
Julien Fischer
jfischer at opturion.com
Fri Jun 12 23:26:28 AEST 2020
On Fri, 12 Jun 2020, Zoltan Somogyi wrote:
> 2020-06-09 10:33 GMT+10:00 Julien Fischer<jfischer at opturion.com>:
>>>>> I couldn't find any. I do think there should be some, but I am not sure
>>>>> whether these should go in e.g. tests/hard_coded, or in a new subdirectory
>>>>> of tests dedicated to tests of option file handling.
>>>>
>>>> It should be a new subdirectory; there's enough complication in
>>>> hard_coded as it is.
>>>
>>> Agreed. However, we will need tests for both (a) correct options files,
>>> and (b) incorrect options files. I think these belong in two separate
>>> directories. Does anyone object to naming these "options_file"
>>> and "invalid_options_file" respectively?
>>
>> Not from me.
>
> The attached updated log and diff extend the ones in my
> original email to add both of these new test directories.
> You may want to have a look at the new parts.
Done; they look fine.
Julien.
More information about the reviews
mailing list