fixes for vscode test parser

[cuza]

Making VSCode test parser happy

[nemacysts]

@cuza what is the current code doing to make VSCode unhappy? is it crashing or something on creating the spark config?

(i've also CC'd @CaptainSame since they recently touched this code)

[cuza]

@nemacysts

==================================== ERRORS ====================================
______________ ERROR collecting tests/cli/test_cmds_spark_run.py _______________
tests/cli/test_cmds_spark_run.py:513: in <module>
    @mock.patch.object(spark_config.SparkConfBuilder(), "get_history_url", autospec=True)
E   AttributeError: module 'service_configuration_lib.spark_config' has no attribute 'SparkConfBuilder'
!!!!!!!!!!!!!!!!!!! Interrupted: 1 errors during collection !!!!!!!!!!!!!!!!!!!!
=========================== 1 error in 2.02 seconds ============================

[mattmb]

Looks minor so I'll merge if the tests pass in a mo rather than closing

[nemacysts]

@cuza do we still need/want this?