Improve appearance_spec and deploy_key_spec

What does this MR do?

Improve tests

Are there points in the code the reviewer needs to double check?

No

Why was this MR needed?

In appearance_spec.rb we can use build instead create record in db. In deploy_key_spec.rb we can remove unused code.

• Tests
  • n/a Added for this feature/bug
  • All builds are passing
• Conform by the merge request performance guides
• Conform by the style guides
• Branch has no merge conflicts with master (if it does - rebase it please)
• Squashed related commits together

What are the relevant issue numbers?