- incorporate fixes from test.py (already in master)

- update BUGS.md

- incorporate fixes from test.py (already in master)
- update BUGS.md
Miguel Barão
1 parent dbbd7c2a
Showing 4 changed files with 21 additions and 13 deletions Show diff stats
.gitignore
BUGS.md
perguntations/__init__.py
perguntations/test.py
@@ -9,4 +9,7 @@ static/lib
 /*.eggnode_modules
 # ignore javascript libraries installed with npm
-node_modules/
 \ No newline at end of file
+node_modules/
+
+# ignore documentation until it is actually written
+doc/
 \ No newline at end of file
 # BUGS
-- quando se clica no texto de uma opcao, salta para outro lado na pagina.
+- codigo `hello   world` nao esta a preservar o whitespace. O renderer de markdown gera a tag <code> que não preserva whitespace. Necessario adicionar <pre>.
+- teste nao esta a mostrar imagens.
+- se houver erros a abrir ficheiros .yaml de perguntas, depois dos testes diz "No errors found".
+- dizer quanto desconta em cada pergunta de escolha multipla
 - ordenacao das notas em /admin nao é numerica, é ascii...
 - mensagems de erro do assembler aparecem na mesma linha na correcao e nao fazerm rendering do `$t`, ver se servidor faz parse do markdown dessas mensagens.
 - impedir os eventos copy/paste. alunos usam isso para trazer codigo ja feito nos computadores. Obrigar a fazer reset? fazer um copy automaticamente?
@@ -12,9 +15,11 @@ ou usar push (websockets?)
 - lidar com eventos unfocus.
 - servidor nao esta a lidar com eventos scroll/resize. ignorar?
 - Test.reset_answers() unused.
+- mudar ref do test para test_id (ref já é usado nas perguntas)
 # TODO
+- nao esta a usar points das perguntas
 - test: mostrar duração do teste com progressbar no navbar.
 - submissao fazer um post ajax?
 - adicionar opcao para eliminar um teste em curso.
@@ -54,6 +59,7 @@ ou usar push (websockets?)
 # FIXED
+- quando se clica no texto de uma opcao, salta para outro lado na pagina.
 - 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.
 - fazer package para instalar perguntations com pip.
 - pymips: nao pode executar syscalls do spim.
@@ -32,7 +32,7 @@ proof of submission and for review.
 '''
 APP_NAME = 'perguntations'
-APP_VERSION = '2019.02.dev1'
+APP_VERSION = '2019.06.dev1'
 APP_DESCRIPTION = __doc__
 __author__ = 'Miguel Barão'
@@ -133,13 +133,11 @@ class TestFactory(dict):
         testfile = path.join(path.expanduser(self['answers_dir']), 'REMOVE-ME')
         try:  # check if answers_dir is a writable directory
-            f = open(testfile, 'w')
+            with open(testfile, 'w') as f:
+                f.write('You can safely remove this file.')
         except OSError:
             logger.critical(f'Can\'t write answers to "{self["answers_dir"]}"')
             raise TestFactoryException()
-        else:
-            with f:
-                f.write('You can safely remove this file.')
         # --- ref
         if 'ref' not in self:
@@ -176,8 +174,8 @@ class TestFactory(dict):
         for i, q in enumerate(self['questions']):
             # normalize question to a dict and ref to a list of references
-            if isinstance(q, str):
-                q = {'ref': [q]}
+            if isinstance(q, str):  # e.g.,    - some_ref
+                q = {'ref': [q]}    # becomes  - ref: [some_ref]
             elif isinstance(q, dict) and isinstance(q['ref'], str):
                 q['ref'] = [q['ref']]
@@ -191,8 +189,9 @@ class TestFactory(dict):
         test = []
         # total_points = 0.0
-        n = 1
-        nerr = 0
+        n = 1     # track question number
+        nerr = 0  # count errors generating questions
+
         for qq in self['questions']:
             # choose one question variant
             qref = random.choice(qq['ref'])
@@ -207,9 +206,9 @@ class TestFactory(dict):
             # some defaults
             if q['type'] in ('information', 'success', 'warning', 'alert'):
-                q.setdefault('points', 0.0)
+                q['points'] = qq.get('points', 0.0)
             else:
-                q.setdefault('points', 1.0)
+                q['points'] = qq.get('points', 1.0)
                 q['number'] = n  # counter for non informative panels
                 n += 1
	@@ -32,7 +32,7 @@ proof of submission and for review.		@@ -32,7 +32,7 @@ proof of submission and for review.
32	'''	32	'''
33		33
34	APP_NAME = 'perguntations'	34	APP_NAME = 'perguntations'
35	-APP_VERSION = '2019.02.dev1'	35	+APP_VERSION = '2019.06.dev1'
36	APP_DESCRIPTION = __doc__	36	APP_DESCRIPTION = __doc__
37		37
38	__author__ = 'Miguel Barão'	38	__author__ = 'Miguel Barão'