Remove test dependency on sumtypes library

[EliahKagan]

Fixes #1797

The WinBashStatus class in test_index.py, introduced in #1745, had used the sumtypes library to express the various cases relevant to xfail markings and troubleshooting and, where applicable, to express some associated debugging information. This includes the Inapplicable case, so sumtypes was used on all systems, not just Windows, which also usefully allowed the other cases to be inspected dynamically on any system, and allowed xfail markings to be written without excess complexity.

However, that only used a small fragment of sumtypes functionality, which can be expressed with only minor inelegance with explicit data classes. This pull request makes that change and removes sumtypes as a test dependency.

Because the sumtypes dependency was apparently causing problems for downstream maintainers (per #1797), I believe this change is justified. I believe the small inelegance of less compact definitions, and of the case classes not having an inheritance relationship to WinBashStatus itself, are the only disadvantages of this changed approach. The actual logic, stored information, and available debugging information is the same, fully useful reprs are still produced, no code change is needed in any of the markings that use WinBashStatus, and the class is still defined on all systems.

Nonetheless I have retested the cases locally (since only two of them occur on CI), verifying that they still work and that the same debugging information remains available.

[Byron]

Thanks a lot for your help!

Looking at the change, I'd even think it's preferable over the 'magic' that was introduced as it's more explicit and on that merit probably more easily maintained.