fix(@opentelemetry/exporter-collector-grpc) regression from #2130 when host specified without protocol

[lizthegrey]

Which problem is this PR solving?

• regression from chore: url validation & README to prevent gRPC footguns. #2130: passing host:port fails because host is used as the URL protocol instead of as a hostname.

Short description of the changes

• Adds table tests for validateAndNormalizeUrl and corrects issues that prevented them from passing.

[codecov]

Codecov Report

Merging #2322 (afaf085) into main (11719ed) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2322   +/-   ##
=======================================
  Coverage   92.77%   92.77%           
=======================================
  Files         145      145           
  Lines        5221     5221           
  Branches     1070     1070           
=======================================
  Hits         4844     4844           
  Misses        377      377           

[lizthegrey]

browser-tests appears to be a flake

[naseemkullah]

lgtm, but I think it would be best to follow the assertion patterns seen in other tests, I left some (untested 😬) comments to adhere to that, this also would mean removing the added chai packages (furthermore i think the hoisted eslint packages are used so probably no need to add them either)

[vmarchaud]

lgtm, i agree with @naseemkullah suggestions though

[lizthegrey]

lgtm, but I think it would be best to follow the assertion patterns seen in other tests, I left some (untested ) comments to adhere to that, this also would mean removing the added chai packages (furthermore i think the hoisted eslint packages are used so probably no need to add them either)

I'll remove the chai packages -- I checked and they were already present as indirect deps but I didn't realize they were only used for tsconfig-paths and not for our stuff.

wrt adding the hoisted eslint -- it makes it easier to have eslint in there so I can run yarn test without pulling all the parent stuff when doing local dev, will defer to what's local project style though.

[lizthegrey]

PTAL

[dyladan]

I don't have permission to update your branch for merge

[lizthegrey]

updated branch.