fix: backwards compatibility of NWMRequest

[aaraney]

Fix backwards compatible handling of NWMRequest config_data_id initializer argument.

dmod.communication -- 0.16.2

Changes

• Fix backwards compatible handling of NWMRequest config_data_id initializer argument.

[christophertubbs]

Either this or the unit tests are wrong - looks like model isn't coming in through data.

[aaraney]

@christophertubbs, nice catch, I just fixed it.

[christophertubbs]

Looks good.

Stylistic opinion:

Using an approach like:

if "request_body" not in data:
    if "model" in data:
        request_body = data["model"][self.get_model_name()]
    else:
        request_body = {}

    if config_data_id is not None:
        request_body['config_data_id'] = config_data_id

    data['request_body'] = {
        self.get_model_name(): request_body
    }

super().__init__(**data)

will have a single terminal point instead of two and might be a little easier to step through and predict. That's not important here since it's not exactly performing the most convoluted operation, but it is something that caught my eye.