chore(metrics): revert dimensions test and split

[heitorlessa]

Resolves #1239

Summary

Changes

In #1240, I repurposed the original test test_exceed_number_of_dimensions to account for service dimension, since the outcome would be the same (SchemaValidationError).

This PR reverts that test change, fix an old typo (we we), and adds a specific test test_exceed_number_of_dimensions_with_service.

User experience

Please share what the user experience looks like before and after this change

Checklist

If your change doesn't seem to apply, please leave them unchecked.

• Meet tenets criteria
• I have performed a self-review of my this change
• Changes are tested
• Changes are documented
• PR title follows conventional commit semantics

Is this a breaking change?

RFC issue number:

Checklist:

• Migration process documented
• Implement warnings (if it can live side by side)

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

[codecov-commenter]

Codecov Report

Merging #1243 (1afa990) into develop (9d8dd1b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1243   +/-   ##
========================================
  Coverage    99.88%   99.88%           
========================================
  Files          119      119           
  Lines         5423     5423           
  Branches       618      618           
========================================
  Hits          5417     5417           
  Misses           2        2           
  Partials         4        4           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9d8dd1b...1afa990. Read the comment docs.