Adding new tests to increase coverage

The only other thing I can think of is that the conjunction of
requires_binary and PY2 should be a disjunction instead?
On Thu, Oct 30, 2014 at 10:10 PM Dan Blanchard <notifications@github.com>
wrote:

> In skll/data/writers.py:
>
> > @@ -353,6 +353,11 @@ def _write_header(self, feature_set, output_file):
> >  class MegaMWriter(FeatureSetWriter):
> >
> >      """ FeatureSetWriter for writing out FeatureSets as MegaM files. """
> > +
> > +    def __init__(self, path, feature_set, **kwargs):
> > +        kwargs['requires_binary'] = True
>
> It doesn't make any sense to me that we would need to have requires_binary
> set to True for MegaM files. That flag is meant for things that use
> DictWriter and the json library that stubbornly do not support unicode
> output on Python 2.
>
> We're just printing stuff to a file, and that should work fine with
> unicode on Python 2. I understand that it was crashing, but I'm not
> convinced this is the correct solution.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EducationalTestingService/skll/pull/191/files#r19648350>
> .
>