Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Data returned by submissions inconsistent #186

Closed
AronBuzogany opened this issue Apr 6, 2024 · 0 comments · Fixed by #187
Closed

Data returned by submissions inconsistent #186

AronBuzogany opened this issue Apr 6, 2024 · 0 comments · Fixed by #187
Assignees

Comments

@AronBuzogany
Copy link
Contributor

Different keys on post and get by id than get on general

@AronBuzogany AronBuzogany self-assigned this Apr 6, 2024
AronBuzogany added a commit that referenced this issue Apr 6, 2024
Vucis pushed a commit that referenced this issue Apr 10, 2024
* init project display

* Fix #175

* added dependencies for #178

* Fix #186

* fixed test

* resolved linting

* fixed download url of submissions

* added title to header

* fixed typos

* merge fail
Vucis added a commit that referenced this issue May 23, 2024
* #15 - Using environment variable UPLOAD_FOLDER

* #15 - Add static typing to the submissions model

* #15 - Fixing linter

* docker-compose added

* #15 - Will fix this later

* #15 - Linter has cyclic imports

* change port

* #15 - Small URL change

* fixed extracting of zip and uploading in project upload directory

* fix
400 when non zip is uploaded

* fixed tests

* added test zip file

* linter fixes

* removed some test files

* linter and test fixes

* import depedency fix

* fix import order

* removed import getenv

* removed valid_project function

* fix fstring

* fix: upload_directory

* removed exist_ok=True

* use path.join

* added url field

* fixed not checking for tuple type anymore

* fixed env var for tests

* fixed env var for tests

* fixed with statements

* using os.path.split instead of regular split

* added exist_ok

* i forgot 💀 fix lmao yeet

* i forgot 💀 fix lmao yeet

* goofy augh fstring

* another small fix

* fixed the 'not fixed eh' problem

* linting

* fix handling fail

* added try block

* linter

* nginx.conf added

* rm conf

* fixed archieved to archived typo (#84)

* fixed archieved to archived typo

* fixed typo descriptions to description

* last change

* tests passed!

* #15 - Spelling

* #15 - Fixing tests the merge broke

* Feature/share link (#72)

* create subfolder for course tests

* added table for share codes

* loading env variables

* added default value to function parameters

* added endpoint for join_codes

* added tests for join_codes

* fixed: linting

* fixed typo:

* fixed import conftest

* fixed merge with singular model naming

* unused file

* added functionlity for downloading files

* reformatting

* linter

* niets veranderd? linter flipt

* small newline change

* removed f-string formatting

* added 404 and 500 cases

* fixed tests

* fixed os.curcwd

* linter

* removed print

* check if file exist and commented code removed

* fixed code duplication

* used basename insteal of .split

* linter fix

* Backend/tests/review models (#83)

* #81 - Updated user model tests, TODO delete

* #81 - Removing an unnecessary file

* #81 - Updated course model tests, TODO delete

* #81 - Updated course_relation model tests

* #81 - Cleanup

* #81 - Project model tests

* #81 - Submission model tests

* #81 - Cleanup

* #81 - Adding a test and also testing GH tests out

* #81 - Adding rollbacks after every error raised

* #81 - Maybe fix

* #81 - Another try

* #81 - This should fix it

* #81 Adding the delete method tests and updating the db_construct

* Tests/cleanup (#104)

* command useless and takes time

* cleaned up courses tests

* cleaned up project tests

* moved testzip to resources folder

* cleaned up users tests

* small cleanup submission tests

* resolved linting

* corrected function docs

* re-using existing fixtures to reduce code duplication

* resolved linter

* removed useless fstring

* removed unused import

* fix badge for workflow in readme.md (#108)

* test example workflow

* check if push argument works

* does the value update test

* Create ci-test-backend.yaml

created a seperate file for the backend test

* added backend badge to readme

* changed yml to yaml

* changed tests to test

* added seperate files for linter and tests for both front and backend

* gave correct names to workflows

* correct names to run name

* removed code dupe and pray for test run

* added both badges for linter and tests from backend

* file changes detected?

* removed unused code parts

* fix: eslint not found error

* added frontend badges

* frontend developper instructions (#110)

* frontend developper instructions

* only 1 backtick fix

* Merge backend authentication into development (#73)

* start authentication

* authentication start decorators

* login_required should work with access token

* backend authentication for most endpoints (very rough draft of functions in authentication.py)

* clean_up_function

* authentication cleanup

* give error when access_token fails

* documentation auth functions

* fixed imports

* actual import fix

* added requests

* authorize submissions

* removed double checks

* start testing setup backend authentication

* poging testen

* github tests check

* user tests with authentication

* auth url accessible hopefully

* change authorization to be easier to deal with since it doesn't matter for tests

* fixed jobCategory -> jobTitle

* fix authentication

* user tests zouden moeten slagen

* fix authentication arguments

* project tests with authentication

* changed auth server id of teacher

* maybe correct primary keys

* second try on primary key of course relations

* further test authentication

* authentication on project assignment files

* auth on course_join_codes and extra tests

* teacher_id in function when necessary

* user tests with authentication

* extra testing

* fixed comments

* lots of testing changes

* should be 1 error test now

* fix tests

* Fix #114 (#115)

* Fix #114

* test 1

* Fix #114 try 2

* fixed misnamed runner

* removed trailing new line

* Merge cleanup of authentication and tests into development (#111)

* start authentication

* authentication start decorators

* login_required should work with access token

* backend authentication for most endpoints (very rough draft of functions in authentication.py)

* clean_up_function

* authentication cleanup

* give error when access_token fails

* documentation auth functions

* fixed imports

* actual import fix

* added requests

* authorize submissions

* removed double checks

* start testing setup backend authentication

* poging testen

* github tests check

* user tests with authentication

* auth url accessible hopefully

* change authorization to be easier to deal with since it doesn't matter for tests

* fixed jobCategory -> jobTitle

* fix authentication

* user tests zouden moeten slagen

* fix authentication arguments

* project tests with authentication

* changed auth server id of teacher

* maybe correct primary keys

* second try on primary key of course relations

* further test authentication

* authentication on project assignment files

* auth on course_join_codes and extra tests

* teacher_id in function when necessary

* user tests with authentication

* extra testing

* fixed comments

* lots of testing changes

* should be 1 error test now

* fix tests

* small linter fix

* should fix the linter

* actually fixed this time

* linter all return statements

* allow admins to patch users

* should pass all tests

* mergeable

* grouped imports

* made requested changes

* requested changes

* line too long

* unused imports

* added models folder to utils

* Enhancement/debug env variable (#128)

* added waitress te requirements to serve in production

* added debug as an env variable and serving with waitress when not in debug mode

* fixed backend to frontend typo and added workflow branch argument (#119)

* fixed backend to frontend typo and added workflow branch argument

* added back 2. back to readme of backend

* changed submission status to enum (#133)

* changed submission status to enum

* resolved linting

* Fix #77 changed grading to float (#134)

* Fixed #112 (#113)

* Fixed #112

* removed new line

* Fix for uploading project files (#120)

* files are now patchable uncluding the files

* fix for uploading a file that is not a zip that it doesn't get saved

* unsaved changes

* buzoghany requested changes

* linter

* added rollbacks

* extra rollback

* Add get courses tests

* Add post courses tests

* Add get course tests

* Add patch course tests

* Add delete course tests

* Removes all usage of str(e) in courses endpoint (#126)

* Removed all usage of str(e) in courses endpoint

* common parts of error put into const var

---------

Co-authored-by: Siebe Vlietinck <71773032+Vucis@users.noreply.github.com>

* changed port to 5001 (#142)

* Removed is_teacher and is_admin, changed to role enum (#127)

* Removed is_teacher and is_admin, changed to role enum

* sql enum type

* usage of enum and fixed sql

* made role serializable

* clean

* docs

* fixed user patch

* args vs json

* test

* check if course name is blank (#100)

* check if course name is blank

* generalized blank string check

* fixed yo request

* fixed conflicts

* cleaned code

* fix

* started initial readme for repo (#122)

* started initial readme for repo

* added workflow badges

* readme edits

* spelling mistakes

* grammar fix

* fixed route /docs and edited projects endpoint (#123)

* fixed route /docs and edited projects endpoint

* linter+test succeed

* fix: remove duplicate json file

* edited env variable for failing tests

* linter

* Base setup for easy translations in frontend (#145)

* i18n setup

* added dependencies

* Fix for issue 102. Submissions get now returns summary information instead of only url (#147)

* Fix #102

* linting

* Correctly uploading submission files (#136)

* 116 - Correctly uploading submission files

* #116 - Spelling

* Remove break

* Remove unused variable

* Specify which files are required

* Remove unnecessary stringify of UPLOAD_FOLDER

* Update the submission_path path.join

* Catching OSErrors when making directories and saving files

* Update the run_tests script

* This should allow frontend to communicate with our backend (#144)

* allow cors

* moved CORS to create_app_with_db

* Test invalid user form (#137)

* added test for invalid user form

* traling whitespace

* post to /users is not allowed

* removed double invalid fixture

* Test for course with invalid field (#125)

* added test for course with invalid field

* authorization header!

* fix

* Merge with backend/tests/auth

* courses get finished

* Feature/backend/documentation (#94)

* cooken on the projects documentation

* #92 - Fixing missing /submissions

* #92 - Fixing apilinter

* requested changes

* openapi object

* changed to yaml file

* small fix

* grammar

* json->yaml

* pyyaml installed in dev.txt

* linter

---------

Co-authored-by: Jarne Clauw <67628242+JarneClauw@users.noreply.github.com>

* Base test class to handle auth (#153)

* Adding auth tokens

* Setup for auth tests

* Fixing tests

* Cleanup

* Spelling

* Remove unnecessary commit

* navbar (#80)

* navbar

* changes

* oops

* close bar on redirect

* linter conflicts for component function

* scalability matters

* useEffect to check login state and adjust link item list

* a little more readable

* No longer authenticating teachers/students through url params (#165)

* Fix #105

* adjusted tests to not use url query as authentication

* nested translation files (#163)

* return deadline aswel on projects (#168)

* Bump vite from 5.1.3 to 5.1.7 in /frontend (#166)

Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 5.1.3 to 5.1.7.
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/v5.1.7/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v5.1.7/packages/vite)

---
updated-dependencies:
- dependency-name: vite
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Using i18n browser detector and route detection to assign languages aswell as menu for users (#156)

* added tag to locales

* added menu to switch between languages

* updated i18n detector dependency

* configured detector to detect language in path

* configured language routes

* added outlet that detects languague and adds it to i18n

* added menu to select language

* resolved linting

* unused statement removed

* Frontend/feature/submission post (#150)

* added dependencies for treeview and code viewer

* created drag and drop upload, treeview and code viewer

* Fix #10

* removed unused dependencies

* created files utils with zip regex checker

* moved function to utils

* removed unused eslint rule

* corrected message on missing required files

* consistent frontend text

* fixed grammar

* resolved package diff

* Fixed inconsistently added auth tokens

* Whoops added it again

* Adding commented out tests

* Adding all the other auth tests

* Tests for students

* Admin tests

* Forgot part of merge

* Fix #179 (#180)

* Fix #179

* adjusted test to new endpoint

* fixed test

* fixed linting

* Whoops

* Hope this fixes it

* Typing and comments

* updating tests

* Set title of header using Portal (#177)

* Set title using Portal

* cleanup

* Removed paths from db for projects and allowing multiple deadlines (#185)

* Fix #181

* resolved syntax errors

* Fix #181

* removed unused statement

* resolved linting issues

* removed print statement

* Created dynamic deadline calender view/input (#190)

* Fix #188

* added mui date time components to dependencies

* fixed editable not being passed as prop

* Backend/feature/submission evaluation  (#67)

* added docker to requirements

* added: functionality to run a dockerized tester on a given submission

* added tests for submission evaluators

* moved file handling functions to seperate file

* added interpretor path

* updated dependencies

* fixed: linting

* added init file for module recognition

* CMD isn't used anymore as it is provided by the compose

* not running utils in docker as it uses docker internally and we don't yet have support for dind

* switched to LF

* changed: running tests not linting

* subfolders for submissions are nog longer made by the evaluator

* removed unused import statement

* linting: added blank line

* added ussage of requirements manifest which allows teachers to manifest which requirements are allowed to be used

* added: helper functions to prepare test cases

* added test case for working with requirements

* disabled pylinter for evaluator test resources

* fixed: linter ignoring all files

* using pylint recursive argument instead

* resolved merging issues

* using docker in docker image instead

* no longer binding socket to image

* passing path instead of project

* hotfix

* binding docker daemon

* starting dind docker daemon in entrypoint script

* removed unused env variable

* running all tests

* fixed linting

---------

Co-authored-by: Aron Buzogany <aronsaps@gmail.com>
Co-authored-by: abuzogan <aron.buzogany@ugent.be>

* Generalizing data field typing tests

* Cleanup + renaming

* Moving format to functions to make it clearer

* cleanup

* Fix test_post_admins

* Fix test_get_courses_wrong_parameter

* Creates project display for students and solves issue 186 (#187)

* init project display

* Fix #175

* added dependencies for #178

* Fix #186

* fixed test

* resolved linting

* fixed download url of submissions

* added title to header

* fixed typos

* merge fail

* Fixing all tests

* cleanup

* Generalizing some query parameter tests

* Fixing linter

* Whoops

* Making authorization tests more explicit

* registering composite types (#197)

* Added the runner used to test submissions as an optional field of project table (#199)

* Fix #198

* added possibility for custom runner

* fixed enum type name

* Running evaluator in synchron when runner is set (#203)

* Fix #200

* created executor

* fixed tests

* resolved linting

* linting

* fix wrong evaluator in test

* adjusted runner to support test

* Check if submission is LATE (#196)

* checking for if deadline is on time

* Fix #82

* removed redundant print statement

* authenticate endpoint

* setting submission status when no runner

* fixed inverted boolean

* router fix (#205)

* Added functionality to download submissions (#206)

* Fix #189

* fixed linter

* fixed linting

* fix the no status on workflow badges (#211)

* dev changes?

* removed branch argument

* removed random changes

* The submissions form for project page (#124)

* fixed beginning of project submission form

* changed

* changes

* changed

* removed print

* form works

* project form is working

* backend fixes

* backend linter

* linter

* fixed language support

* frontend linter

* packagelock

* linter

* letsgoo

* unused import :nerd:

* augh

* siebe review

* aron pr review

* merge conf

* pr fixed

* veel

* :)

* idk how it works but it does

* changes, a lot of them

* does this fix

* conftest

* dev changes?

* tests are working

* working regexes

* veel hihi

* removed unused React

* working form

* linter and tests

* small changes

* pr review jeej

* tag and route

* removed CORS

* removed unused imports

* hopefully aron is happy nowgit pushgit pushgit push!

* comment changed

* re added decorator (#214)

* Backend/project submission authorization issue (#210)

* Fixing authorization for project

* Fixing submission authorization issues

* cleanup

* Remove print and fix comments

* Fixing submissions

* Fixing projects

* Cleanup

* Fix filter placement

* error pages (#93)

* initial draft for error pages

* indentation fixed

* docs

* error pages v1

* indentation fixed

* layout adjusted, using RouterProvider instead of BrowserRouter to render errors correctly, tests added

* linter

* documentation

* linter error parameters

* Added i18n, changed general structure of App component and got rid of fade animation for the image

* linting, package-lock.json fix

* package file

* changes to paramters in ErrorPage.tsx

* use of keyPrefix

* keyPrefix

* fix

* Added endpoint to add a user using a join code (#218)

* Fix #71

* added course join endpoint

* internal server error is 500

* fixed code docs

* Addding a displayname to a user object (#221)

* Homepage student + landing page (#194)

Homepages added

* Full authentication (#216)

* backend-authentication

* fix projects

* remove extra requirements

* fix test-auth_server

* added: frontend redirect link

* fix backend auth

* fix error + code_challenge in frontend link

* line too long + frontend linter errors

* actual linter fix

* removed code_verifier until milestone 3

* added ability to log out and added everything to init

* redis added later

* added login button to header

* linter fix unused import

* add display_name to user creation

* test tests

* linter fix

* cookies test

* flask test cookies?

* added logout button

* with client

* Project overview (#219)

* goeie styling yallah

* pls werk

* werkt hopelijk

* linter

---------

Co-authored-by: Siebe Vlietinck <71773032+Vucis@users.noreply.github.com>

* Project overview page (#222)

* start homepage

* added calender functions and titlecard

* homepage changes

* homepage changes

* Added: Homepage and Student Homepage

* homepage changes

* project projects parser

* test passed

* linter

* pr review

* Project card refactor

* homepage change fix

* Revert "Merge remote-tracking branch 'origin/backend/projectendpoint-fix' into frontend/feature/homepage"

This reverts commit 645443a, reversing
changes made to e5451b6.

* homepage changes

* rm comment

* pr changes

* project overview

* project overview

* project overview done

* pr changes

* project page changes

* added support for no deadline projects

* link changed

* link changed

* pr changes

* merged with HomeStudent.tsx

* projects overview page

* wording change

* homepage change

* homepage change

* API URL to API_HOST

* end point with /me

* project pages

* home

* create button

* create button

* opt

* deadline fix

* deadline fix

* deadline fix

* deadline fix

* creds

---------

Co-authored-by: gerwoud <gerwoud@hotmail.be>

* Course views for teacher (#161)

* teacher detailed course base project list

* basic

* scrollers

* doc

* backtick strings

* base all courses page

* changed height

* first finished version of teacher course views

* teacher all course page reviewed

* display time until deadline

* more searchbars and translations

* changed file structure for maintainability

* titles and absolute create button

* searchbars update url and remain on reload

* user and admin listing

* reworked form and gave unique ids to all list items

* whoops linter ssst

* added .env to gitignore

* placeholder for loading courses

* debounce seachbars

* better debouncing more hooks

* setup for loader but doesnt work yet bc bad router

* loaders

* course detail teacher course join codes menu

* join code link points to app host now

* accident

* fixed empty filters, placeholders and teacher in admin list

* delete

* removed wannabe popup errorer

* unuserud

* package

* fix

* uhh

* u

* Fix authentication bugs (#226)

* fixed invalid_token_loader and added function to get csrf_cookie

* changed name of file to be correct

* small fixes

* added csrf to all fetches

* forgot an import

* formatting

* formatting

* added custom fetch function

* removed e

* Frontend/enhancement/join code endpoint (#248)

* Fix #247

* fixed linting

* using authenticated fetch

* unused import

* Fixes several buggs in join code endpoint (#251)

* Fix #249

* linting

* fix home/header login button (#228)

fix home/header login button #227 #241

* add label (#258)

* fix logout on server (#264)

* Fix #277 (#278)

* Fix #276 (#281)

* Changed AUTHENTICATION_URL .env variable (#268)

* changed authentication_url to test_authentication_url

* line too long

* Creates a general code evaluator (#256)

* Fix #255

* using apt to install node instead

* fixed using join instead of dirname

Co-authored-by: Jarne Clauw <67628242+JarneClauw@users.noreply.github.com>

---------

Co-authored-by: Jarne Clauw <67628242+JarneClauw@users.noreply.github.com>

* Authentication fix for backend tests (#280)

* fixed invalid_token_loader and added function to get csrf_cookie

* changed name of file to be correct

* small fixes

* added csrf to all fetches

* forgot an import

* formatting

* formatting

* added custom fetch function

* removed e

* some backend tests fixed hopefully

* added display names to test auth server

* set propagate exceptions to TRUE

* added checks in test auth

* csrf test

* get csrf token from response

* using cookie_jar

* return 401 on test auth when necessary

* get csrf token out of set-cookie headers

* fixed all user tests

* correct import this time

* actual correct import

* added default empty string so login does not crash

* fixed courses test

* fixed share link test

* fix project tests

* fixed submission tests

* change docker command

* removed authentication in no_auth test and correct authentication for project tests

* trying parameterized tests

* extra fixes

* actually correct fixes this time

* errors fixed

* fixed csrf error when wrong authentication

* Fix fill in the course selector (#261)

* fixed the issue

* make let const

* refactor api url

* refactor api url

* use effect

* use effect

* renamed for clarity

* use date not hours (#289)

* Cleanup of authentication tests (#285)

* Cleanup of authentication tests

* Adding tests where no csrf token is given

* Submissions download authorization (#293)

* Fixing

* Removing a test

* Courses now converts deadlines from string to Date right after fetch (#295)

* adjusted courses frontend to use deadlines Date instantiated right after fetch instead of str str 2d

* lintr

* removed TODO

* deadlines now have their own datatype!

* Submission authorization issues (#301)

* fix

* Fixing linter

* Submission issues (#304)

* Change the returned data from submission patch

* Updating the filtering on /submissions

* Return bad request on invalid keys in data

* Little fix

* Little fix

* fixing the f linter

* change path (#305)

* course path (#308)

* Navbar link to homepage added (#262)

* navbar link added

* rm i18

* Fix status on ProjectCards (#307)

* status fix

* rm space

* Fix #253 (#254)

* Seeder for easier demo and frontend testing (#291)

* basic seeder provider, course titles, usernames and random uid

* seeder creates and populates a few courses with random students where given uid is teacher

* ffix

* relocate, weird project deadline bug tho

* seeder go bvv

* fun with linter

* lint

* removed leading spaces

* batch operations and documentation cleanup

* bad env name

* more batch and better try except finally

* specific error

* close

* projects have 0-2 deadlines now, also randomized numprojects from fixed 2 to random 1-3

* 1-3

* import not good

* toml test

* toml

* faker is dev

* linter mad

* titles now in txt file

* completed toml info

* fixed parsing error

---------

Co-authored-by: Aron Buzogany <aronsaps@gmail.com>

* Redirecting when creation of project is succesful or showing error message on failure (#313)

* Fix #275

* removed unused import

* enhanced project submission error message in dutch

* When doing GET on courses, only returning courses you are authorized to see (#282)

* Fix #242

* fixed linting

* Added functionality for url queries

* fixed tests maybe

* changed error code on wrong parameter in tests to 500

* actually changed the assert this time

* courses currently ignores wrong filters

* ignore filter tests on courses for now

---------

Co-authored-by: Siebe Vlietinck <siebe.vlietinck@ugent.be>

* Fix #239 (#316)

* Fix #238 (#317)

* Fix #323 (#324)

* added paper islands, made project cards of equal size, displaying username instead of uid (#311)

* Added header on error pages and moved env import outside fetchme function (#319)

* added header to error pages

* moved env import outside function

* added check on posting project (#321)

* project form upload in user documentation (#250)

* project form upload

* added dutch translation

* PR review and styling

* siebe pr

* aron pr

* aron pr

* When no run_tests.sh is given when creating a project, cannot submit. (#326)

* trnanslation changes

* runner check

* changed everything to have 1 env var name (#332)

* Fix #274 (#314)

* added clipboard to copy the join code and redirecting to course when … (#325)

* added clipboard to copy the join code and redirecting to course when user already in course

* styling

* added dutch translation of python evaluator (#333)

* added dutch translation of python evaluator

* siebe pr

* Added readme for user guide (#330)

* added readme

* siebe pr

* Textfield performance issue (#334)

* added useTransition to wrap useState

* linter

* changed documentation location (#337)

* redirect lang fix (#339)

* add lang path (#341)

* uses display name instead of uid (#328)

* uses display name instead of uid

* re added authentication

* fix failing test

* changed APP var

* removed console log

* replaced path and merge conflicts

* removed 404 for submissions

* fixed page spamming backend with requests

* type error

---------

Co-authored-by: Aron Buzogany <aronsaps@gmail.com>

* Using react mui tabs in project view instead of implementing it again (#343)

* Fix #279

* linting

* Fix flickering with data loader (#348)

* added loader

* added loader

* linter

* fix name

* Seeder should generate unique submission times now (#354)

* unique deadlines

* unique submiss times

* max 10 days into past

* Project form/drag drop translations (#359)

* added some translations

* consistency!

* fixed axios.post to use authentication (#363)

* del (#365)

* outdated code (#358)

Co-authored-by: Warre Provoost <133233646+warreprovoost@users.noreply.github.com>

* teacher added as student to course for real this time (#368)

* Submission tests (#362)

* Cleanup of authentication tests

* Adding tests where no csrf token is given

* Adding authentication tests

* Adding authorization tests

* Fixing most of the issues rn, but waiting on completion of other issue

* fix

* Fixing linter

* Fixing auth tests

* All tests written, but fixes needed

* Fixing most tests

* Fixing all tests except for the evaluator

* Trying to fix tests

* Fixing

* linter

* fix

* Ignore bad filters on fetches to endpoints (#360)

* test

* submissions now ignore bad filters

* small oopsies

* getatribute(Model, key, None), i forgot the None

* table__columns

* Teacher is now listed as admin on course page (#349)

* teacher is also listed, useMemo instead of useEffect to prevent listing twice

* useMemo bad, reworked the student and admin fetchers

* all fetches straight to loader fetches Me objects after fetching admins and students, no need for useffect

* fixed loader and seeder

* fix join codes bug (#374)

* Fix en-us bug (#366)

* fix en-us bug on homepage

* should be fixed everywhere now

* actually fixed hopefully

* fix bug req. change

* Fix the filter tests (#376)

* changed 200 to 400

* changed != to ==

* added filter functionlity to submissions

* removed prints

* linter

* linter again

* filters that aren't fields are now ignored in the code, tests for this are fixed

* yeet

* idk tbh

* Documentation/fix/GitHub link (#378)

* added table to frontend readme

* added .env vars to the README

* updated readme

* changed URL of api part

* changed github url

* changes not for this pr reverted

* newline goofy augh

* patch score in backend when edited in the frontend and i18n for page (#355)

* score patch is working

* changes

* added error message and i18n support

* added tooltip instead of alert :)

* linter

* update frontend readme variables (#356)

* added table to frontend readme

* added .env vars to the README

* updated readme

* changed URL of api part

* removed code verfier

* Frontend testing setup (#290)

* testing setup

* changed from jest to vitest and changed npm scripts to be easier to work with

* hopefully final version of config files

* fixed double import react bug

* setup finished

* linter fix

* forgot to push this

* any type

* requested changes

* vitest coverage generates coverage files

* fix JWT refresh to use database (#383)

* Update backend README (#357)

* added table to frontend readme

* added .env vars to the README

* added readme vars

* changes

* added explanation

* removed changes to frontend readme to exclude it from this pr

* remove code verifier

* added JWT application

* POST_HOST name

* added rfc link

* added microsoft admin link

* updated link

* fix requirements/docker dependency bug (#392)

* Fix #265 (#353)

* removed edit link (#396)

* added user guide links (#397)

* added user guide links

* added specific runner link

* small styling issue

* Add overview button to admin page of projects (#384)

* changes bekijken

* beaucoup

* edit title and description functionality

* linter :nerd:

* frontend linter

---------

Co-authored-by: Siebe Vlietinck <71773032+Vucis@users.noreply.github.com>

* Fixed join code copy to clipboard (#404)

* fixed join code copy

* removed :

* Only return projects where u are student if visible for students is true (#403)

* projects visible for students

* lint

* added conditional rendering for edit funtionality (#402)

* added translation for homepage of user guide (#400)

* added translation

* fix

* Groups backend (#338)

* temp bad

* db_constr, model and first attempt at endpoint for group

* group prim key (group_id,project_id) added delete endpoint to leave groups, next up is test

* allow students in max 1 group

* model tests

* lint

* group menu frontend

* hm

* working endpoint for create and delete group

* translations

* begone front

* front removal

* lintr

* fixed changes, untested tho

* groups locked var should not mess up all older code

* only student or teacher can get groups ; unlock groups

* linter mad

* Very mad lintr

* vscode linter errors should be more obvi

* removed some teacher_id = None

* removed unused import

* bad prints

* unzipping submissions on submission (#406)

* unzipping submissions

* run_test -> run_tests

* run_test -> run_tests

* Fixed deadline time left (#407)

* fixed deadline time left

* > to >=

* Fix #408 (#409)

Co-authored-by: Warre Provoost <133233646+warreprovoost@users.noreply.github.com>

* Filter on conditions the students projects (#411)

* projects visible for students

* lint

* filter on conditions

* lint

* fixed bad merge

* show red dots on home page (#412)

* Project issues (#410)

* Fixing the legacy warning

* Adding auth decorators to the project submission download enpoints

* Fixing query parameter tests

* remove bad param filter

* Project tests (#416)

* Auth tests

* Fixing the legacy warning

* Adding auth decorators to the project submission download enpoints

* Fixing auth tests

* Broken data field and query parameter tests

* Fixing query parameter tests

* More tests but first fixing some issues

* working tests

* remove bad param filter

* stuff

* added 403 for overview page (#414)

* added 403 for overview page

* linter

* added good response.ok placement and status code for 404

* linter

* Student course overview (#417)

* student overview

* linter

* linter

* linter

* check if teacher

* fixed authorizations in projectview (#418)

* fixed authorizations in projectview

* linting

* added locales

* redirecting to homepage when user not logged in (#421)

* redirecting to homepage when user not logged in

* linting

* only fetching admins when needed (#420)

* Fixed url flickering (#423)

* redirecting to homepage when user not logged in

* linting

* fixed url flickering

* fix seeder path (#422)

* added explanation (#425)

* added general evaluator documentation (#424)

* added general evaluator documentation

* fix 1

* fix 2

* fix yallah

* final fix i believe

* Header tests frontend (#391)

* file structure

* added hompage not logged in tests and header login functionality test

* linting + pass current tests hopefully

* more linting

* extra header tests

* other way of testing center button

* third option

* language test

* ignore screenshots

* logged in test

* formatting

* extra formatting

* deleted unused files

* intercept doesn't work on runner

* removed comments

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Jarne Clauw <67628242+JarneClauw@users.noreply.github.com>
Co-authored-by: warre <warre.provoost@ugent.be>
Co-authored-by: gerwoud <gerwoud@hotmail.be>
Co-authored-by: Siebe Vlietinck <71773032+Vucis@users.noreply.github.com>
Co-authored-by: Warre Provoost <133233646+warreprovoost@users.noreply.github.com>
Co-authored-by: Gerwoud Van den Eynden <62761483+Gerwoud@users.noreply.github.com>
Co-authored-by: Cedric Mekeirle <143823820+JibrilExe@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Matisse Sulzer <160239816+Matisse-Sulzer@users.noreply.github.com>
Co-authored-by: Siebe Vlietinck <siebe.vlietinck@ugent.be>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant