Skip to content

Conversation

@ssestak
Copy link
Contributor

@ssestak ssestak commented Nov 13, 2025

Summary

This PR adds a focused test suite for GraphQLAPIKit with 6 new tests, bringing the total from 16 to 22 tests.

Changes

Tests Added:

  • GraphQLAPIAdapterTests (5 tests):

    • testAdapterInitializationWithMinimalParameters
    • testAdapterInitializationWithNetworkTracer
    • testAdapterInitializationWithCustomHeaders
    • testAdapterInitializationWithCustomURLSessionConfiguration
    • testAdapterConformsToProtocol
  • RequestHeadersTests (1 test):

    • testRequestHeadersProtocolImplementation

Configuration:

  • Added test target to Package.swift with required dependencies
  • Added .swiftlint.yml configuration for code quality

Test Strategy

Tests are focused and maintainable, avoiding:

  • Redundant checks of framework behavior
  • Duplicate test scenarios
  • Tests requiring complex mocking infrastructure

All 22 tests are passing ✅

Inspiration

Test structure and patterns inspired by:

🤖 Generated with Claude Code

@ssestak ssestak requested a review from jmarek41 January 7, 2026 14:09
Added 6 new tests covering core adapter functionality:
- GraphQLAPIAdapterTests: 5 tests for initialization with various configurations (minimal, network tracer, custom headers, URL session config) and protocol conformance
- RequestHeadersTests: 1 test verifying RequestHeaders protocol implementation

Total test count increased from 16 to 22 tests, all passing.
Tests are focused and maintainable, avoiding redundant checks of framework behavior.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@ssestak ssestak changed the base branch from feature/Add-support-of-FTNetworkTracer to main January 13, 2026 15:17
@ssestak ssestak merged commit 9a446a3 into main Jan 13, 2026
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