- Idempotent hourly rollup (run_rollup, INSERT ... ON CONFLICT DO UPDATE into
analytics_rollups_hourly) + Tokio background task (startup then hourly)
- GET /v1/cards/{id}/analytics/geo — country-grouped visit counts
- GeoResolver trait: real MaxMind under the `geoip` feature, no-op default;
raw IP resolved then discarded (only hash + coarse country/city persisted)
- analytics_events now store country/city; rollups read query for the dashboard
2 new tests (geo grouping, rollup aggregation + idempotency); 75 backend tests.
fmt + clippy clean; geoip feature compiles.
Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
201 lines
5.8 KiB
Rust
201 lines
5.8 KiB
Rust
//! Integration tests for analytics ingest + summary (PRD §13.5, §18).
|
|
|
|
mod common;
|
|
|
|
use axum::http::StatusCode;
|
|
use serde_json::json;
|
|
|
|
use common::{unique_handle, TestApp};
|
|
|
|
async fn create_and_publish(app: &TestApp, token: &str) -> (String, String) {
|
|
let handle = unique_handle();
|
|
let def = json!({
|
|
"face": { "layers": [], "background": { "type": "solid", "value": "#101014" } },
|
|
"back": { "layers": [] }
|
|
});
|
|
let (_, created) = app
|
|
.request(
|
|
"POST",
|
|
"/v1/cards",
|
|
Some(token),
|
|
Some(json!({"handle": handle, "definition": def})),
|
|
)
|
|
.await;
|
|
let id = created["id"].as_str().unwrap().to_string();
|
|
app.request(
|
|
"POST",
|
|
&format!("/v1/cards/{id}/publish"),
|
|
Some(token),
|
|
None,
|
|
)
|
|
.await;
|
|
(id, handle)
|
|
}
|
|
|
|
#[tokio::test]
|
|
async fn profile_visit_recorded_on_public_lookup() {
|
|
let app = require_app!();
|
|
let token = app.register_and_token().await;
|
|
let (id, handle) = create_and_publish(&app, &token).await;
|
|
|
|
// Two public lookups => two profile visits.
|
|
app.request("GET", &format!("/v1/cards/handle/{handle}"), None, None)
|
|
.await;
|
|
app.request("GET", &format!("/v1/cards/handle/{handle}"), None, None)
|
|
.await;
|
|
|
|
let (status, body) = app
|
|
.request(
|
|
"GET",
|
|
&format!("/v1/cards/{id}/analytics"),
|
|
Some(&token),
|
|
None,
|
|
)
|
|
.await;
|
|
assert_eq!(status, StatusCode::OK);
|
|
assert_eq!(body["totalVisits"], 2);
|
|
assert_eq!(body["visits24h"], 2);
|
|
}
|
|
|
|
#[tokio::test]
|
|
async fn client_event_ingest_increments_summary() {
|
|
let app = require_app!();
|
|
let token = app.register_and_token().await;
|
|
let (id, _) = create_and_publish(&app, &token).await;
|
|
|
|
let (status, _) = app
|
|
.request(
|
|
"POST",
|
|
"/v1/analytics/event",
|
|
None,
|
|
Some(json!({"card_id": id, "event_type": "contact_save"})),
|
|
)
|
|
.await;
|
|
assert_eq!(status, StatusCode::OK);
|
|
|
|
let (_, body) = app
|
|
.request(
|
|
"GET",
|
|
&format!("/v1/cards/{id}/analytics"),
|
|
Some(&token),
|
|
None,
|
|
)
|
|
.await;
|
|
assert_eq!(body["contactSaves"], 1);
|
|
}
|
|
|
|
#[tokio::test]
|
|
async fn ingest_rejects_server_only_event_type() {
|
|
let app = require_app!();
|
|
let token = app.register_and_token().await;
|
|
let (id, _) = create_and_publish(&app, &token).await;
|
|
|
|
// profile_visit is server-originated; clients can't post it.
|
|
let (status, body) = app
|
|
.request(
|
|
"POST",
|
|
"/v1/analytics/event",
|
|
None,
|
|
Some(json!({"card_id": id, "event_type": "profile_visit"})),
|
|
)
|
|
.await;
|
|
assert_eq!(status, StatusCode::BAD_REQUEST);
|
|
assert_eq!(body["code"], "validation");
|
|
}
|
|
|
|
#[tokio::test]
|
|
async fn geo_breakdown_groups_by_country() {
|
|
let app = require_app!();
|
|
let token = app.register_and_token().await;
|
|
let (id, _) = create_and_publish(&app, &token).await;
|
|
|
|
// Two client events with a forwarded IP → FakeGeo resolves both to US.
|
|
for _ in 0..2 {
|
|
app.request_with_headers(
|
|
"POST",
|
|
"/v1/analytics/event",
|
|
None,
|
|
Some(json!({ "card_id": id, "event_type": "contact_save" })),
|
|
&[("x-forwarded-for", "203.0.113.7")],
|
|
)
|
|
.await;
|
|
}
|
|
|
|
let (status, body) = app
|
|
.request(
|
|
"GET",
|
|
&format!("/v1/cards/{id}/analytics/geo"),
|
|
Some(&token),
|
|
None,
|
|
)
|
|
.await;
|
|
assert_eq!(status, StatusCode::OK);
|
|
let rows = body.as_array().unwrap();
|
|
assert_eq!(rows.len(), 1);
|
|
assert_eq!(rows[0]["country"], "US");
|
|
assert_eq!(rows[0]["visits"], 2);
|
|
}
|
|
|
|
#[tokio::test]
|
|
async fn hourly_rollup_aggregates_and_is_idempotent() {
|
|
use cardclaws_db::queries::analytics;
|
|
use uuid::Uuid;
|
|
|
|
let app = require_app!();
|
|
let token = app.register_and_token().await;
|
|
let (id, handle) = create_and_publish(&app, &token).await;
|
|
let card_id = Uuid::parse_str(&id).unwrap();
|
|
|
|
// Generate two profile visits and one qr_scan (via a share resolve).
|
|
app.request("GET", &format!("/v1/cards/handle/{handle}"), None, None)
|
|
.await;
|
|
app.request("GET", &format!("/v1/cards/handle/{handle}"), None, None)
|
|
.await;
|
|
let (_, share) = app
|
|
.request(
|
|
"POST",
|
|
&format!("/v1/cards/{id}/share"),
|
|
Some(&token),
|
|
Some(json!({"modality": "qr"})),
|
|
)
|
|
.await;
|
|
app.get_redirect(&format!("/v1/s/{}", share["token"].as_str().unwrap()))
|
|
.await;
|
|
|
|
let affected = analytics::run_rollup(&app.db).await.unwrap();
|
|
assert!(affected >= 1);
|
|
|
|
let rollups = analytics::rollups(&app.db, card_id).await.unwrap();
|
|
let totals = rollups
|
|
.iter()
|
|
.fold((0, 0), |(v, q), r| (v + r.visits, q + r.qr_scans));
|
|
assert_eq!(totals.0, 2, "two profile visits rolled up");
|
|
assert_eq!(totals.1, 1, "one qr scan rolled up");
|
|
|
|
// Re-running must not double-count (ON CONFLICT DO UPDATE).
|
|
analytics::run_rollup(&app.db).await.unwrap();
|
|
let rollups2 = analytics::rollups(&app.db, card_id).await.unwrap();
|
|
let totals2 = rollups2
|
|
.iter()
|
|
.fold((0, 0), |(v, q), r| (v + r.visits, q + r.qr_scans));
|
|
assert_eq!(totals, totals2, "rollup is idempotent");
|
|
}
|
|
|
|
#[tokio::test]
|
|
async fn analytics_summary_requires_ownership() {
|
|
let app = require_app!();
|
|
let owner = app.register_and_token().await;
|
|
let (id, _) = create_and_publish(&app, &owner).await;
|
|
|
|
let intruder = app.register_and_token().await;
|
|
let (status, _) = app
|
|
.request(
|
|
"GET",
|
|
&format!("/v1/cards/{id}/analytics"),
|
|
Some(&intruder),
|
|
None,
|
|
)
|
|
.await;
|
|
assert_eq!(status, StatusCode::NOT_FOUND);
|
|
}
|