Skip to content

E
edx-platform
Commit d02dc37d by cahrens
Options

On course about page, display course end date if no about page HTML blog (end_date.html) exists. 

#224
parent 56d5f13b
Showing with 100 additions and 20 deletions
common/lib/xmodule/xmodule/course_module.py
...@@ -652,7 +652,12 @@ class CourseDescriptor(CourseFields, SequenceDescriptor): ...@@ -652,7 +652,12 @@ class CourseDescriptor(CourseFields, SequenceDescriptor):
@property @property
def end_date_text(self): def end_date_text(self):
return time.strftime("%b %d, %Y", self.end) """
Returns the end date for the course formatted as a string.
If the course does not have an end date set (course.end is None), an empty string will be returned.
"""
return '' if self.end is None else time.strftime("%b %d, %Y", self.end)
@property @property
def forum_posts_allowed(self): def forum_posts_allowed(self):
......
common/lib/xmodule/xmodule/tests/test_course_module.py
...@@ -54,7 +54,7 @@ class IsNewCourseTestCase(unittest.TestCase): ...@@ -54,7 +54,7 @@ class IsNewCourseTestCase(unittest.TestCase):
self.addCleanup(datetime_patcher.stop) self.addCleanup(datetime_patcher.stop)
@staticmethod @staticmethod
def get_dummy_course(start, announcement=None, is_new=None, advertised_start=None): def get_dummy_course(start, announcement=None, is_new=None, advertised_start=None, end=None):
"""Get a dummy course""" """Get a dummy course"""
system = DummySystem(load_error_modules=True) system = DummySystem(load_error_modules=True)
...@@ -65,6 +65,7 @@ class IsNewCourseTestCase(unittest.TestCase): ...@@ -65,6 +65,7 @@ class IsNewCourseTestCase(unittest.TestCase):
is_new = to_attrb('is_new', is_new) is_new = to_attrb('is_new', is_new)
announcement = to_attrb('announcement', announcement) announcement = to_attrb('announcement', announcement)
advertised_start = to_attrb('advertised_start', advertised_start) advertised_start = to_attrb('advertised_start', advertised_start)
end = to_attrb('end', end)
start_xml = ''' start_xml = '''
<course org="{org}" course="{course}" <course org="{org}" course="{course}"
...@@ -72,13 +73,14 @@ class IsNewCourseTestCase(unittest.TestCase): ...@@ -72,13 +73,14 @@ class IsNewCourseTestCase(unittest.TestCase):
start="{start}" start="{start}"
{announcement} {announcement}
{is_new} {is_new}
{advertised_start}> {advertised_start}
{end}>
<chapter url="hi" url_name="ch" display_name="CH"> <chapter url="hi" url_name="ch" display_name="CH">
<html url_name="h" display_name="H">Two houses, ...</html> <html url_name="h" display_name="H">Two houses, ...</html>
</chapter> </chapter>
</course> </course>
'''.format(org=ORG, course=COURSE, start=start, is_new=is_new, '''.format(org=ORG, course=COURSE, start=start, is_new=is_new,
announcement=announcement, advertised_start=advertised_start) announcement=announcement, advertised_start=advertised_start, end=end)
return system.process_xml(start_xml) return system.process_xml(start_xml)
...@@ -161,3 +163,11 @@ class IsNewCourseTestCase(unittest.TestCase): ...@@ -161,3 +163,11 @@ class IsNewCourseTestCase(unittest.TestCase):
descriptor = self.get_dummy_course(start='2012-12-31T12:00') descriptor = self.get_dummy_course(start='2012-12-31T12:00')
assert(descriptor.is_newish is True) assert(descriptor.is_newish is True)
def test_end_date_text(self):
# No end date set, returns empty string.
d = self.get_dummy_course('2012-12-02T12:00')
self.assertEqual('', d.end_date_text)
d = self.get_dummy_course('2012-12-02T12:00', end='2014-9-04T12:00')
self.assertEqual('Sep 04, 2014', d.end_date_text)
common/test/data/test_about_blob_end_date/README.md 0 → 100644
Test course for checking the end date displayed on the course about page.
This course has both an end_date HTML "blob", and it also has a course end date set.
The end_date "blob" has higher precedence and will show.
See also test_end course.
\ No newline at end of file
common/test/data/test_about_blob_end_date/about/end_date.html 0 → 100644
Learning never ends
\ No newline at end of file
common/test/data/test_about_blob_end_date/course.xml 0 → 100644
<course org="edX" course="test_about_blob_end_date" url_name="2012_Fall"/>
\ No newline at end of file
common/test/data/test_about_blob_end_date/policies/2012_Fall.json 0 → 100644
{
"course/2012_Fall": {
"graceperiod": "2 days 5 hours 59 minutes 59 seconds",
"start": "2015-07-17T12:00",
"end": "2015-09-17T12:00",
"display_name": "Test About Blob End Date",
"graded": "true"
}
}
common/test/data/test_end/README.md 0 → 100644
Test course for checking the end date displayed on the course about page.
This course does not have an end_date HTML "blob", but it does have a course end date set.
Therefore the course end date should show on the course about page.
See also test_about_blob_end_date course.
\ No newline at end of file
common/test/data/test_end/course.xml 0 → 100644
<course org="edX" course="test_end" url_name="2012_Fall"/>
\ No newline at end of file
common/test/data/test_end/policies/2012_Fall.json 0 → 100644
{
"course/2012_Fall": {
"graceperiod": "2 days 5 hours 59 minutes 59 seconds",
"start": "2015-07-17T12:00",
"end": "2015-09-17T12:00",
"display_name": "Test End",
"graded": "true"
}
}
lms/djangoapps/courseware/tests/test_views.py
import logging from mock import MagicMock
from mock import MagicMock, patch
import datetime import datetime
import factory
import unittest
import os
from django.test import TestCase from django.test import TestCase
from django.http import Http404, HttpResponse from django.http import Http404
from django.conf import settings from django.conf import settings
from django.test.utils import override_settings from django.test.utils import override_settings
from django.contrib.auth.models import User from django.contrib.auth.models import User
from django.test.client import RequestFactory from django.test.client import RequestFactory
from student.models import CourseEnrollment from student.models import CourseEnrollment
from xmodule.modulestore.django import modulestore, _MODULESTORES from xmodule.modulestore.django import modulestore
from xmodule.modulestore.exceptions import InvalidLocationError,\
ItemNotFoundError, NoPathToItem
import courseware.views as views import courseware.views as views
from xmodule.modulestore import Location from xmodule.modulestore import Location
from .factories import UserFactory
class Stub(): class Stub():
pass pass
...@@ -55,7 +48,6 @@ class TestJumpTo(TestCase): ...@@ -55,7 +48,6 @@ class TestJumpTo(TestCase):
def test_jumpto_invalid_location(self): def test_jumpto_invalid_location(self):
location = Location('i4x', 'edX', 'toy', 'NoSuchPlace', None) location = Location('i4x', 'edX', 'toy', 'NoSuchPlace', None)
jumpto_url = '%s/%s/jump_to/%s' % ('/courses', self.course_name, location) jumpto_url = '%s/%s/jump_to/%s' % ('/courses', self.course_name, location)
expected = 'courses/edX/toy/2012_Fall/courseware/Overview/'
response = self.client.get(jumpto_url) response = self.client.get(jumpto_url)
self.assertEqual(response.status_code, 404) self.assertEqual(response.status_code, 404)
...@@ -124,3 +116,26 @@ class ViewsTestCase(TestCase): ...@@ -124,3 +116,26 @@ class ViewsTestCase(TestCase):
request, 'bar', ()) request, 'bar', ())
self.assertRaisesRegexp(Http404, 'No data*', views.jump_to, request, self.assertRaisesRegexp(Http404, 'No data*', views.jump_to, request,
'dummy', self.location) 'dummy', self.location)
def test_no_end_on_about_page(self):
# Toy course has no course end date or about/end_date blob
self.verify_end_date(self.course_id)
def test_no_end_about_blob(self):
# test_end has a course end date, no end_date HTML blob
self.verify_end_date('edX/test_end/2012_Fall', "Sep 17, 2015")
def test_about_blob_end_date(self):
# test_about_blob_end_date has both a course end date, and an end_date HTML blob
# HTML blob wins
self.verify_end_date('edX/test_about_blob_end_date/2012_Fall', "Learning never ends")
def verify_end_date(self, course_id, expected_end_text=None):
request = self.request_factory.get('foo')
request.user = self.user
result = views.course_about(request, course_id)
if expected_end_text is not None:
self.assertContains(result, "Classes End")
self.assertContains(result, expected_end_text)
else:
self.assertNotContains(result, "Classes End")
lms/templates/courseware/course_about.html
...@@ -144,10 +144,20 @@ ...@@ -144,10 +144,20 @@
<li><div class="icon course-number"></div><p>Course Number</p><span class="course-number">${course.number}</span></li> <li><div class="icon course-number"></div><p>Course Number</p><span class="course-number">${course.number}</span></li>
<li><div class="icon start"></div><p>Classes Start</p><span class="start-date">${course.start_date_text}</span></li> <li><div class="icon start"></div><p>Classes Start</p><span class="start-date">${course.start_date_text}</span></li>
## End date should come from course.xml, but this is a quick hack ## We plan to ditch end_date (which is not stored in course metadata),
% if get_course_about_section(course, "end_date"): ## but for backwards compatibility, show about/end_date blob if it exists.
<li><div class="icon end"></div><p>Classes End</p><span class="final-date">${get_course_about_section(course, "end_date")}</span></li> % if get_course_about_section(course, "end_date") or course.end:
% endif <li>
<div class="icon end"></div>
<p>Classes End</p><span class="final-date">
% if get_course_about_section(course, "end_date"):
${get_course_about_section(course, "end_date")}
% else:
${course.end_date_text}
% endif
</span>
</li>
% endif
% if get_course_about_section(course, "effort"): % if get_course_about_section(course, "effort"):
<li><div class="icon effort"></div><p>Estimated Effort</p><span class="start-date">${get_course_about_section(course, "effort")}</span></li> <li><div class="icon effort"></div><p>Estimated Effort</p><span class="start-date">${get_course_about_section(course, "effort")}</span></li>
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment