tests: serializers and loaders

inveniosoftware / flask-resources

REST APIs for Flask

https://flask-resources.readthedocs.io

MIT License

3 stars 21 forks source link

tests: serializers and loaders #24

Closed ppanero closed 4 years ago

ppanero commented 4 years ago

requires https://github.com/inveniosoftware/flask-resources/pull/21

adds tests for serializers and loaders:

Some tests are ultra simple, even trivial.
For now they might not test anything needed, because the complexity has not arrived yet.
It is nice to have them so they evelove/force us to test the code when implementing changes/features.

Many other interfaces/classes still need testing.

Closes https://github.com/inveniosoftware/flask-resources/issues/11

ppanero commented 4 years ago

@fenekku do you think this PR still adds value? or we can close it?

fenekku commented 4 years ago

Yes, we can close it. We are testing most of these things now and error serialization will test the remaining bits that were in this PR. I think we can mark #11 as closed as well.