trento-project / web / 33a61f4c0dbe463188ffea3be9fd38aaa02e38b8

88%

push

github

Activity log pagination and filtering (#2786)

* Pull in Flop as a dependency

.. and add config

* Setup schema level config and flop defaults

* Adds some tests

* Updates test

test now uses factory inserted columns and Enum operations on these for assertions.

* Implements pagination endpoint

WIP

* Address PR comment

Makes use of `insert_list`

* Adds test for pagination

Changes query params for supporting pagination; removes unused field in view.

* Fixes credo

* Updates view/test

- Some renaming
- Removes one test

* mix format

* General refactor; Addresses PR comments

- Simplifies implementation of parse_params according to suggestion
- Moves parse_params function to context module
- Removes some dead/unused code
- Adds error response of 400 bad request
- Some general refactoring/default removal

* Adds some docs

According to PR comment

* Refactors schema

According to PR comment

* Update lib/trento/activity_log.ex

Co-authored-by: Nelson Kopliku <nelson.kopliku@suse.com>

* Update lib/trento/activity_log.ex

PR suggestion.

Co-authored-by: Emanuele De Cupis <ema.decu@gmail.com>

* Fixes openapi schema

Needs to be nested and in specific order. Re-adds a removed field (id).

* Fixes/Rebase clean-up

for a rebase

* Adds test for date ranges

* Small refactor of the context module

- prevents type errors
- corrects incorrect comment
- minor refactoring

* Renaming

According to PR comments

* Removes unnecessary guard

* More tests; removes some response fields

According to PR comments

* Disables api-bc-check dependency

Temporary, to allow CI runs, to be reverted

* Revert "Disables api-bc-check dependency"

This reverts commit 28aa22ca5.

* Makes function private

* Makes function private

---------

Co-authored-by: Emanuele De Cupis <ema.decu@gmail.com>
Co-authored-by: Nelson Kopliku <nelson.kopliku@suse.com>

18 of 20 new or added lines in 5 files covered. (90.0%)

2673 of 3156 relevant lines covered (84.7%)

45.86 hits per line