Revive `test_Far_GetConfig()` in macrotest.lua.

Open MKadaner opened this issue 2 years ago • 4 comments

Far Manager version

3.0.6099.0

OS version

N/A

Other software

No response

Steps to reproduce

Uncomment the test and run ci pipeline.

Expected behavior

The test passes.

Actual behavior

The test fails.

Feb 01 '23 07:02 MKadaner

@shmuz would you happen to remember why it was commented in 6b808a110570692552a1a722ae07f5a94124a518?

Feb 01 '23 18:02 alabuzhev

AFAICR I did it because it became a boring task to regularly follow after the frequent changes (deletions, additions, renames) in the configuration variables. Yes I accepted that those changes (as opposed to API changes) were legal but it was too boring.

Feb 01 '23 19:02 shmuz

Thanks, I guess it's better to leave it as is then.

Feb 01 '23 19:02 alabuzhev

I respectfully disagree. The test as it is, is a dead code. It should either be exhumated or exterminated.

Feb 02 '23 04:02 MKadaner