Recover commented out code in tests and mark with pytest.mark.skip instead

[DanielYang59]

Summary

• Recover commented out code in tests and mark with pytest.mark.skip instead (with TODO: need someone to fix this message for easy search), though currently ERA is ignored:

  pymatgen/pyproject.toml

  Line 192 in c06abf1

  "ERA", # Check for commented-out code

Rationale

Commented out code ages badly if not covered by any linter, perhaps we should mark them with pytest.mark.skip instead until someone come and fix them (hopefully).

[DanielYang59]

Looks like there isn't more commented out code in tests (searched with #\s+def and #\s+class , using \s+ in case multiple spaces are inserted).

Also we only have 104 lines of code commented in tests (might include a few false positive), should be possible to enable ERA for tests at some point.

[DanielYang59]

Thanks!