Add simple unit tests for server #32
Merged
Conversation
This file contains 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
Contributor
eyurtsev
commented
Nov 12, 2023
eyurtsev
commented
- Adds a simple unit tests to test one of the api endpoints
eyurtsev
commented
Nov 12, 2023
| import os | ||
|
|
||
| # Temporary handling of environment variables for testing | ||
| os.environ["REDIS_URL"] = "redis://localhost:6379/3" |
There was a problem hiding this comment.
pretty hacky, but we can update afterwards to pick up settings properly
nfcampos
reviewed
Nov 12, 2023
| } | ||
|
|
||
| # Check not visible to other users | ||
| headers = {"Cookie": "opengpts_user_id=2 flushdb"} |
hwchase17
added a commit
that referenced
this pull request
Nov 13, 2023
* start statefullness * cr * cr * cr * Fix * with_types * cr * Fix uuids * expose session_id * cr * cr * Add thread and assistant storage endpoints * Add deps * Add message list to output value of list_threads * Return created objects * Frontend: store bots in backend * Update api * Finish frontend * Finish frontend * x * x * x * x * x * x * x * x * x * x * x * x * secrets * Update deps * Fix gpt4 * fe: sort chats and configs * Lint * Lint * Comment * Storage helper funcs * Fix chat order * Add loading state for thread history * Add more loading states * Add research preview banner * Add scoping on user_id * Lint * Fix chat history * Finish file upload api and frontend * Clear files when switcing configs * Make assistants shareable * Some work for feedback buttons * x * x * x * x * x * x * x * x * x * x * x * x * x * x * x * Add redis service to CI (#24) Add redis service to CI (still not used by tests) * Set up tests with redis (#26) - Add testing with redis - Fix bug in ingest runnable - Add RunnableBindingBase to this repo until it's merged and released via langchain * x (#27) * x * cr * cr * cr * cr * Make tests work * Format, lint * Add new env varaibles to make file * Comment out code that's not been released on langchain * Fix missing deps * Display bot name * Fix stream state not resetting when switching threads * Display error message * Display file errors, accummulate files * Fix bottom padding, move public link button to top * Add icon to view config for bot used in current chat * Add tool markdown descriptions * Add feedback buttons * Add tests for endpoints involving storage on server (#32) Add tests for endpoints involving storage * x * x (#34) * Add some validation to endpoints (#35) Add a bit more validation to endpoints * Use models with longer context windows * Update gcp env file * Remove dupe * Use disabled var * Disable autofocus on new bot name * Add libmagic * Lint * Better public bot share link * Add feature public bots * Localize access to REDIS_URL (#36) * Localize a bit access to REDIS_URL * cr --------- Co-authored-by: Nuno Campos <nuno@boringbits.io> Co-authored-by: Eugene Yurtsev <eyurtsev@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.