1
100%
main: 100%

Ran 09 May 2024 07:12PM UTC

Files 30

Run time 0s

Badge

Embed ▾

Committed 09 May 2024 07:11PM UTC coverage: 99.859% (-0.1%) from 100.0%

Job # eb8d5b68d7a0106202bdda13040c788d039111e0.1

Build Type

push

github

Committed by

web-flow

Commit Message

fix: `Spear.append/3` `raw?: true` raw return (#97)

Issue #96

Passing a `raw?: true` option to `Spear.append/3` or `Spear.Client/append`
continued to return `:ok` instead of `{:ok, AppendResp.t()}`.

This contradicted the docs and the `Spear.append_batch/5` behaviour.

Solution:

This commit adds the conditional case in `Spear.append/3`
to return `{:ok, AppendResp.t()}` when `raw?` is `true`.

`Spear.Client.append` will also be fixed as a result.

Additional Notes:

* Added to `SpearTest`: to test `append/3` for both raw true and false
* Added to `SpearTest`: to test `append_batch/5` for both raw true and false.
* Updated the `Spear.append/3` type spec to add the missing
  `{:ok, AppendResp.t()}` return type.
* Updated the `Spear.Client.append/3` and `Spear.Client.append/4`
  callback specs to add the missing `{:ok, AppendResp.t()}` return type.

Run Details

710 of 711 relevant lines covered (99.86%)

72.56 hits per line

NFIBrokerage / spear / eb8d5b68d7a0106202bdda13040c788d039111e0 / 1
100%
main: 100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job eb8d5b68d7a0106202bdda13040c788d039111e0.1

NFIBrokerage / spear / eb8d5b68d7a0106202bdda13040c788d039111e0 / 1 100% main: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job eb8d5b68d7a0106202bdda13040c788d039111e0.1

NFIBrokerage / spear / eb8d5b68d7a0106202bdda13040c788d039111e0 / 1
100%
main: 100%

README BADGES
x