Split database code into separate modules.

This commit is contained in:
projectmoon 2020-10-24 13:08:23 +00:00 committed by ProjectMoon
parent 4033b343e7
commit 5410c53513
7 changed files with 443 additions and 388 deletions

View File

@ -124,6 +124,7 @@ async fn calculate_dice_amount<'a>(pool: &'a DicePoolWithContext<'a>) -> Result<
let variables = pool let variables = pool
.1 .1
.db .db
.variables
.get_user_variables(&pool.1.room_id, &pool.1.username) .get_user_variables(&pool.1.room_id, &pool.1.username)
.await?; .await?;
@ -583,7 +584,8 @@ mod tests {
let db = Database::new(&tempdir().unwrap()).unwrap(); let db = Database::new(&tempdir().unwrap()).unwrap();
let ctx = Context::new(&db, "roomid", "username", "message"); let ctx = Context::new(&db, "roomid", "username", "message");
db.set_user_variable(&ctx.room_id, &ctx.username, "myvariable", 10) db.variables
.set_user_variable(&ctx.room_id, &ctx.username, "myvariable", 10)
.await .await
.expect("could not set myvariable to 10"); .expect("could not set myvariable to 10");

View File

@ -1,6 +1,6 @@
use crate::cofd::dice::{roll_pool, DicePool, DicePoolWithContext}; use crate::cofd::dice::{roll_pool, DicePool, DicePoolWithContext};
use crate::context::Context; use crate::context::Context;
use crate::db::DataError; use crate::db::errors::DataError;
use crate::dice::ElementExpression; use crate::dice::ElementExpression;
use crate::error::BotError; use crate::error::BotError;
use crate::help::HelpTopic; use crate::help::HelpTopic;
@ -120,7 +120,12 @@ impl Command for GetAllVariablesCommand {
} }
async fn execute(&self, ctx: &Context) -> Execution { async fn execute(&self, ctx: &Context) -> Execution {
let value = match ctx.db.get_user_variables(&ctx.room_id, &ctx.username).await { let value = match ctx
.db
.variables
.get_user_variables(&ctx.room_id, &ctx.username)
.await
{
Ok(variables) => { Ok(variables) => {
let mut variable_list = variables let mut variable_list = variables
.into_iter() .into_iter()
@ -154,6 +159,7 @@ impl Command for GetVariableCommand {
let name = &self.0; let name = &self.0;
let value = match ctx let value = match ctx
.db .db
.variables
.get_user_variable(&ctx.room_id, &ctx.username, name) .get_user_variable(&ctx.room_id, &ctx.username, name)
.await .await
{ {
@ -181,6 +187,7 @@ impl Command for SetVariableCommand {
let value = self.1; let value = self.1;
let result = ctx let result = ctx
.db .db
.variables
.set_user_variable(&ctx.room_id, &ctx.username, name, value) .set_user_variable(&ctx.room_id, &ctx.username, name, value)
.await; .await;
@ -207,6 +214,7 @@ impl Command for DeleteVariableCommand {
let name = &self.0; let name = &self.0;
let value = match ctx let value = match ctx
.db .db
.variables
.delete_user_variable(&ctx.room_id, &ctx.username, name) .delete_user_variable(&ctx.room_id, &ctx.username, name)
.await .await
{ {

395
src/db.rs
View File

@ -1,402 +1,29 @@
use byteorder::LittleEndian; use crate::db::errors::DataError;
use sled::transaction::abort; use crate::db::variables::Variables;
use sled::transaction::{TransactionError, TransactionalTree, UnabortableTransactionError}; use sled::Db;
use sled::{Db, Tree};
use std::collections::HashMap;
use std::path::Path; use std::path::Path;
use std::str;
use thiserror::Error;
use zerocopy::byteorder::I32;
use zerocopy::{AsBytes, LayoutVerified};
/// User variables are stored as little-endian 32-bit integers in the pub mod errors;
/// database. This type alias makes the database code more pleasant to pub mod schema;
/// read. pub mod variables;
type LittleEndianI32Layout<'a> = LayoutVerified<&'a [u8], I32<LittleEndian>>;
const METADATA_KEY: &'static str = "metadata";
const VARIABLE_COUNT_KEY: &'static str = "variable_count";
#[derive(Clone)] #[derive(Clone)]
pub struct Database { pub struct Database {
db: Db, db: Db,
variables: Tree, pub(crate) variables: Variables,
rooms: Tree, //rooms: Tree,
}
//TODO better combining of key and value in certain errors (namely
//I32SchemaViolation).
#[derive(Error, Debug)]
pub enum DataError {
#[error("value does not exist for key: {0}")]
KeyDoesNotExist(String),
#[error("expected i32, but i32 schema was violated")]
I32SchemaViolation,
#[error("expected string, but utf8 schema was violated: {0}")]
Utf8chemaViolation(#[from] std::str::Utf8Error),
#[error("internal database error: {0}")]
InternalError(#[from] sled::Error),
#[error("transaction error: {0}")]
TransactionError(#[from] sled::transaction::TransactionError),
#[error("unabortable transaction error: {0}")]
UnabortableTransactionError(#[from] UnabortableTransactionError),
}
/// This From implementation is necessary to deal with the recursive
/// error type in the error enum. We defined a transaction error, but
/// the only place we use it is when converting from
/// sled::transaction::TransactionError<DataError>. This converter
/// extracts the inner data error from transaction aborted errors, and
/// forwards anything else onward as-is, but wrapped in DataError.
impl From<TransactionError<DataError>> for DataError {
fn from(error: TransactionError<DataError>) -> Self {
match error {
TransactionError::Abort(data_err) => data_err,
TransactionError::Storage(storage_err) => {
DataError::TransactionError(TransactionError::Storage(storage_err))
}
}
}
}
fn to_key(room_id: &str, username: &str, variable_name: &str) -> Vec<u8> {
let mut key = vec![];
key.extend_from_slice(room_id.as_bytes());
key.extend_from_slice(username.as_bytes());
key.extend_from_slice(variable_name.as_bytes());
key
}
fn metadata_key(room_id: &str, username: &str, metadata_key: &str) -> Vec<u8> {
let mut key = vec![];
key.extend_from_slice(room_id.as_bytes());
key.extend_from_slice(METADATA_KEY.as_bytes());
key.extend_from_slice(username.as_bytes());
key.extend_from_slice(metadata_key.as_bytes());
key
}
fn room_variable_count_key(room_id: &str, username: &str) -> Vec<u8> {
metadata_key(room_id, username, VARIABLE_COUNT_KEY)
}
fn to_prefix(room_id: &str, username: &str) -> Vec<u8> {
let mut prefix = vec![];
prefix.extend_from_slice(room_id.as_bytes());
prefix.extend_from_slice(username.as_bytes());
prefix
}
/// Convert bytes to an i32 with zero-copy deserialization. An error
/// is returned if the bytes do not represent an i32.
fn convert_i32(raw_value: &[u8]) -> Result<i32, DataError> {
let layout = LittleEndianI32Layout::new_unaligned(raw_value.as_ref());
if let Some(layout) = layout {
let value: I32<LittleEndian> = *layout;
Ok(value.get())
} else {
Err(DataError::I32SchemaViolation)
}
}
/// Use a transaction to atomically alter the count of variables in
/// the database by the given amount. Count cannot go below 0.
fn alter_room_variable_count(
variables: &TransactionalTree,
room_id: &str,
username: &str,
amount: i32,
) -> Result<i32, DataError> {
let key = room_variable_count_key(room_id, username);
let mut new_count = match variables.get(&key)? {
Some(bytes) => convert_i32(&bytes)? + amount,
None => amount,
};
if new_count < 0 {
new_count = 0;
}
let db_value: I32<LittleEndian> = I32::new(new_count);
variables.insert(key, db_value.as_bytes())?;
Ok(new_count)
} }
impl Database { impl Database {
pub fn new<P: AsRef<Path>>(path: P) -> Result<Database, DataError> { pub fn new<P: AsRef<Path>>(path: P) -> Result<Database, DataError> {
let db = sled::open(path)?; let db = sled::open(path)?;
let variables = db.open_tree("variables")?; let variables = db.open_tree("variables")?;
let rooms = db.open_tree("rooms")?; //let rooms = db.open_tree("rooms")?;
Ok(Database { Ok(Database {
db: db.clone(), db: db.clone(),
variables: variables, variables: Variables(variables),
rooms: rooms, //rooms: rooms,
}) })
} }
pub async fn get_user_variables(
&self,
room_id: &str,
username: &str,
) -> Result<HashMap<String, i32>, DataError> {
let prefix = to_prefix(&room_id, &username);
let prefix_len: usize = prefix.len();
let variables: Result<Vec<_>, DataError> = self
.variables
.scan_prefix(prefix)
.map(|entry| match entry {
Ok((key, raw_value)) => {
//Strips room and username from key, leaving behind name.
let variable_name = str::from_utf8(&key[prefix_len..])?;
Ok((variable_name.to_owned(), convert_i32(&raw_value)?))
}
Err(e) => Err(e.into()),
})
.collect();
//Convert I32 to hash map. Can we do this in the first mapping
//step instead? For some reason this is faster.
variables.map(|entries| entries.into_iter().collect())
}
pub async fn get_variable_count(
&self,
room_id: &str,
username: &str,
) -> Result<i32, DataError> {
let key = room_variable_count_key(room_id, username);
if let Some(raw_value) = self.variables.get(&key)? {
convert_i32(&raw_value)
} else {
Ok(0)
}
}
pub async fn get_user_variable(
&self,
room_id: &str,
username: &str,
variable_name: &str,
) -> Result<i32, DataError> {
let key = to_key(room_id, username, variable_name);
if let Some(raw_value) = self.variables.get(&key)? {
convert_i32(&raw_value)
} else {
Err(DataError::KeyDoesNotExist(String::from_utf8(key).unwrap()))
}
}
pub async fn set_user_variable(
&self,
room_id: &str,
username: &str,
variable_name: &str,
value: i32,
) -> Result<(), DataError> {
self.variables
.transaction(|tx| {
let key = to_key(room_id, username, variable_name);
let db_value: I32<LittleEndian> = I32::new(value);
let old_value = tx.insert(key, db_value.as_bytes())?;
//Only increment variable count on new keys.
if let None = old_value {
match alter_room_variable_count(&tx, room_id, username, 1) {
Err(e) => abort(e),
_ => Ok(()),
}
} else {
Ok(())
}
})
.map_err(|e| e.into())
}
pub async fn delete_user_variable(
&self,
room_id: &str,
username: &str,
variable_name: &str,
) -> Result<(), DataError> {
self.variables
.transaction(|tx| {
let key = to_key(room_id, username, variable_name);
if let Some(_) = tx.remove(key.clone())? {
match alter_room_variable_count(&tx, room_id, username, -1) {
Err(e) => abort(e),
_ => Ok(()),
}
} else {
abort(DataError::KeyDoesNotExist(String::from_utf8(key).unwrap()))
}
})
.map_err(|e| e.into())
}
}
#[cfg(test)]
mod tests {
use super::*;
use tempfile::tempdir;
//Room Variable count tests
#[tokio::test]
async fn alter_room_variable_count_test() {
let db = Database::new(&tempdir().unwrap()).unwrap();
let alter_count = |amount: i32| {
db.variables
.transaction(|tx| {
match alter_room_variable_count(&tx, "room", "username", amount) {
Err(e) => abort(e),
_ => Ok(()),
}
})
.expect("got transaction failure");
};
async fn get_count(db: &Database) -> i32 {
db.get_variable_count("room", "username")
.await
.expect("could not get variable count")
}
//addition
alter_count(5);
assert_eq!(5, get_count(&db).await);
//subtraction
alter_count(-3);
assert_eq!(2, get_count(&db).await);
}
#[tokio::test]
async fn alter_room_variable_count_cannot_go_below_0_test() {
let db = Database::new(&tempdir().unwrap()).unwrap();
db.variables
.transaction(
|tx| match alter_room_variable_count(&tx, "room", "username", -1000) {
Err(e) => abort(e),
_ => Ok(()),
},
)
.expect("got transaction failure");
let count = db
.get_variable_count("room", "username")
.await
.expect("could not get variable count");
assert_eq!(0, count);
}
#[tokio::test]
async fn empty_db_reports_0_room_variable_count_test() {
let db = Database::new(&tempdir().unwrap()).unwrap();
let count = db
.get_variable_count("room", "username")
.await
.expect("could not get variable count");
assert_eq!(0, count);
}
#[tokio::test]
async fn set_user_variable_increments_count() {
let db = Database::new(&tempdir().unwrap()).unwrap();
db.set_user_variable("room", "username", "myvariable", 5)
.await
.expect("could not insert variable");
let count = db
.get_variable_count("room", "username")
.await
.expect("could not get variable count");
assert_eq!(1, count);
}
#[tokio::test]
async fn update_user_variable_does_not_increment_count() {
let db = Database::new(&tempdir().unwrap()).unwrap();
db.set_user_variable("room", "username", "myvariable", 5)
.await
.expect("could not insert variable");
db.set_user_variable("room", "username", "myvariable", 10)
.await
.expect("could not update variable");
let count = db
.get_variable_count("room", "username")
.await
.expect("could not get variable count");
assert_eq!(1, count);
}
// Set/get/delete variable tests
#[tokio::test]
async fn set_and_get_variable_test() {
let db = Database::new(&tempdir().unwrap()).unwrap();
db.set_user_variable("room", "username", "myvariable", 5)
.await
.expect("could not insert variable");
let value = db
.get_user_variable("room", "username", "myvariable")
.await
.expect("could not get value");
assert_eq!(5, value);
}
#[tokio::test]
async fn delete_variable_test() {
let db = Database::new(&tempdir().unwrap()).unwrap();
db.set_user_variable("room", "username", "myvariable", 5)
.await
.expect("could not insert variable");
db.delete_user_variable("room", "username", "myvariable")
.await
.expect("could not delete value");
let result = db.get_user_variable("room", "username", "myvariable").await;
assert!(result.is_err());
assert!(matches!(result, Err(DataError::KeyDoesNotExist(_))));
}
#[tokio::test]
async fn get_missing_variable_returns_key_does_not_exist() {
let db = Database::new(&tempdir().unwrap()).unwrap();
let result = db.get_user_variable("room", "username", "myvariable").await;
assert!(result.is_err());
assert!(matches!(result, Err(DataError::KeyDoesNotExist(_))));
}
#[tokio::test]
async fn remove_missing_variable_returns_key_does_not_exist() {
let db = Database::new(&tempdir().unwrap()).unwrap();
let result = db
.delete_user_variable("room", "username", "myvariable")
.await;
assert!(result.is_err());
assert!(matches!(result, Err(DataError::KeyDoesNotExist(_))));
}
} }

42
src/db/errors.rs Normal file
View File

@ -0,0 +1,42 @@
use sled::transaction::{TransactionError, UnabortableTransactionError};
use thiserror::Error;
//TODO better combining of key and value in certain errors (namely
//I32SchemaViolation).
#[derive(Error, Debug)]
pub enum DataError {
#[error("value does not exist for key: {0}")]
KeyDoesNotExist(String),
#[error("expected i32, but i32 schema was violated")]
I32SchemaViolation,
#[error("expected string, but utf8 schema was violated: {0}")]
Utf8chemaViolation(#[from] std::str::Utf8Error),
#[error("internal database error: {0}")]
InternalError(#[from] sled::Error),
#[error("transaction error: {0}")]
TransactionError(#[from] sled::transaction::TransactionError),
#[error("unabortable transaction error: {0}")]
UnabortableTransactionError(#[from] UnabortableTransactionError),
}
/// This From implementation is necessary to deal with the recursive
/// error type in the error enum. We defined a transaction error, but
/// the only place we use it is when converting from
/// sled::transaction::TransactionError<DataError>. This converter
/// extracts the inner data error from transaction aborted errors, and
/// forwards anything else onward as-is, but wrapped in DataError.
impl From<TransactionError<DataError>> for DataError {
fn from(error: TransactionError<DataError>) -> Self {
match error {
TransactionError::Abort(data_err) => data_err,
TransactionError::Storage(storage_err) => {
DataError::TransactionError(TransactionError::Storage(storage_err))
}
}
}
}

22
src/db/schema.rs Normal file
View File

@ -0,0 +1,22 @@
use crate::db::errors::DataError;
use byteorder::LittleEndian;
use zerocopy::byteorder::I32;
use zerocopy::LayoutVerified;
/// User variables are stored as little-endian 32-bit integers in the
/// database. This type alias makes the database code more pleasant to
/// read.
type LittleEndianI32Layout<'a> = LayoutVerified<&'a [u8], I32<LittleEndian>>;
/// Convert bytes to an i32 with zero-copy deserialization. An error
/// is returned if the bytes do not represent an i32.
pub(super) fn convert_i32(raw_value: &[u8]) -> Result<i32, DataError> {
let layout = LittleEndianI32Layout::new_unaligned(raw_value.as_ref());
if let Some(layout) = layout {
let value: I32<LittleEndian> = *layout;
Ok(value.get())
} else {
Err(DataError::I32SchemaViolation)
}
}

354
src/db/variables.rs Normal file
View File

@ -0,0 +1,354 @@
use crate::db::errors::DataError;
use crate::db::schema::convert_i32;
use byteorder::LittleEndian;
use sled::transaction::abort;
use sled::transaction::TransactionalTree;
use sled::Tree;
use std::collections::HashMap;
use std::str;
use zerocopy::byteorder::I32;
use zerocopy::AsBytes;
const METADATA_KEY: &'static str = "metadata";
const VARIABLE_COUNT_KEY: &'static str = "variable_count";
#[derive(Clone)]
pub struct Variables(pub(crate) Tree);
fn to_key(room_id: &str, username: &str, variable_name: &str) -> Vec<u8> {
let mut key = vec![];
key.extend_from_slice(room_id.as_bytes());
key.extend_from_slice(username.as_bytes());
key.extend_from_slice(variable_name.as_bytes());
key
}
fn metadata_key(room_id: &str, username: &str, metadata_key: &str) -> Vec<u8> {
let mut key = vec![];
key.extend_from_slice(room_id.as_bytes());
key.extend_from_slice(METADATA_KEY.as_bytes());
key.extend_from_slice(username.as_bytes());
key.extend_from_slice(metadata_key.as_bytes());
key
}
fn room_variable_count_key(room_id: &str, username: &str) -> Vec<u8> {
metadata_key(room_id, username, VARIABLE_COUNT_KEY)
}
fn to_prefix(room_id: &str, username: &str) -> Vec<u8> {
let mut prefix = vec![];
prefix.extend_from_slice(room_id.as_bytes());
prefix.extend_from_slice(username.as_bytes());
prefix
}
/// Use a transaction to atomically alter the count of variables in
/// the database by the given amount. Count cannot go below 0.
fn alter_room_variable_count(
variables: &TransactionalTree,
room_id: &str,
username: &str,
amount: i32,
) -> Result<i32, DataError> {
let key = room_variable_count_key(room_id, username);
let mut new_count = match variables.get(&key)? {
Some(bytes) => convert_i32(&bytes)? + amount,
None => amount,
};
if new_count < 0 {
new_count = 0;
}
let db_value: I32<LittleEndian> = I32::new(new_count);
variables.insert(key, db_value.as_bytes())?;
Ok(new_count)
}
impl Variables {
pub async fn get_user_variables(
&self,
room_id: &str,
username: &str,
) -> Result<HashMap<String, i32>, DataError> {
let prefix = to_prefix(&room_id, &username);
let prefix_len: usize = prefix.len();
let variables: Result<Vec<_>, DataError> = self
.0
.scan_prefix(prefix)
.map(|entry| match entry {
Ok((key, raw_value)) => {
//Strips room and username from key, leaving behind name.
let variable_name = str::from_utf8(&key[prefix_len..])?;
Ok((variable_name.to_owned(), convert_i32(&raw_value)?))
}
Err(e) => Err(e.into()),
})
.collect();
//Convert I32 to hash map. Can we do this in the first mapping
//step instead? For some reason this is faster.
variables.map(|entries| entries.into_iter().collect())
}
pub async fn get_variable_count(
&self,
room_id: &str,
username: &str,
) -> Result<i32, DataError> {
let key = room_variable_count_key(room_id, username);
if let Some(raw_value) = self.0.get(&key)? {
convert_i32(&raw_value)
} else {
Ok(0)
}
}
pub async fn get_user_variable(
&self,
room_id: &str,
username: &str,
variable_name: &str,
) -> Result<i32, DataError> {
let key = to_key(room_id, username, variable_name);
if let Some(raw_value) = self.0.get(&key)? {
convert_i32(&raw_value)
} else {
Err(DataError::KeyDoesNotExist(String::from_utf8(key).unwrap()))
}
}
pub async fn set_user_variable(
&self,
room_id: &str,
username: &str,
variable_name: &str,
value: i32,
) -> Result<(), DataError> {
self.0
.transaction(|tx| {
let key = to_key(room_id, username, variable_name);
let db_value: I32<LittleEndian> = I32::new(value);
let old_value = tx.insert(key, db_value.as_bytes())?;
//Only increment variable count on new keys.
if let None = old_value {
match alter_room_variable_count(&tx, room_id, username, 1) {
Err(e) => abort(e),
_ => Ok(()),
}
} else {
Ok(())
}
})
.map_err(|e| e.into())
}
pub async fn delete_user_variable(
&self,
room_id: &str,
username: &str,
variable_name: &str,
) -> Result<(), DataError> {
self.0
.transaction(|tx| {
let key = to_key(room_id, username, variable_name);
if let Some(_) = tx.remove(key.clone())? {
match alter_room_variable_count(&tx, room_id, username, -1) {
Err(e) => abort(e),
_ => Ok(()),
}
} else {
abort(DataError::KeyDoesNotExist(String::from_utf8(key).unwrap()))
}
})
.map_err(|e| e.into())
}
}
#[cfg(test)]
mod tests {
use super::*;
use tempfile::tempdir;
fn create_test_instance() -> Variables {
Variables(
sled::open(&tempdir().unwrap())
.unwrap()
.open_tree("variables")
.unwrap(),
)
}
//Room Variable count tests
#[tokio::test]
async fn alter_room_variable_count_test() {
let variables = create_test_instance();
let alter_count = |amount: i32| {
variables
.0
.transaction(|tx| {
match alter_room_variable_count(&tx, "room", "username", amount) {
Err(e) => abort(e),
_ => Ok(()),
}
})
.expect("got transaction failure");
};
async fn get_count(variables: &Variables) -> i32 {
variables
.get_variable_count("room", "username")
.await
.expect("could not get variable count")
}
//addition
alter_count(5);
assert_eq!(5, get_count(&variables).await);
//subtraction
alter_count(-3);
assert_eq!(2, get_count(&variables).await);
}
#[tokio::test]
async fn alter_room_variable_count_cannot_go_below_0_test() {
let variables = create_test_instance();
variables
.0
.transaction(
|tx| match alter_room_variable_count(&tx, "room", "username", -1000) {
Err(e) => abort(e),
_ => Ok(()),
},
)
.expect("got transaction failure");
let count = variables
.get_variable_count("room", "username")
.await
.expect("could not get variable count");
assert_eq!(0, count);
}
#[tokio::test]
async fn empty_db_reports_0_room_variable_count_test() {
let variables = create_test_instance();
let count = variables
.get_variable_count("room", "username")
.await
.expect("could not get variable count");
assert_eq!(0, count);
}
#[tokio::test]
async fn set_user_variable_increments_count() {
let variables = create_test_instance();
variables
.set_user_variable("room", "username", "myvariable", 5)
.await
.expect("could not insert variable");
let count = variables
.get_variable_count("room", "username")
.await
.expect("could not get variable count");
assert_eq!(1, count);
}
#[tokio::test]
async fn update_user_variable_does_not_increment_count() {
let variables = create_test_instance();
variables
.set_user_variable("room", "username", "myvariable", 5)
.await
.expect("could not insert variable");
variables
.set_user_variable("room", "username", "myvariable", 10)
.await
.expect("could not update variable");
let count = variables
.get_variable_count("room", "username")
.await
.expect("could not get variable count");
assert_eq!(1, count);
}
// Set/get/delete variable tests
#[tokio::test]
async fn set_and_get_variable_test() {
let variables = create_test_instance();
variables
.set_user_variable("room", "username", "myvariable", 5)
.await
.expect("could not insert variable");
let value = variables
.get_user_variable("room", "username", "myvariable")
.await
.expect("could not get value");
assert_eq!(5, value);
}
#[tokio::test]
async fn delete_variable_test() {
let variables = create_test_instance();
variables
.set_user_variable("room", "username", "myvariable", 5)
.await
.expect("could not insert variable");
variables
.delete_user_variable("room", "username", "myvariable")
.await
.expect("could not delete value");
let result = variables
.get_user_variable("room", "username", "myvariable")
.await;
assert!(result.is_err());
assert!(matches!(result, Err(DataError::KeyDoesNotExist(_))));
}
#[tokio::test]
async fn get_missing_variable_returns_key_does_not_exist() {
let variables = create_test_instance();
let result = variables
.get_user_variable("room", "username", "myvariable")
.await;
assert!(result.is_err());
assert!(matches!(result, Err(DataError::KeyDoesNotExist(_))));
}
#[tokio::test]
async fn remove_missing_variable_returns_key_does_not_exist() {
let variables = create_test_instance();
let result = variables
.delete_user_variable("room", "username", "myvariable")
.await;
assert!(result.is_err());
assert!(matches!(result, Err(DataError::KeyDoesNotExist(_))));
}
}

View File

@ -1,7 +1,7 @@
use crate::cofd::dice::DiceRollingError; use crate::cofd::dice::DiceRollingError;
use crate::commands::CommandError; use crate::commands::CommandError;
use crate::config::ConfigError; use crate::config::ConfigError;
use crate::db::DataError; use crate::db::errors::DataError;
use thiserror::Error; use thiserror::Error;
#[derive(Error, Debug)] #[derive(Error, Debug)]