Fix tests and add workflow to run tests and format code

.github/workflows/test.yml

@@ -0,0 +1,37 @@
+name: Tests and Formatting
+
+on:
+  workflow_dispatch:
+  push:
+    branches: [ main ]
+  pull_request:
+    branches: [ main ]
+jobs:
+  everything:
+    name: Rust/JS Checks/Formatting
+    runs-on: ubuntu-latest
+    steps:
+      - name: Check out
+        uses: actions/checkout@v3
+      - uses: Swatinem/rust-cache@v2
+      - name: Run cargo fmt
+        id: fmt
+        run: cargo fmt --all -- --check
+        continue-on-error: true
+      - name: Run cargo clippy
+        id: clippy
+        continue-on-error: true
+        run: cargo clippy --all -- -D warnings
+      - name: Run cargo test
+        id: test
+        continue-on-error: true
+        run: cargo test
+      - name: Check if code is properly formatted
+        if: steps.fmt.outcome != 'success'
+        run: exit 1
+      - name: Check if clippy is happy
+        if: steps.clippy.outcome != 'success'
+        run: exit 1
+      - name: Check if test succeeded
+        if: steps.test.outcome != 'success'
+        run: exit 1

examples/003-migration/src/lib.rs

@@ -16,7 +16,7 @@ fn create_account(
         return Err(username.error("username 'admin' is not allowed").into());
     }
 
-    let _res = conn.transaction::<_, ft_sdk::Error, _>(|c| {
+    conn.transaction::<_, ft_sdk::Error, _>(|c| {
         // do a select query to see if username is already taken
         if diesel::select(diesel::dsl::exists(
             account_user::table.filter(account_user::username.eq(&username.0)),

ft-sdk/Cargo.toml

@@ -36,3 +36,4 @@ uuid.workspace = true
 
 [dev-dependencies]
 pretty_assertions = "1"
+cookie.workspace = true

ft-sdk/src/data.rs

@@ -113,13 +113,13 @@ pub fn binary<S: AsRef<str>>(content: bytes::Bytes, content_type: S) -> Result {
 /// a 200-OK response, with an HTML meta-refresh tag to redirect the browser.
 ///
 /// ```rust
-///  let cookie = cookie::Cookie::build((ft_sdk::auth::SESSION_KEY, sid))
-///         .domain(host.without_port())
-///         .path("/")
+///  let cookie = cookie::Cookie::build((ft_sdk::auth::SESSION_KEY, "some-uniq-key"))
+///         .domain("127.0.0.1")
+///         .path("")
 ///         .max_age(cookie::time::Duration::seconds(34560000))
 ///         .same_site(cookie::SameSite::Strict)
 ///         .build();
-/// ft_sdk::data::browser_redirect_with_cookie("/", cookie)
+/// ft_sdk::data::browser_redirect_with_cookie("/", http::HeaderValue::from_str(cookie.to_string().as_str()).unwrap());
 /// ```
 pub fn browser_redirect_with_cookie<S: AsRef<str>>(url: S, c: http::HeaderValue) -> Result {
     Ok(ft_sdk::chr::CHR::new(Output::Redirect(

ft-sys/src/diesel_sqlite/mod.rs

@@ -8,5 +8,5 @@ mod types;
 
 pub use backend::Sqlite;
 pub use connection::SqliteConnection;
-pub use sqlite_value::{Cursor, SqliteValue};
 pub(crate) use no_instrumentation::NoInstrumentation;
+pub use sqlite_value::{Cursor, SqliteValue};