[feature]: Add Flask + SQLite Sample App with Keploy Integration #2798#62
Open
eatulrajput wants to merge 4 commits intokeploy:mainfrom
Open
[feature]: Add Flask + SQLite Sample App with Keploy Integration #2798#62eatulrajput wants to merge 4 commits intokeploy:mainfrom
eatulrajput wants to merge 4 commits intokeploy:mainfrom
Conversation
There was a problem hiding this comment.
Pull Request Overview
Adds a sample Flask + SQLite application with full CRUD operations for student data and integrates Keploy to auto-generate and replay API tests.
- Introduces Flask app (
app.py) and SQLAlchemy model (models.py) with helper validation. - Adds Keploy configuration and auto-generated HTTP test cases for GET and POST endpoints.
- Provides setup docs, dependencies, ignore rules, and coverage settings.
Reviewed Changes
Copilot reviewed 14 out of 15 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| requirements.txt | Lists framework dependencies |
| models.py | Defines Student model and initializes db |
| app.py | Implements CRUD routes with input validation |
| flask-sqlite/keploy/test-set-0/*.yaml | Auto-generated Keploy HTTP test specifications |
| flask-sqlite/keploy.yml | Keploy configuration for recording and testing |
| flask-sqlite/.gitignore | Ignores build artifacts, DB files, and logs |
| flask-sqlite/.coveragerc | Coverage tool configuration |
| README.md | Setup instructions and API documentation |
Comments suppressed due to low confidence (2)
flask-sqlite/requirements.txt:2
- The PyPI package name uses a hyphen, not an underscore. Change to
Flask-SQLAlchemy==3.1.1to ensure it installs correctly.
Flask_SQLAlchemy==3.1.1
flask-sqlite/app.py:54
- There are no Keploy tests covering the
PUT /students/<id>andDELETE /students/<id>endpoints. Consider adding recordings for update and delete operations to ensure full coverage.
@app.route('/students/<int:id>', methods=['PUT'])
| self: s1 | ||
| inCi: false | ||
|
|
||
| # Visit [https://keploy.io/docs/running-keploy/configuration-file/] to learn about using keploy through configration file. |
There was a problem hiding this comment.
There's a typo in the comment: configration should be configuration.
Suggested change
| # Visit [https://keploy.io/docs/running-keploy/configuration-file/] to learn about using keploy through configration file. | |
| # Visit [https://keploy.io/docs/running-keploy/configuration-file/] to learn about using keploy through configuration file. |
Signed-off-by: Atul Rajput <eatulrajput@gmail.com>
Signed-off-by: Atul Rajput <eatulrajput@gmail.com>
Signed-off-by: Atul Rajput <eatulrajput@gmail.com>
|
Hey @eatulrajput, Thank you for raising the pr. Our team will review the pr soon |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
pratik-mahalle
left a comment
There was a problem hiding this comment.
Hi @eatulrajput, This is a great use case! Can you please share the screenshot or recording if you have so that we will get the better view of it?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
[feature]: Add Flask + SQLite Sample App with Keploy Integration - #2798
Issue : keploy/keploy#2798
This PR adds a new sample application that demonstrates how to use Flask with SQLite and integrate it with Keploy for API testing.
Features:
CRUD APIs for managing student data
Uses SQLite (no external DB setup needed)
Built with Flask and Flask-SQLAlchemy