cc_resolv_conf: introduce tests and stabilise output across Python versions

[OddBloke]

This is a follow-up to #144 which fixed the rendering behaviour.

While writing the tests, CI failed due to dict iteration differences across Python versions, so this also sorts output so that we will produce the same output across Python versions.

[OddBloke]

On Thu, Mar 12, 2020 at 02:50:40PM -0700, Ryan Harper wrote: raharper commented on this pull request. > + }, + "options spam foo:some_value bar:other_value", + ), + ], + ) + def test_flags_and_options( + self, m_write_file, params, expected_extra_line + ): + generate_resolv_conf("templates/resolv.conf.tmpl", params) + + expected_content = EXPECTED_HEADER + if expected_extra_line is not None: + # If we have any extra lines, expect a trailing newline + expected_content += "\n".join([expected_extra_line, ""]) + assert [ + mock.call(mock.ANY, expected_content, mode=mock.ANY) Should we assert that write_file got /etc/resolv.conf as the target for write_files ?

Done (in two additional tests).

[raharper]

Thanks Dan!