fix: add nowait for asyncio disconnect

[hyeongguen-song]

please check this.

relative pr redis/redis-py#2356

without this, you can meet this

redis/asyncio/connection.py:767: TypeError

    async def send_packed_command(
        self, command: Union[bytes, str, Iterable[bytes]], check_health: bool = True
    ) -> None:
        if not self.is_connected:
            await self.connect()
        elif check_health:
            await self.check_health()
    
        try:
            if isinstance(command, str):
                command = command.encode()
            if isinstance(command, bytes):
                command = [command]
            if self.socket_timeout:
                await asyncio.wait_for(
                    self._send_packed_command(command), self.socket_timeout
                )
            else:
                self._writer.writelines(command)
                await self._writer.drain()
        except asyncio.TimeoutError:
            await self.disconnect(nowait=True)
            raise TimeoutError("Timeout writing to socket") from None
        except OSError as e:
            await self.disconnect(nowait=True)
            if len(e.args) == 1:
                err_no, errmsg = "UNKNOWN", e.args[0]
            else:
                err_no = e.args[0]
                errmsg = e.args[1]
            raise ConnectionError(
                f"Error {err_no} while writing to socket. {errmsg}."
            ) from e
        except Exception:
>           await self.disconnect(nowait=True)
E           TypeError: FakeConnection.disconnect() got an unexpected keyword argument 'nowait'

[cunla]

Can you add a test for it?

[hyeongguen-song]

@cunla
you can test it with above 4.4.0rc2

[cunla]

Thx for writing this, but the test does not test the disconnect() method you changed..

[hyeongguen-song]

I don't know what should I do.. should I have to test nowait of redis-py?
When you test with 4.4.0rc2 without this change and run test, you can see error.

[cunla]

Hi, as you can see, tests fail for redis-py below 4.4.
Please update your implementation to **kwargs and add a test for using this parameter.

[cunla]

It passes, the failure is due to a github action bad permissions which I will fix separately.