Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Parse set arguments #49

Merged
merged 4 commits into from
Aug 30, 2024
Merged

Parse set arguments #49

merged 4 commits into from
Aug 30, 2024

Conversation

gillchristian
Copy link
Collaborator

  • Implemented the get behavior
  • Implemented the conditional set behavior (NX | XX)
  • TTL not implemented

@@ -97,4 +194,272 @@ mod tests {
assert_eq!(res, Frame::Simple("OK".to_string()));
assert_eq!(store.lock().get("key1"), Some(Bytes::from("2")));
}

#[tokio::test]
async fn ttl_exat_and_xx_behavior() {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

exat was not used on this test right?

@gillchristian gillchristian force-pushed the feat/cmds/set-arguments branch from 985c8f9 to 3046bcc Compare August 23, 2024 07:38
@gillchristian gillchristian force-pushed the feat/cmds/set-arguments branch from 3046bcc to b82a531 Compare August 23, 2024 08:05
@gillchristian gillchristian merged commit 36802bf into master Aug 30, 2024
0 of 2 checks passed
@gillchristian gillchristian deleted the feat/cmds/set-arguments branch August 30, 2024 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants