Add commit_hash to end to end test

[mojganii]

This PR enables iOS end-to-end tests to receive a commit_hash as an input, allowing the workflow to check out that specific commit. This input is optional, and if no value is provided, the workflow will use the latest commit from the branch that triggered it. The primary reason for this change is that iOS settings migration tests require an older commit version. By reusing iOS end-to-end tests for these cases and consolidating workflows and jobs in one place, it becomes necessary to fetch the repository at a specific reference.

---

This change is 

[rablador]

Reviewed all commit messages.
Reviewable status: 0 of 1 files reviewed, 3 unresolved discussions (waiting on @mojganii)

---

.github/workflows/ios-end-to-end-tests.yml line 18 at r1 (raw file):

        type: string
        required: false
        

Remove extra spaces.

---

.github/workflows/ios-end-to-end-tests.yml line 85 at r1 (raw file):

        id: set-commit-hash
        run: |
          if [ -z "${{ inputs.commit_hash }}" ]; then

I think it makes more sense to check for the existence of something rather than the opposite. Ie:
if [ -n "${{ inputs.commit_hash }}" ]; then

---

.github/workflows/ios-end-to-end-tests.yml line 91 at r1 (raw file):

          fi
        shell: bash
        

Remove extra spaces.

[rablador]

Reviewed 1 of 1 files at r1.
Reviewable status: all files reviewed, 3 unresolved discussions (waiting on @mojganii)

[mojganii]

Reviewable status: 0 of 1 files reviewed, 3 unresolved discussions (waiting on @rablador)

---

.github/workflows/ios-end-to-end-tests.yml line 18 at r1 (raw file):

Previously, rablador (Jon Petersson) wrote…

Remove extra spaces.

that is intentional

---

.github/workflows/ios-end-to-end-tests.yml line 85 at r1 (raw file):

Previously, rablador (Jon Petersson) wrote…

I think it makes more sense to check for the existence of something rather than the opposite. Ie:
if [ -n "${{ inputs.commit_hash }}" ]; then

makes more sense

---

.github/workflows/ios-end-to-end-tests.yml line 91 at r1 (raw file):

Previously, rablador (Jon Petersson) wrote…

Remove extra spaces.

that is intentional

[mojganii]

Reviewable status: 0 of 1 files reviewed, 3 unresolved discussions (waiting on @rablador)

---

.github/workflows/ios-end-to-end-tests.yml line 18 at r1 (raw file):

Previously, mojganii wrote…

that is intentional

Removed!

---

.github/workflows/ios-end-to-end-tests.yml line 91 at r1 (raw file):

Previously, mojganii wrote…

that is intentional

Removed!

[buggmagnet]

Reviewable status: 0 of 1 files reviewed, 3 unresolved discussions (waiting on @rablador)

[rablador]

Reviewed 1 of 1 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved

Go home github, you're drunk