isinstance sometimes puts closing paren in wrong place

[bmerry]

class TestFoo(unittest.TestCase):
    def test_foo(self):
        self.assertIsInstance(x, y)
        self.assertIsInstance(x, y)

gets rewritten to

class TestFoo:
    def test_foo(self):
        assert isinstance(x), y
        assert isinstance(x, y)

I did a bit of debugging, and I think the issue is that the search for a second comma (indicating a failure message) is too aggressive and continues past the end of the call and onto the next line. It needs to be constrained to not search past the close paren.

[dannysepler]

great catch. appreciate you tracking down the reason

[dannysepler]

fixed in v1.4.1