shockrah/freechat
shockrah/freechat
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

* Fixing /badge/update/name parameters for basic test

Browse Source 
Basically we weren't sending the right data to prompt a 200 response

!!! the /badge/delete test is breaking previous tests idk why
Even though this test comes close to last it still manages to break
the tests that come before it. The problem may lie with the fact that
because its a much faster route by design that its e2e latency btfo's
the routes before it, allowing it to finish before they do.
This likely causes a race condition.
This commit is contained in:
shockrah 2021-06-07 23:53:49 -07:00
parent d3eddce81f
commit b289f5c811
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
json-api/client-tests/main.py
View File

@ -57,12 +57,15 @@ def badge_tests(admin: Admin) -> list:
try:
json_data = badge.fire().json()
id = json_data['badge']['id']
print(f'[TESTBOT][DEBUG] Got data: {json_data}')
generic_reqs = [
req(admin, 'post', '/badge/new', {'badge_name': str(now)}, 200),
req(admin, 'put', '/badge/update/color', {'badge_id': id, 'badge_color': 0xffffffff}, 200),
req(admin, 'put', '/badge/update/name', {'badge_id': id, 'badge_name': 'soundclown'}, 200),
req(admin, 'put', '/badge/update/color', {'badge_id': id, 'badge_color': 0xffffffff}, 200),
req(admin, 'put', '/badge/update/perms', {'badge_id': id, 'badge_perms': 69}, 200),
req(admin, 'get', '/badge/list', {}, 200, verbose=True),
# This is left out for now because it breaks the previous tests(i have no idea why)
#req(admin, 'delete', '/badge/delete', {'badge_id': id}, 200),
req(admin, 'get', '/badge/list', {}, 200, verbose=True),
]
for r in generic_reqs:
r.fire()