Fixes Pylint E1120 (no-value-for-parameter) violations and removes global ignore

[matt-graham]

Part of #1181.

Removes if __name__ == "__main__" blocks from tests/test_enhanced_lifestyle.py and tests/test_rti.py test modules which call out to test functions to allow running the modules as scripts. In both cases the call signatures of the test functions called have changed and so running as a script would have raised an error. As in both files this has been the case for a while, I suspect this means no-one is regularly using this approach for running tests, and in general using pytest should be preferred anyway, so I've just removed these blocks altogether rather than trying to fix. We probably also want to update the wiki to remove the example test module skeleton to remove the corresponding if __name__ == "__main__" block.

Also refactors test_scale_run_script_deterministic in tests/test_determinism.py to avoid a Pylint E1120 false positive on unpacking final_population_dataframes list in to call to pandas.testing.assert_frame_equal which appears to be because it cannot determine the length of the list and so whether both the required left and right arguments to pandas.testing.assert_frame_equal will be passed values. An error message was previously being constructed alongside the call to pandas.testing.assert_frame_equal but nothing was actually being done with it, presumably as I initially wrote this as a plain assert statement intending the error message to be the optional message component of the statement, but then changed to using the pandas function instead at some point. The code has now been refactored to use a plain assert with pandas.DataFrame.equals which both avoids the false positive and ensures the error message will be correctly displayed.