[#228] Makes the type attribute required

Also fixes tests and adds some should fail tests
Merge pull request #241 from IATI/228-other-identifier-type

[#228] Makes the type attribute required