Fix test-runner bootstrap failing if apt lock is already held

[MarkusPettersson98]

Seems to be working: https://github.com/mullvad/mullvadvpn-app/actions/runs/10616373857

---

This change is 

[linear]

DES-869 Make test runners using `apt` wait for `dpkg` lock if necessary

[dlon]

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @MarkusPettersson98)

---

test/scripts/ssh-setup.sh line 132 at r1 (raw file):

    # held, which happens sporadically. It is fine to wait for
    # some time if it means that the test run can continue.
    apt -o DPkg::Lock::Timeout=60 "$@"

We should probably also wait for the lock when installing the app (in test-runner).

[MarkusPettersson98]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @dlon)

---

test/scripts/ssh-setup.sh line 132 at r1 (raw file):

Previously, dlon (David Lönnhager) wrote…

We should probably also wait for the lock when installing the app (in test-runner).

That's probably a good idea. Working on it

[dlon]

Reviewable status: complete! all files reviewed, all discussions resolved

[dlon]

Reviewed 1 of 1 files at r3, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @MarkusPettersson98)

---

test/test-runner/src/package.rs line 155 at r3 (raw file):

    if purge {
        action = "apt --purge";
        cmd.args(["--purge", name]);

I think this should be purge now, not --purge.

[dlon]

Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @MarkusPettersson98)

---

test/test-runner/src/package.rs line 152 at r3 (raw file):

async fn uninstall_apt(name: &str, env: HashMap<String, String>, purge: bool) -> Result<()> {
    let action;
    let mut cmd = Command::new("/usr/bin/apt");

Can DPkg::Lock::Timeout be set here too?

[MarkusPettersson98]

Reviewable status: 1 of 2 files reviewed, 2 unresolved discussions (waiting on @dlon)

---

test/test-runner/src/package.rs line 152 at r3 (raw file):

Previously, dlon (David Lönnhager) wrote…

Can DPkg::Lock::Timeout be set here too?

Done

---

test/test-runner/src/package.rs line 155 at r3 (raw file):

Previously, dlon (David Lönnhager) wrote…

I think this should be purge now, not --purge.

Done

[dlon]

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