issue #583 add StorageMethod enum

[sosnovsky]

This PR replaces boolean values for passphrases storage with StorageMethod enum

close #583

---

Tests:

• Tests added or updated - updated PassPhraseStorageTests to use new storageMethod param

---

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

• addresses the issue it closes (if any)
• code is readable and understandable
• is accompanied with tests, or tests are not needed
• is free of vulnerabilities
• is documented clearly and usefully, or doesn't need documentation

[tomholub]

Looks good!

[sosnovsky]

@tomholub new builds are failing on semaphore with such error log

fatal: Unable to read current working directory: No such file or directory
fatal: fetch-pack: invalid index-pack output
bash: cd: /Users/semaphore/git/flowcrypt-ios: No such file or directory

Do you know possible reason? Maybe some configs were changed?

[tomholub]

@sosnovsky please write them on support - checkout is a semaphore cli command that is supposed to work. So the problem is in their systems this time. https://semaphoreci.com/contact - send them a link to the failing pipeline.

[Kharchevskyi]

@tomholub @sosnovsky it looks that job for TypeScript - Core standalone tests are not configured properly.
Checkout operation performs twice, once inside prologue

prologue:
        commands:
          - checkout

and second time in lane

- name: TypeScript - Core standalone tests
          commands:
            - sem-version node 12
            - checkout

Are there any reasons to do this twice?

[tomholub]

I don't think there is any reason for doing it twice, checkout in prologue should be enough.

[tomholub]

@Kharchevskyi thank you - that was it! 👍

[Kharchevskyi]

i think this PR can be closed #620