Enable greentea testing on CM3DS_MPS2 target

[matetothpal]

Set copy_method and reset_method in targets.json which is needed for
htrun.

Description

Enable greentea testing on CM3DS_MPS2 target.
Set copy_method and reset_method in targets.json which is needed for htrun.

Status

READY/IN DEVELOPMENT/HOLD

Migrations

If this PR changes any APIs or behaviors, give a short description of what API users should do when this PR is merged.

NO

Related PRs

Support reset and flash for ARM MPS2 board.: ARMmbed/htrun#160

branch	PR
other_pr_production	link
other_pr_master	link

Todos

• Tests
• Documentation

Deploy notes

Notes regarding the deployment of this PR. These should note any
required changes in the build environment, tools, compilers, etc.

Steps to test or reproduce

Outline the steps to test or reproduce the PR here.

[bridadan]

LGTM, this shouldn't require morph testing

[AnotherButler]

@mattot01 Thanks for the PR.

Also, we recommend our contributors follow Chris Beam’s seven rules of great commit messages to keep the commit history clear. We find the commit.template feature particularly helpful.

To match this format, please delete the period at the end of your subject line. (We use merged PRs in our release notes.) Thanks for your contribution.

[matetothpal]

@AnotherButler Thank you for the suggestions, I updated the commit message.

[bridadan]

@0xc0170 I believe this should be ok without further testing. Nothing that morph runs will extend the coverage in this case.

[theotherjimmy]

@bridadan What do these keys do? How does htrun pick them up?

[bridadan]

Greentea will do its best to find the targets.json file to find out information about the targets. It can then see if it has to do anything target specific to run tests on it (change its flashing strategy, change how long it waits for flashing, etc). This kind of target specific information is generally discouraged, however in this case there wasn't much choice since the target was already produced.

In this case, Greentea has to change how it flashes the binary and how it resets the device (since it doesn't support reset via the serial break).

[0xc0170]

@mattot01 Is there any other way to do this? This does not look right, to have these 2 in targets.json file. As @bridadan highlighted above, there's not much to do, is that true?

[bridadan]

@0xc0170 This is the correct way to do this for this platform. It doesn't contain a standard mbed interface so we have to modify the test tools behavior.

[theotherjimmy]

@bridadan I still don't like adding htrun specific info to targets.json. That being said, I'm not going to block it for that reason.

[bridadan]

@theotherjimmy agreed, there's just literally no other place to put it at the moment.