You were wondering if you had enough tests for this code, and I think you do...at least, I can't think of any additional useful tests that could add. All of the new attributes are simply strings that are copied into the snippet or XML, so it's not like there's any special behaviour for any of them that needs to get tested.