I'll do it during the weekend.
jb:)


2013/2/7 Diogo Baeder <notifications@github.com>

> In tests/unit/core/test_roles.py:
>
> > -            self.role.ensure_line('this line in', '/some/file', sudo=True)
> > -            execute.assert_called_with('echo "this line in" >> /some/file', stdout=False, sudo=True, user=None)
> > -
> > -    @istest
> > -    def inserts_line_with_specific_user(self):
> > -        with self.mock_role_method('has_line') as has_line, self.execute_mock() as execute:
> > -            has_line.return_value = False
> > -            self.role.ensure_line('this line in', '/some/file', owner='foo')
> > -            execute.assert_called_with('echo "this line in" >> /some/file', stdout=False, sudo=False, user='foo')
> > +    def proper_content_in_line_file(self):
> > +        with self.mock_role_method("execute_python_script") as execute, \
> > +                self.mock_role_method("put_file"), \
> > +                self.mock_role_method("create_remote_temp_file") as tmp_file:
> > +            tmp_file.return_value = "/tmp/foo"
> > +            self.role.ensure_line("foo bar", "/etc/baz")
> > +        # self.assertTrue(isinstance(put_file.mock_calls[0][1][0], StringIO))
>
> Do you think you can finish this by next week? Otherwise I'm thinking of
> releasing 0.6.1 without these changes, as we need to have a synchronized
> documentation with the provy version that was released.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/python-provy/provy/pull/113/files#r2926606>.
>
>