MeltanoLabs / MeltanoLabs/tap-gitlab
Add tests
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 12
- Forks
- 29
- Avg merge
- 4d 1h
- Merged PRs (30d)
- 1
Description
In GitLab by @tomelliff on Oct 21, 2020, 13:09
I've noticed that very few taps and targets seem to have tests but I felt particularly uncomfortable making an untested change despite the fact it was very small. https://github.com/transferwise/pipelinewise-tap-postgres/tree/master/tests shows a high quality tap that does have tests and it doesn't look too painful.
Would you accept a small MR to add the start of some tests for now and then we can expand them as we go and insist on tests for any new functionality?
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by inspecting the tap-gitlab repository's current test layout and identifying the extraction paths that lack coverage. Compare the referenced pipelinewise-tap-postgres tests, then define a small initial test set and verify it with the repository's existing test process; the issue names no specific files or command.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- testing-qa
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100