Fix unicode error in subsection

[sarina]

@chrisndodge this fixes a bug brought up on the mailing list, see my response to this: https://groups.google.com/forum/#!topic/edx-code/cdsQafpz0aA

@wedaly I don't know the best place to add a test for creating a course with sections, subsections, and problem names with Unicode, but obviously this is a needed test! Can you please advise?

[wedaly]

@sarina For this PR, it's sufficient to add unit tests for just the functions you changed.

We should also audit the fixtures used in view-level tests in lms/courseware and add non-ASCII unicode to course names/sections/subsections. This is going to be a longer task -- could you create a ticket in the LMS backlog for this?

[wedaly]

👍

[adampalay]

👍

[sarina]

thanks @davestgermain ! Tests look great.

[sarina]

oop - hadn't rebased so tests failed. Rebased to pick up fixes.

[davestgermain]

@sarina I did, too! Fun!

[sarina]

@davestgermain hmmmmm i hope that doesn't cause issues with the branch. I also squashed the two commits as logically they should be - the first commit shouldn't independently pass tests.

[davestgermain]

@sarina Oh, hmm. Check again; I may have clobbered that squash.

I'll just back away slowly...

[sarina]

@davestgermain that's fine. I'm going to let you handle merging this PR - I wasn't sure who should take responsibility since I initially opened it. Squashing would be preferred, I think, but no huge deal this once.