-
Notifications
You must be signed in to change notification settings - Fork 5
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
sprint 1 #15
Merged
Merged
sprint 1 #15
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
* create sign up card * add api/sign_up to postRequests * add api/sign_in to postRequests * change sign in username to email * reformat * remove sign up * create page for task_6 * add sign in * rename sign_up.test.tsx to task_6.test.tsx * add register and login to fetching * fix mismatched braces in fetching.ts * add token to headers in fetching.ts * save token to local storage * fix unable to find element * delete commented code * rename sign up and sign in to register and login respectively * split login and register into separate pages, add input validation, and take field data out of url * add user event library * add testing for login * add testing for register * format * fix eslint errors * fix eslint error * fix localstorage not being defined * fix localstorage not being defined --------- Co-authored-by: julia <[email protected]>
* Completed Tasks 4 & 5 with unit tests. Utilized the utils.test.ts to ensure tasks 4 & 5 unit tests ran correctly. Created a seperate utility file for creating JWT tokens * Changed filenames to snake_case. Deleted deps.ts file. Created function for hashing. Removed the try/catch with the error status 500. * Fixed issue with not being able to read .env files with secret JWT key. Need to use �[0m�[38;5;245mrunning 3 tests from ./api/register_component/user_registration.test.ts�[0m POST /api/register - Successful Registration ... �[0m�[32mok�[0m �[0m�[38;5;245m(6ms)�[0m POST /api/register - Missing Fields ... �[0m�[32mok�[0m �[0m�[38;5;245m(0ms)�[0m POST /api/register - Duplicate Email ... �[0m�[32mok�[0m �[0m�[38;5;245m(3ms)�[0m �[0m�[38;5;245mrunning 1 test from ./api/user_login_component/user_login.test.ts�[0m POST /api/userLogin - Successful Login ... �[0m�[32mok�[0m �[0m�[38;5;245m(7ms)�[0m �[0m�[38;5;245mrunning 1 test from ./hello.test.ts�[0m hello test ... �[0m�[32mok�[0m �[0m�[38;5;245m(1ms)�[0m �[0m�[38;5;245mrunning 5 tests from ./in_memory/in_memory.test.ts�[0m Generate session ID ... �[0m�[32mok�[0m �[0m�[38;5;245m(1ms)�[0m storeData and retrieveData are successful ... �[0m�[32mok�[0m �[0m�[38;5;245m(0ms)�[0m storeData retrieves null for non-existent sessionId ... �[0m�[32mok�[0m �[0m�[38;5;245m(0ms)�[0m deleteData removes data correctly ... �[0m�[32mok�[0m �[0m�[38;5;245m(0ms)�[0m clearAllData removes all session data ... �[0m�[32mok�[0m �[0m�[38;5;245m(0ms)�[0m �[0m�[38;5;245mrunning 2 tests from ./upload/job_description_upload.test.ts�[0m Job Description - Valid Input ... �[0m�[32mok�[0m �[0m�[38;5;245m(9ms)�[0m Job Description - Exceeds Character Limit ... �[0m�[32mok�[0m �[0m�[38;5;245m(0ms)�[0m �[0m�[38;5;245mrunning 3 tests from ./upload/resume_upload.test.ts�[0m Valid PDF ... �[0m�[32mok�[0m �[0m�[38;5;245m(8ms)�[0m Resume Upload - Invalid File Type ... �[0m�[32mok�[0m �[0m�[38;5;245m(1ms)�[0m Oversized File ... �[0m�[32mok�[0m �[0m�[38;5;245m(31ms)�[0m �[0m�[38;5;245mrunning 0 tests from ./util/util.test.ts�[0m �[0m�[32mok�[0m | 15 passed | 0 failed �[0m�[38;5;245m(448ms)�[0m to allow for reading .env files * Finished task 11 along with tests. Created a folder to hold one of the test PDF's and extracted text for testing. Needed to remove the .env from the .gitignore as the name change to .env.local seemed to cause the file to not be found. Updated the command to run tests. * Ran the deno fmt command. Updated the deno.yml file with the correct testing command. * parse text from File object instead of file path * Updated pre-commit file to match deno test in deno.yml * improve jwt helpers * use json for login and registration endpoints * Trying to modify the routes.ts to use middlware to minimize changes to current working file upload api's. * use token for session data * add test for jwt * salt hashes * Created test file for session_middleware.ts. * test session middleware --------- Co-authored-by: julia <[email protected]>
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.
No description provided.