Remove a couple of non-test tests - syntax conformance

[eileenmcnaughton]

Overview

Remove a couple of non-test tests - syntax conformance

Before

of the two tests removed

• one does an early return and no testing
• maybe the other is useful - but it isn't testing what it says it is - since lack of $params array is no longer an error - it will instead be failing on no mandatory fields - which I feel is tested elsewhere

After

poof

Technical Details

SyntaxConformanceTests used to be really helpful - I feel like we now run too many for their value & they take a long time so a bit of a cull might be in order -

Comments

Actually this one is probably more pointless

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[seamuslee001]

@eileenmcnaughton I guess this is just testing that an APIv3 Exception or similar is thrown if no params are passed in right?

[eileenmcnaughton]

yeah - I think it would be a mandatory field validation error - perhaps it is of some value - but it just feels like we have too many of these tests now