Update version checking to warn that version is deprecated and error if there is no .serviceconfig version

[dixonjoel]

• This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?

Fixes #908
This gives a warning if the 'version' parameter of MeasurementService is used. It also errors if there is a 'version' parameter, but no 'version' in the .serviceconfig file. The rule is if you pass a 'version' parameter, you must have the same 'version' in the .serviceconfig file.

Why should this Pull Request be merged?

Gives clearer guidance on the latest state of the version parameters.

What testing has been done?

Existing tests plus added one test to verify a RuntimeError in the case where there is a 'version' parameter passed but no version in the .serviceconfig file.

[github-actions]

Test Results

    40 files  ± 0      40 suites  ±0   50m 52s ⏱️ + 2m 39s
   694 tests + 5     694 ✅ + 6      0 💤 ±0  0 ❌  - 1 
16 860 runs  +70  15 790 ✅ +71  1 070 💤 ±0  0 ❌  - 1 

Results for commit 72efbd5. ± Comparison against base commit 10328bb.

This pull request removes 2 and adds 7 tests. Note that renamed tests count towards both.

tests.unit.test_service ‑ test___service_config___create_measurement_service___service_info_matches_service_config[example.v1.serviceconfig--provided_interfaces1-provided_annotations1]
tests.unit.test_service ‑ test___service_config___create_measurement_service_with_version___version_is_used[example.v1.serviceconfig-2.0.1]

tests.acceptance.test_client_generator ‑ test___non_streaming_measurement___create_client___render_without_mypy_error
tests.unit.test_measurement_client ‑ test___enum_annotations___validate_and_transform_enum_annotations___returns_expected_enum_annotations[{"DC Volts": 0, "2-Wire Resistance": 1, "5 1/2": 2}-{"DC_Volts": 0, "k_2_Wire_Resistance": 1, "k_5_1_2": 2}]
tests.unit.test_measurement_client ‑ test___enum_annotations___validate_and_transform_enum_annotations___returns_expected_enum_annotations[{"NONE": 0, "RED": 1, "GREEN": 2}-{"NONE": 0, "RED": 1, "GREEN": 2}]
tests.unit.test_measurement_client ‑ test___invalid_enum_annotations___validate_and_transform_enum_annotations___raises_invalid_enum_value_error
tests.unit.test_service ‑ test___service_config___create_measurement_service___service_info_matches_service_config[example.v1.serviceconfig-1.0.1-provided_interfaces1-provided_annotations1]
tests.unit.test_service ‑ test___service_config___create_measurement_service_with_version___service_config_has_no_version
tests.unit.test_service ‑ test___service_config___create_measurement_service_with_version___version_is_used[example.v1.serviceconfig-1.0.1]

♻️ This comment has been updated with latest results.

[nick-beer]

LGTM - assuming someone else will grant explicit approval for these changes.

[bkeryan]

These tests are generating warnings:

============================== warnings summary ===============================
tests/unit/test_service.py::test___service_config___create_measurement_service_with_version___version_differs_from_service_config
tests/unit/test_service.py::test___service_config___create_measurement_service_with_version___service_config_has_no_version
tests/unit/test_service.py::test___service_config___create_measurement_service_with_version___version_is_used[example.serviceconfig-1.0.1]
tests/unit/test_service.py::test___service_config___create_measurement_service_with_version___version_is_used[example.v1.serviceconfig-1.0.1]
  D:\a\measurement-plugin-python\measurement-plugin-python\packages\service\ni_measurement_plugin_sdk_service\measurement\service.py:222: DeprecationWarning: The 'version' constructor parameter is deprecated. Specify the version using the 'version' field in the .serviceconfig file instead.
    warnings.warn(

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
- generated xml file: D:\a\measurement-plugin-python\measurement-plugin-python\packages\service\test_results\nims-windows-latest-py3.9-all-extras.xml -

Fixed in #942