← Back to issue list

tests: add the missing quote on autopkgtest run

View original Github issue

Metadata

Project
snapcraft
Number
#1756
Type
pull request
State
merged
Author
come-maiz
Labels
Created
Updated
Closed

Current evaluation

Merged a single-line fix adding a missing quote to the autopkgtest run command. Approved and passed CI, resolving a syntax error that broke test execution.

Suggested action:

No scores available.

Issue body

- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)? - [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)? - [x] Have you successfully run `./runtests.sh static`? - [x] Have you successfully run `./runtests.sh unit`? ----- I must have touched backspace by mistake when I was moving the suite to the nightly, or something, because this worked here: https://travis-ci.org/snapcore/snapcraft/jobs/302748980 I'm sorry. I added the quote, and now I can run this locally: http://paste.ubuntu.com/26029313/ Hopefully travis will not give us any other surprises.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged a single-line fix adding a missing quote to the autopkgtest run command. Approved and passed CI, resolving a syntax error that broke test execution.
qwen3.6-35b-a3b-mtp-q6 Merged a fix for a missing quote in the autopkgtest run command. The syntax error was accidentally introduced during a suite migration. Local and Travis tests now pass successfully.
qwen3.6-35b-a3b-mtp-q6 Merged a fix for a missing quote in the autopkgtest run command. The character was accidentally removed during a suite migration, verified locally, and successfully integrated.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1716 tests: split the integration autopkgtests snapcraft merged Merged after splitting integration autopkgtests to eliminate timeout failures. The change adds minor per-test setup overhead but resolves recurring execution timeouts. Approved by two reviewers with passing CI checks.
72%
#2188 tests: support running spread suite in autopkgtests snapcraft merged Merged. Adds an autopkgtest suite to run the spread test suite during releases. CI failures were resolved by fixing missing build dependencies and temporarily disabling armhf execution. Approved by one reviewer and merged.
71%
#1162 tests: pass the autopkgtest secret to the container snapcraft merged Merged a one-line change to pass the autopkgtest secret to the container, resolving Launchpad bug #1668318. Approved by one reviewer.
71%
#293 Add the unit tests to the autopkgtest suite. snapcraft merged Merged changes adding unit tests to the autopkgtest suite. The update modified two files with six added lines, was approved by sergiusens, and resolved launchpad bug LP: #1541421.
71%
#3494 tests: use candidate for autopkgtests snapcraft closed Closed without merging. The single-line update to use candidate for autopkgtests was abandoned without review or CI validation.
71%
#2018 tests: minor fixes to autopkgtests snapcraft merged Merged minor autopkgtest fixes by sergiusens. The changes updated 4 files with 17 additions and 3 deletions to correct test configurations.
70%