Fix failing tests

README.md

@@ -71,6 +71,9 @@ Custom `ipfs.toml` must define the following keys,
 whose default values are listed as follows:
 
 ```toml
+[api]
+base = 'http://127.0.0.1:5001/api/v0'
+
 [gateway]
 base = 'http://127.0.0.1:8080/ipfs'
 fallback = 'https://ipfs.io'

src/acanban/project.py

@@ -119,7 +119,7 @@ async def edit(uuid: str) -> ResponseReturnValue:
 async def report(uuid: str) -> ResponseReturnValue:
     """Return the projects' report infomation and forms."""
     user = await current_user.pluck('projects', 'role')
-    project = await pluck(uuid, ('id', 'name', 'report'), user['projects'])
+    project = await pluck(uuid, ('id', 'name', 'report'), user.get('projects'))
     query = r.table('files').get_all(*project['report']['revisions'])
     async with current_app.db_pool.connection() as conn:
         revisions = [file async for file in await query.run(conn)]
@@ -134,7 +134,7 @@ async def report(uuid: str) -> ResponseReturnValue:
 async def report_upload(uuid: str) -> ResponseReturnValue:
     """Handle report upload."""
     user = await current_user.pluck('role', 'projects')
-    await pluck(uuid, projects=user['projects'])
+    await pluck(uuid, projects=user.get('projects'))
     if user['role'] != 'student': raise Unauthorized
     action = r.row['report']['revisions'].append(await ipfs_add())
     async with current_app.db_pool.connection() as conn:
@@ -148,7 +148,7 @@ async def report_upload(uuid: str) -> ResponseReturnValue:
 async def report_eval(uuid: str) -> ResponseReturnValue:
     """Handle report evaluation."""
     user = await current_user.pluck('role', 'projects')
-    await pluck(uuid, projects=user['projects'])
+    await pluck(uuid, projects=user.get('projects'))
     if user['role'] == 'student': raise Unauthorized
     form = await request.form
     updated = {'grade': float(form['grade']), 'comment': form['comment']}

src/acanban/user.py

@@ -43,13 +43,11 @@ async def view_user_profile(username: str) -> ResponseReturnValue:
         if user is None:
             raise NotFound
         project_uuids = user.get('projects', None)
-        if project_uuids is not None:
-            project_list = r.table('projects').get_all(*project_uuids)
-            projects = await project_list.run(conn)
-            return await render_template(
-                'user.html', user=user, projects=projects)
-        else:
+        if project_uuids is None:
             return await render_template('user.html', user=user)
+        project_list = r.table('projects').get_all(*project_uuids)
+        projects = await project_list.run(conn)
+        return await render_template('user.html', user=user, projects=projects)
 
 
 @blueprint.route('/<username>/edit', methods=['GET', 'POST'])

tests/test_user.py

@@ -20,12 +20,14 @@
 from http import HTTPStatus as Status
 
 from conftest import ClientFactory, parametrize
+from pytest import param
 from quart.testing import QuartClient
 
 
 @parametrize(('username', 'status_code'),
-             (('nexistepas', Status.NOT_FOUND),
-              ('silasl', Status.OK)))
+             (param('nexistepas', Status.NOT_FOUND, id='not exist'),
+              param('adaml', Status.OK, id='can have projects'),
+              param('silasl', Status.OK, id='cannot have projects')))
 async def test_get(username: str, status_code: int,
                    client: QuartClient) -> None:
     """Test GET user root endpoint."""