MAINT: Add test for `ase.io.castep.read_bands`
This MR add a test for ase.io.castep.read_bands
(cf. original implementation !1259 (merged)) together with a minor refactoring, hopefully increasing the test coverage a bit.
The original data used for the test is also attached castep.zip.
The pipeline error is coming from Appveyor which i turn on my forked repository due to the issue reported in !3346 (merged) and not relevant to this MR.
Checklist
-
I am familiar with ASE's contribution guidelines. -
Doc strings in code changed in this MR are up to date. -
Unit tests have been added for new or changed code. -
Issue is resolved via "closes #XXXX" if applicable.