cayleygraph / cayleygraph/cayley

Improve Integration Tests using `.Save()` and `.Or()`

Open
#238 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

good first issue Test Coverage
Dominant language
Go
Stars
15.1k
Forks
1.2k
PR merge metrics
No merged PRs in 30d

Description

gocov test -coverpkg=./graph/iterator . | gocov report shows we cover the base iterators in our integration test with a code coverage of 53.45% (as of this writing). Most of these come from lack of testing the Optional iterator or the Or iterator.

Simple enough: add some (hopefully grunty) integration/benchmark queries to cayley_test.go and raise this number considerably.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with cayley_test.go and review the existing integration and benchmark queries. Run the gocov command from the issue to establish the current coverage, then add queries that exercise the Optional and Or iterators. Done means the integration coverage is substantially higher than 53.45%.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
testing
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.