[BEAM-2223] Do not register TestSparkPipelineOptions to avoid hard hamcrest dep

[kennknowles]

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

• Make sure the PR title is formatted like:
  [BEAM-<Jira issue #>] Description of pull request
• Make sure tests pass via mvn clean verify.
• Replace <Jira issue #> in the title with the actual Jira issue
  number, if there is one.
• If this contribution is large, please file an Apache
  Individual Contributor License Agreement.

---

We have an undocumented requirement that no subclass of TestPipelineOptions can be registered. We do not indicate a hard dependency on hamcrest, but test options has one and it will crash on registration (or classloading in general) if the user does not provide it.

I see this change as a short term solution, making the Spark registrar analogous to all the other runners. It allows Java 8 examples to get past the point of registration.

[kennknowles]

run spark validatesrunner

[kennknowles]

R: @davorbonaci

[kennknowles]

CC: @aaltay

[davorbonaci]

LGTM

[kennknowles]

This actually will not work, due to how PipelineOptions and TestPipeline interact. Since it will not work as-is, I will close this and do all revision on #3002 which contains the commits in this PR.