Endpoints need to check `...` is empty

[dsweber2]

It's possible to hand non-existent parameters to the endpoints because ... isn't checked. The following call should throw an error for example, but doesn't:

epidatr::pub_covidcast(
            source = "chng",
            signals = "smoothed_adj_outpatient_covid",
            geo_type = "state",
            time_type = "day",
            geo_values = "*",
            time_values = epidatr::epirange(from = "20170101", to = "20240101"),
            issues = "*",
            fetch_params = fetch_params_list(return_empty = TRUE, timeout_seconds = timeout_seconds)
          )

since it should be fetch_args and fetch_args_list

[dsweber2]

the function to check this is rlang::check_dots_empty()

[nmdefries]

@dsweber2 Not all endpoints have .... Should I add it and the check to all of them to force args to bind by name?

[dsweber2]

That seems like a reasonable idea for consistency's sake. Otoh, we don't have arguments that are usually easily confused for one another though.

[nmdefries]

Okay, then let's not add dots everywhere right now.