Fix set expire (replace push_integer by push_bulk)

[YunoL]

Fixes set expire

[Darksonn]

Can you please elaborate on the motivation behind this change? Did you test it with push_int and find that it doesn't work?

[YunoL]

yes, the exemple provided in https://docs.rs/mini-redis/latest/mini_redis/blocking_client/struct.BlockingClient.html#method.set_expires produice
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: "ERR Protocol error: expected '$', got ':'"', src/main.rs:9:50 note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
when i test it

[Darksonn]

I'm unable to reproduce that myself, though the assert!(val.is_some()); seems like it should test for .is_none() instead. How are you testing this?

Can you include the output with RUST_BACKTRACE=1?

[YunoL]

I created a repo to reproduce the error: https://github.com/YunoL/set_expire, the panic come from client.set_expires("foo", "bar".into(), ttl).unwrap();

here is the backtrace:

RUST_BACKTRACE=1 cargo run
    Finished dev [unoptimized + debuginfo] target(s) in 0.02s
     Running `target/debug/test_redis`
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: "ERR Protocol error: expected '$', got ':'"', src/main.rs:9:50
stack backtrace:
   0: rust_begin_unwind
             at /rustc/d5a82bbd26e1ad8b7401f6a718a9c57c96905483/library/std/src/panicking.rs:575:5
   1: core::panicking::panic_fmt
             at /rustc/d5a82bbd26e1ad8b7401f6a718a9c57c96905483/library/core/src/panicking.rs:64:14
   2: core::result::unwrap_failed
             at /rustc/d5a82bbd26e1ad8b7401f6a718a9c57c96905483/library/core/src/result.rs:1791:5
   3: core::result::Result<T,E>::unwrap
             at /rustc/d5a82bbd26e1ad8b7401f6a718a9c57c96905483/library/core/src/result.rs:1113:23
   4: test_redis::main
             at ./src/main.rs:9:5
   5: core::ops::function::FnOnce::call_once
             at /rustc/d5a82bbd26e1ad8b7401f6a718a9c57c96905483/library/core/src/ops/function.rs:507:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.