From 23f1db87e6ed2d697e2d2b8c9988c773693e13e8 Mon Sep 17 00:00:00 2001
From: Miguel Barão <miguel.barao@gmail.com>
Date: Sun, 18 Nov 2018 16:57:19 +0000
Subject: [PATCH] updated BUGS.md and minor changes in the code

---
 BUGS.md | 8 ++++----
 app.py  | 6 +++---
 test.py | 2 +-
 3 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/BUGS.md b/BUGS.md
index 3ba06fa..8843348 100644
--- a/BUGS.md
+++ b/BUGS.md
@@ -1,9 +1,7 @@
 
 # BUGS
 
-- numeric interval deve converter respostas que usam virgulas para pontos decimais
 - a revisao do teste não mostra as imagens.
-- se aluno tem teste activo e é allowed uma segunda vez, deve manter o mesmo teste. adicionar opcao para eliminar um teste em curso.
 - melhorar o botao de autorizar (desliga-se), usar antes um botao?
 e.g. retornar None quando nao ha alteracoes relativamente à última vez.
 ou usar push (websockets?)
@@ -16,8 +14,8 @@ ou usar push (websockets?)
 
 # TODO
 
-- decorador para user 0, evita o "if uid==0" em muitas funcoes.
-- gerar teste qd o prof autoriza? melhor nao, pode apagar o teste em curso.
+- adicionar opcao para eliminar um teste em curso.
+- gerar teste qd o prof autoriza? melhor nao, pode apagar o teste em curso. gerar previamente e manter uma pool de testes gerados?
 - enviar resposta de cada pergunta individualmente.
 - experimentar gerador de svg que inclua no markdown da pergunta e ver se funciona. 
 - suportar cotacao to teste diferente de 20 (e.g. para juntar perguntas em papel). opcao "points: 18" que normaliza total para 18 em vez de 20.
@@ -61,6 +59,8 @@ ou usar push (websockets?)
 
 # FIXED
 
+- decorador para user 0, evita o "if uid==0" em muitas funcoes.
+- numeric interval deve converter respostas que usam virgulas para pontos decimais
 - self.testapp.get_json_filename_of_test(test_id) retorna None quando test_id nao existe.
 - o eventloop está a bloquear. correção do teste é blocking. usar threadpoolexecutor?
 - substituir get_event_loop por get_runnint_loop (ver https://docs.python.org/3/library/asyncio-eventloop.html)
diff --git a/app.py b/app.py
index 756b5a6..33fff7f 100644
--- a/app.py
+++ b/app.py
@@ -138,10 +138,10 @@ class App(object):
     # -----------------------------------------------------------------------
     async def generate_test(self, uid):
         if uid in self.online:
-            logger.info(f'Student {uid}: generating new test.')
-            student_id = self.online[uid]['student']
+            logger.info(f'Student {uid}: started generating new test.')
+            student_id = self.online[uid]['student']  # {number, name}
             self.online[uid]['test'] = await self.testfactory.generate(student_id)
-            logger.debug(f'Student {uid}: test ok.')
+            logger.debug(f'Student {uid}: finished generating test.')
             return self.online[uid]['test']
         else:
             # this implies an error in the code. should never be here!
diff --git a/test.py b/test.py
index a9d8a83..ea5e2fa 100644
--- a/test.py
+++ b/test.py
@@ -216,7 +216,7 @@ class Test(dict):
         self['finish_time'] = None
         self['state'] = 'ONGOING'
         self['comment'] = ''
-        logger.info(f'Student {self["student"]["number"]}: starting test.')
+        logger.info(f'Student {self["student"]["number"]}:  new test.')
 
     # -----------------------------------------------------------------------
     # Removes all answers from the test (clean)
--
libgit2 0.21.2