Khan / Khan/genqlient

Make it easier to integration test config options

Open
#261 0 comments 0 reactions 0 assignees View on GitHub
help wanted internal needs design
Dominant language
Go
Stars
1.3k
Forks
143
Avg merge
4h 50m
Merged PRs (30d)
1

Description

Our integration tests all run with a single config, which makes it hard to integration test a change like #252. Ideally we should do some reworking so we can test them against several configs, like we do in the snapshot tests. We'll have to think about what the best way is.

Once we do, we should add integration tests for #252, which should be fairly easy once the tooling is there.

Contributor guide

Open the contributing guide

Research direction

Compare how integration tests configure and run against the single current configuration with the snapshot tests that already support several configurations. Decide how the integration test tooling should support multiple configs, then add coverage for the change described in #252 and verify the integration suite runs against each configuration.

Written by the indexing model from the issue text.

Assessment

Tech stack
go, graphql
Domain
testing, tooling
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.