Removing xfails.

[germa89]

The pytest.mark.xfail should be used when the implementation is not correct, or similar. So eventually, this will get fixed.

XFAIL should not be used as a default when the test raises an exception because in that case, the implementation is behaving as expected and it should be a PASS.

[germa89]

Requesting so many reviewers just to validate my thoughts about XFAIL.

[codecov]

Codecov Report

Merging #2216 (c8daeca) into main (2221a7b) will increase coverage by 0.79%.
Report is 62 commits behind head on main.
The diff coverage is 94.25%.

@@            Coverage Diff             @@
##             main    #2216      +/-   ##
==========================================
+ Coverage   86.43%   87.22%   +0.79%     
==========================================
  Files          45       45              
  Lines        7971     8245     +274     
==========================================
+ Hits         6890     7192     +302     
+ Misses       1081     1053      -28     

[RobPasMue]

Agreed - avoid using xfail if possible, good approach @germa89