Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Sanic support: parser, test app, tests #287

Closed
wants to merge 4 commits into from
Closed

Add Sanic support: parser, test app, tests #287

wants to merge 4 commits into from

Conversation

dnikolayev
Copy link

We commonly use webargs + flask.
However, there was a requirement to build a project using Sanic.
To have the same style support with common practice Sanic support was added.
Please review.
@sloria, to enable Sanic support webtest-sanic package was released. The code of webtest-sanic is fully based on your library webtest-aiohttp and all creds saved.

@sloria
Copy link
Member

sloria commented Sep 17, 2018

Hi @dnikolayev . Thanks for your work on this!

In order to keep this library small and easy to maintain, we've stopped adding support for new frameworks, so I'm going to close this.

For now, I recommend adding webargs-sanic to https://github.com/sloria/webargs/wiki/Ecosystem . I'll forward people to that library when asked about sanic support.

Also, would it help if CommonTestCase were exposed in the PyPI package so you could import it in webargs-sanic?

@sloria sloria closed this Sep 17, 2018
@dnikolayev
Copy link
Author

Ok, I see. I already started to update docs and check contributing policies to pass everything, LOL

As for the "CommonTestCase" - Well, I guess so.
Could you please release it? Will it take much time?

@sloria
Copy link
Member

sloria commented Sep 17, 2018

@dnikolayev I'll try to make a new release with CommonTestCase after work today

@lafrech
Copy link
Member

lafrech commented Sep 17, 2018

In order to keep this library small and easy to maintain, we've stopped adding support for new frameworks

Maybe we ought to update CONTRIBUTING.rst to mention that.

@dnikolayev
Copy link
Author

Thnx @sloria
I will wait.
While you're preparing the shared version of test class - I used a copy of it here: https://github.com/EndurantDevs/webargs-sanic
Will change to shared version once you create it. Probably it will be the great idea to have it shared between the core project and versions for other frameworks.

Thanks again for your help!

sloria added a commit that referenced this pull request Sep 17, 2018
sloria added a commit that referenced this pull request Sep 17, 2018
sloria added a commit that referenced this pull request Sep 17, 2018
sloria added a commit that referenced this pull request Sep 18, 2018
sloria added a commit that referenced this pull request Sep 18, 2018
sloria added a commit that referenced this pull request Sep 18, 2018
* "Github" -> "GitHub"
* strict=True only applies to marshmallow 2
* Add note about framework integration in CONTRIBUTING

#287 (comment)
@sloria sloria mentioned this pull request Sep 18, 2018
sloria added a commit that referenced this pull request Sep 18, 2018
* "Github" -> "GitHub"
* strict=True only applies to marshmallow 2
* Add note about framework integration in CONTRIBUTING

#287 (comment)
@sloria
Copy link
Member

sloria commented Sep 18, 2018

@dnikolayev OK, 4.1.0 includes webargs.testing.CommonTestCase.

@sloria
Copy link
Member

sloria commented Sep 18, 2018

Also, I've updated CONTRIBUTING.rst. Good idea, @lafrech

@dnikolayev
Copy link
Author

I removed a copy of common tests. webargs.testing.CommonTestCase is used in tests.
Also, I've extended tests locally, including testing files validation via Sanic's test client.
Package is released, you can always send people who needs integration with Sanic to webargs-sanic

@sloria, Thanks for your help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants