Skip to content

E2E tests#11

Merged
marinofelipe merged 15 commits intomainfrom
tests
Sep 22, 2025
Merged

E2E tests#11
marinofelipe merged 15 commits intomainfrom
tests

Conversation

@marinofelipe
Copy link
Contributor

@marinofelipe marinofelipe commented Sep 22, 2025

Introduce E2E tests that run spmgraph commands via Process.

TODO

  • Update lint test, once the other PRs are merged

@marinofelipe marinofelipe self-assigned this Sep 22, 2025
@marinofelipe marinofelipe requested a review from a team as a code owner September 22, 2025 08:50
@gyg-pr-tool gyg-pr-tool bot added ready-to-merge All required checks are passing risk:low large labels Sep 22, 2025
Base automatically changed from dependabot to main September 22, 2025 13:04
@gyg-pr-tool gyg-pr-tool bot removed the ready-to-merge All required checks are passing label Sep 22, 2025
Copilot AI review requested due to automatic review settings September 22, 2025 14:22
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Introduces comprehensive E2E tests for the spmgraph command-line tool that validate core functionality by running actual processes. The PR adds a dedicated test plan configuration, fixture package for testing, and comprehensive test coverage for all major commands.

Key changes:

  • Adds E2E test suite that runs spmgraph commands via Process to test real-world usage
  • Creates fixture package structure with targets and dependencies for testing
  • Implements test plan configuration with Address Sanitizer enabled for graphviz compatibility

Reviewed Changes

Copilot reviewed 19 out of 20 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
spmgraph.xctestplan Test plan configuration with environment variables and command-line arguments for E2E testing
Tests/SPMGraphExecutableTests/SPMGraphExecutableE2ETests.swift Comprehensive E2E test suite covering all major spmgraph commands
Tests/Fixtures/Package/* Test fixture package with targets, dependencies, and test files
Sources/SPMGraphVisualize/SPMGraphVisualize.swift Updates to support AbsolutePath for output file handling
Sources/Core/Extensions/ProcessInfo+Core.swift Helper extension to detect test environment
Package.swift Adds SPMGraphExecutableTests target
.github/workflows/pull-request.yml Updates CI to use xcodebuild with test execution

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@marinofelipe marinofelipe merged commit 3593433 into main Sep 22, 2025
5 checks passed
@marinofelipe marinofelipe deleted the tests branch September 22, 2025 23:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants