Skip to content

Use a consistent container CLI path in integration tests#1621

Merged
katiewasnothere merged 1 commit into
apple:mainfrom
zdk:fix/1602-consistent-cli-path
Jun 2, 2026
Merged

Use a consistent container CLI path in integration tests#1621
katiewasnothere merged 1 commit into
apple:mainfrom
zdk:fix/1602-consistent-cli-path

Conversation

@zdk

@zdk zdk commented May 31, 2026

Copy link
Copy Markdown
Contributor

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

Motivation and Context

See #1602

Testing

  • Tested locally
  • Added/updated tests
  • Added/updated docs

Standardize on bin/container:
- Export CONTAINER_CLI_PATH=$(ROOT_DIR)/bin/container in the integration
  and coverage-integration targets.
- Default the test helper's executablePath to bin/container, dropping the
  .build discovery and the unused binaryAttributesNotFound error case.

Fixes apple#1602
@jglogan jglogan requested a review from katiewasnothere June 1, 2026 17:02
@github-actions

github-actions Bot commented Jun 1, 2026

Copy link
Copy Markdown

Code Coverage

Tier Line Coverage
Unit 34.19%
Integration 19.73%
Combined 53.33%

@katiewasnothere katiewasnothere merged commit 34cff59 into apple:main Jun 2, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants