My change was merely renaming a variable to make it clear that it's not the
model_data for the module but for the descriptor. The resulting code is
compile == the previous code. I do not believe such changes necessitate
writing tests which the initial coder neglected.


On Tue, Jun 25, 2013 at 9:57 AM, Jay Zoldak <notifications@github.com>wrote:

> In cms/djangoapps/contentstore/views/session_kv_store.py:
>
> > @@ -2,27 +2,27 @@
> >
> >
> >  class SessionKeyValueStore(KeyValueStore):
> > -    def __init__(self, request, model_data):
> > -        self._model_data = model_data
> > +    def __init__(self, request, descriptor_model_data):
>
> Please add unit tests for the code in this file.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-platform/pull/239/files#r4866255>
> .
>