Skip to content

Skip GPU detection test in GitHub Actions#33

Merged
joone merged 2 commits intomainfrom
copilot/skip-failed-gpu-test-case
Feb 16, 2026
Merged

Skip GPU detection test in GitHub Actions#33
joone merged 2 commits intomainfrom
copilot/skip-failed-gpu-test-case

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 16, 2026

The GPU detection test fails in CI because lspci is unavailable in GitHub Actions runners.

Changes

  • Wrap GPU detection test with GITHUB_ACTIONS === false conditional
  • Test now skips in CI, runs locally where GPU info is accessible
  • Follows existing pattern used for other environment-specific tests (apache2, file size checks)
// Before: unconditional test
it("Detect GPUs on this system", function () {
  let stdout = execSync(
    `MOCHA_ENV=test node ${LOZ_BIN} "Detect GPUs on this system"`,
  ).toString();
  expect(stdout).to.include("VGA compatible controller");
});

// After: conditional on local environment
if (GITHUB_ACTIONS === false) {
  it("Detect GPUs on this system", function () {
    let stdout = execSync(
      `MOCHA_ENV=test node ${LOZ_BIN} "Detect GPUs on this system"`,
    ).toString();
    expect(stdout).to.include("VGA compatible controller");
  });
}

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: joone <1979160+joone@users.noreply.github.com>
Copilot AI changed the title [WIP] Skip or change failed GPU test case in Linux Command Test Skip GPU detection test in GitHub Actions Feb 16, 2026
Copilot AI requested a review from joone February 16, 2026 20:01
@joone joone marked this pull request as ready for review February 16, 2026 20:01
@joone joone merged commit cd89d41 into main Feb 16, 2026
0 of 3 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.

2 participants