TireSwingSoftware / openassign-server

OpenAssign server intended for use by a separate client via RPC
BSD 3-Clause "New" or "Revised" License
6 stars 2 forks source link

Failing test pr_services.pr_tests.TestScormServer #108

Open jc0n opened 12 years ago

jc0n commented 12 years ago
ERROR: test_mark_completed (pr_services.pr_tests.TestScormServer)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/jcon/Desktop/projects/openassign-server/pr_services/pr_tests.py", line 3785, in test_mark_completed
    response = self.client.get(sco['url'])
  File "/usr/local/lib/python2.6/dist-packages/django/test/client.py", line 445, in get
    response = super(Client, self).get(path, data=data, **extra)
  File "/usr/local/lib/python2.6/dist-packages/django/test/client.py", line 229, in get
    return self.request(**r)
  File "/usr/local/lib/python2.6/dist-packages/django/core/handlers/base.py", line 111, in get_response
    response = callback(request, *callback_args, **callback_kwargs)
  File "/home/jcon/Desktop/projects/openassign-server/pr_services/scorm_system/scorm_server.py", line 84, in return_player
    t = loader.get_template('scorm_player')
  File "/usr/local/lib/python2.6/dist-packages/django/template/loader.py", line 157, in get_template
    template, origin = find_template(template_name)
  File "/usr/local/lib/python2.6/dist-packages/django/template/loader.py", line 138, in find_template
    raise TemplateDoesNotExist(name)
TemplateDoesNotExist: scorm_player
jc0n commented 12 years ago

The template is actually missing in the repository. It should be included or the test should be marked as an expected failure for now.

jc0n commented 12 years ago

Just came across this which explains a few things.

https://github.com/TireSwingSoftware/openassign-server/blob/master/pr_services/scorm_system/templates/README.txt

There should be a file here called "scorm_player".  However, ARI does not own the 
rights to it, and thus cannot release it as open-source.  So, we will have to find 
another option.```
mhrivnak commented 12 years ago

This will almost certainly become irrelevant soon. We are going to implement a new cloud-based scorm service, so this test will likely go away completely.