dwyl / dwyl/phoenix-ecto-append-only-log-example

Coverage ...

Open
#10 7 comments 0 reactions 0 assignees View on GitHub
chore enhancement help wanted starter technical
Dominant language
Elixir
Stars
82
Forks
5
PR merge metrics
No merged PRs in 30d

Description

@dwyl we have a "Gold Standard" for examples/tutorials which _must_ be followed without exception.
Tutorials aimed at _beginners_ should _always_ have "_complete tests_" to _eliminate_ any "_excuse_" people may have for _not_ writing tests in their projects.

An example is _incomplete_ without having 100% test coverage because
it ***either*** means that there is **_superfluous_ code** (_which can be removed_)
***OR*** there is **_untested_ code**; functionality that is "magic". _Both_ are ["_below expectations_"](https://www.youtube.com/watch?v=hTghEXKNj7g).

At present this example has 56% coverage:
https://codecov.io/github/dwyl/phoenix-ecto-append-only-log-example?branch=master

![image](https://user-images.githubusercontent.com/194400/47010349-a31fa780-d136-11e8-990e-8d32865f5913.png)

# Todo

+ [ ] fix this: [![codecov.io](https://img.shields.io/codecov/c/github/dwyl/phoenix-ecto-append-only-log-example/master.svg?style=flat-square)](http://codecov.io/github/dwyl/phoenix-ecto-append-only-log-example?branch=master)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.