Feature: Macros handling to mbed test

[screamerbg]

This PR introduces 3 related features:

• Add support for macros handling to mbed test
• Pass macros only to mbed-os/tools/test.py and not to mbedgt to enable workflow where mbed test ... -D SOME_MACRO will both compile the tests with the macro and also run mbed greentea without getting the no such option: -D error.
• Add macro MBED_TEST_MODE to mbed test to allow the main application to exclude main(). See https://github.com/ARMmbed/pelion-ready-example/blob/master/main.cpp#L18

[bridadan]

EDIT: oh dear, this is a bad idea, please ignore. Thanks @theotherjimmy

nitpick/simplification: You can simplify this block by altering the function:

    def get_macros(self, macros=[]):

This allows you to remove this line

Suggested change

macros = more_macros or []

[theotherjimmy]

@bridadan please don't suggest this. https://docs.python-guide.org/writing/gotchas/#mutable-default-arguments

[theotherjimmy]

That being said, please change False to None, as it's the pythonic "value is missing" value.

[bridadan]

Yikes, thanks for the heads up. I learned something today.

[screamerbg]

@bridadan see Jimmy's comment and also the extra commit. I had it before the way you suggested :)

[bridadan]

If you want to add to the macros that could be found in MACROS.txt file, then you'd want something like this:

Suggested change

	macros = f.read().splitlines()
	macros.extend(f.read().splitlines())

Otherwise the supplied macros from the function arguments will be dropped.

[screamerbg]

Good catch. I'll change it later.

[theotherjimmy]

Suggested change

	def test_(toolchain=None, target=None, macro=False, compile_list=False, run_list=False, compile_only=False, run_only=False, tests_by_name=None, source=False, profile=False, build=False, clean=False, test_spec=None, app_config=None, test_config=None, coverage=None, make_program=None, new=None, generator=None, regex=None, unittests=None, build_data=None, greentea=None, icetea=None):
	def test_(toolchain=None, target=None, macro=False, compile_list=False,
	run_list=False, compile_only=False, run_only=False, tests_by_name=None,
	source=False, profile=False, build=False, clean=False, test_spec=None,
	app_config=None, test_config=None, coverage=None,
	make_program=None, new=None, generator=None, regex=None,
	unittests=None, build_data=None, greentea=None, icetea=None):

[theotherjimmy]

@screamerbg Do you have time to make the suggested changes? would you like me to help make them?

[bridadan]

Needs the update to the MACROS.txt I mentioned.

Updated macros.txt code.