Codecov Report

Merging #125 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #125   +/-   ##
=======================================
  Coverage   46.67%   46.67%           
=======================================
  Files           4        4           
  Lines         632      632           
  Branches       64       64           
=======================================
  Hits          295      295           
- Misses        322      324    +2     
+ Partials       15       13    -2

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ccef002...226e996. Read the comment docs.

Thanks. I refactored this together with the help of @GriceTurrble #65, also we have good test coverage for this part.
This is bugged for Values like 0 or False. See the open #106.
If i get you right, you think we should also remove empty iterables?

If i remember correct the value should never be any type of a python iterable. See the next line where we check this:

if isinstance(value, (dict, list, set, tuple)):
            message = 'expected string or datetime datatype, got {},'\
                'for key {} and value {}'.format(
                    type(value), key, str(value))
            raise MWSError(message)

First i thought i should test with duck typing but this is clearer.
So if you have an example where we have to stringify an iterable you could show us.