Remove "it should" syntax in failure tests
Remove legacy-facing Failure#[] method

Also deletes the legacy tests that fail without this backward compatibility
Make Failure instance attributes readonly