Remove unnecessary do_test_pending/do_test_finished pairs in test_attachment.js and test_detectAttachmentCharset.js as async_run_tests already manages this for us, and doesn't like the fact that bug 863311 added optional parameters for them. r=unit-test-only-bustage-fix