Fix RFV period buyers with historical scoring
All checks were successful
Build and Deploy / build-and-deploy (push) Successful in 1m34s
All checks were successful
Build and Deploy / build-and-deploy (push) Successful in 1m34s
This commit is contained in:
@@ -7,6 +7,7 @@ const PRODUCT_NAME_SQL = `
|
||||
ELSE NULLIF(TRIM(regexp_replace(split_part(COALESCE(produto_descricao, 'Unknown'), ' TAMANHO', 1), '${SIZE_SUFFIX_SQL_PATTERN}', '', 'i')), '')
|
||||
END
|
||||
`;
|
||||
const CUSTOMER_KEY_SQL = "COALESCE(NULLIF(cliente_fone, ''), 'name:' || COALESCE(NULLIF(cliente_nome, ''), 'Cliente Desconhecido'))";
|
||||
|
||||
const normalizeDateParam = (value) => {
|
||||
if (!value) return null;
|
||||
@@ -235,7 +236,8 @@ const getClientAnalytics = async (range = {}) => {
|
||||
const { params, whereClause } = buildDateFilter(range);
|
||||
const result = await pool.query(`
|
||||
SELECT
|
||||
COALESCE(NULLIF(cliente_nome, ''), 'Cliente Desconhecido') as name,
|
||||
${CUSTOMER_KEY_SQL} as customer_key,
|
||||
MAX(COALESCE(NULLIF(cliente_nome, ''), 'Cliente Desconhecido')) as name,
|
||||
MAX(NULLIF(cliente_fone, '')) as phone,
|
||||
COALESCE(SUM(quantidade), 0) as quantity_purchased,
|
||||
COALESCE(SUM(quantidade * valor_unitario), 0) as total_spent,
|
||||
@@ -243,11 +245,12 @@ const getClientAnalytics = async (range = {}) => {
|
||||
MAX(data_pedido_date) as last_purchase_date
|
||||
FROM orders
|
||||
${whereClause}
|
||||
GROUP BY COALESCE(NULLIF(cliente_nome, ''), 'Cliente Desconhecido')
|
||||
GROUP BY customer_key
|
||||
ORDER BY total_spent DESC;
|
||||
`, params);
|
||||
|
||||
return result.rows.map(row => ({
|
||||
customerKey: row.customer_key,
|
||||
name: row.name,
|
||||
phone: row.phone || '',
|
||||
quantityPurchased: toNumber(row.quantity_purchased),
|
||||
@@ -259,32 +262,30 @@ const getClientAnalytics = async (range = {}) => {
|
||||
|
||||
const getRfmAnalytics = async (range = {}) => {
|
||||
const { params, whereClause } = buildDateFilter(range);
|
||||
const normalizedStart = normalizeDateParam(range.start);
|
||||
const normalizedEnd = normalizeDateParam(range.end);
|
||||
const tagReference = getPreviousDate(normalizedStart) || normalizedEnd;
|
||||
const recencyReferenceDate = tagReference ? '$1::date' : 'CURRENT_DATE';
|
||||
const historyParams = tagReference ? [tagReference] : [];
|
||||
const recencyReferenceDate = normalizedEnd ? '$1::date' : 'CURRENT_DATE';
|
||||
const historyParams = normalizedEnd ? [normalizedEnd] : [];
|
||||
|
||||
const [periodResult, historyResult] = await Promise.all([
|
||||
pool.query(`
|
||||
SELECT
|
||||
MAX(cliente_nome) as name,
|
||||
cliente_fone as phone,
|
||||
${CUSTOMER_KEY_SQL} as customer_key,
|
||||
MAX(COALESCE(NULLIF(cliente_nome, ''), 'Cliente Desconhecido')) as name,
|
||||
MAX(NULLIF(cliente_fone, '')) as phone,
|
||||
COALESCE(SUM(quantidade * valor_unitario), 0) as monetary,
|
||||
COUNT(DISTINCT COALESCE(NULLIF(pedido_id, ''), data_pedido || '_' || valor_pedido::text))::int as frequency,
|
||||
COALESCE(SUM(quantidade), 0) as quantity_purchased,
|
||||
MAX(data_pedido_date) as last_purchase_date
|
||||
FROM orders
|
||||
${whereClause}
|
||||
AND cliente_fone IS NOT NULL
|
||||
AND cliente_fone != ''
|
||||
GROUP BY cliente_fone
|
||||
GROUP BY customer_key
|
||||
ORDER BY monetary DESC;
|
||||
`, params),
|
||||
pool.query(`
|
||||
SELECT
|
||||
MAX(cliente_nome) as name,
|
||||
cliente_fone as phone,
|
||||
${CUSTOMER_KEY_SQL} as customer_key,
|
||||
MAX(COALESCE(NULLIF(cliente_nome, ''), 'Cliente Desconhecido')) as name,
|
||||
MAX(NULLIF(cliente_fone, '')) as phone,
|
||||
COALESCE(SUM(quantidade * valor_unitario), 0) as monetary,
|
||||
COUNT(DISTINCT COALESCE(NULLIF(pedido_id, ''), data_pedido || '_' || valor_pedido::text))::int as frequency,
|
||||
COALESCE(SUM(quantidade), 0) as quantity_purchased,
|
||||
@@ -293,30 +294,30 @@ const getRfmAnalytics = async (range = {}) => {
|
||||
FROM orders
|
||||
WHERE data_pedido_date IS NOT NULL
|
||||
AND data_pedido_date <= ${recencyReferenceDate}
|
||||
AND cliente_fone IS NOT NULL
|
||||
AND cliente_fone != ''
|
||||
GROUP BY cliente_fone;
|
||||
GROUP BY customer_key;
|
||||
`, historyParams)
|
||||
]);
|
||||
|
||||
const historyClients = buildRfmClients(historyResult.rows.map(row => ({
|
||||
customerKey: row.customer_key,
|
||||
name: row.name,
|
||||
phone: row.phone,
|
||||
phone: row.phone || '',
|
||||
monetary: toNumber(row.monetary),
|
||||
frequency: toNumber(row.frequency),
|
||||
quantityPurchased: toNumber(row.quantity_purchased),
|
||||
lastPurchaseDate: row.last_purchase_date,
|
||||
recencyDays: toNumber(row.recency_days)
|
||||
})));
|
||||
const tagsByPhone = new Map(historyClients.map(client => [client.phone, client]));
|
||||
const tagsByCustomerKey = new Map(historyClients.map(client => [client.customerKey, client]));
|
||||
|
||||
const clients = periodResult.rows.map(row => {
|
||||
const taggedClient = tagsByPhone.get(row.phone);
|
||||
const taggedClient = tagsByCustomerKey.get(row.customer_key);
|
||||
if (!taggedClient) {
|
||||
const newCustomerSegment = getRfmSegment(3, 1);
|
||||
return {
|
||||
customerKey: row.customer_key,
|
||||
name: row.name,
|
||||
phone: row.phone,
|
||||
phone: row.phone || '',
|
||||
monetary: toNumber(row.monetary),
|
||||
frequency: toNumber(row.frequency),
|
||||
quantityPurchased: toNumber(row.quantity_purchased),
|
||||
@@ -334,8 +335,9 @@ const getRfmAnalytics = async (range = {}) => {
|
||||
|
||||
return {
|
||||
...taggedClient,
|
||||
customerKey: row.customer_key,
|
||||
name: row.name,
|
||||
phone: row.phone,
|
||||
phone: row.phone || '',
|
||||
monetary: toNumber(row.monetary),
|
||||
frequency: toNumber(row.frequency),
|
||||
quantityPurchased: toNumber(row.quantity_purchased),
|
||||
|
||||
@@ -191,28 +191,30 @@ test('buildRfmClients scores segments from RFM history when period totals are sm
|
||||
assert.equal(yesterdayBuyer.monetaryScore, 3);
|
||||
});
|
||||
|
||||
test('getRfmAnalytics groups period buyers by their RFM tag before the selected period', async () => {
|
||||
test('getRfmAnalytics classifies period buyers by history through the selected range end', async () => {
|
||||
const originalQuery = pool.query;
|
||||
const calls = [];
|
||||
|
||||
pool.query = async (sql, params) => {
|
||||
calls.push({ sql, params });
|
||||
const isHistoryQuery = sql.includes('recency_days');
|
||||
const endDate = params[0];
|
||||
const referenceDate = params[0];
|
||||
|
||||
if (isHistoryQuery) {
|
||||
return {
|
||||
rows: [
|
||||
{
|
||||
customer_key: '1',
|
||||
name: 'Cliente Ontem',
|
||||
phone: '1',
|
||||
monetary: 5000,
|
||||
frequency: 20,
|
||||
quantity_purchased: 20,
|
||||
last_purchase_date: '2026-06-14',
|
||||
recency_days: endDate === '2026-06-13' ? 0 : 1
|
||||
recency_days: referenceDate === '2026-06-14' ? 0 : 1
|
||||
},
|
||||
{
|
||||
customer_key: '2',
|
||||
name: 'Cliente Antigo',
|
||||
phone: '2',
|
||||
monetary: 50,
|
||||
@@ -222,6 +224,7 @@ test('getRfmAnalytics groups period buyers by their RFM tag before the selected
|
||||
recency_days: 164
|
||||
},
|
||||
{
|
||||
customer_key: '3',
|
||||
name: 'Cliente Medio',
|
||||
phone: '3',
|
||||
monetary: 100,
|
||||
@@ -229,6 +232,16 @@ test('getRfmAnalytics groups period buyers by their RFM tag before the selected
|
||||
quantity_purchased: 2,
|
||||
last_purchase_date: '2026-03-01',
|
||||
recency_days: 105
|
||||
},
|
||||
{
|
||||
customer_key: 'name:Cliente Sem Fone',
|
||||
name: 'Cliente Sem Fone',
|
||||
phone: null,
|
||||
monetary: 1000,
|
||||
frequency: 10,
|
||||
quantity_purchased: 10,
|
||||
last_purchase_date: '2026-06-14',
|
||||
recency_days: 1
|
||||
}
|
||||
]
|
||||
};
|
||||
@@ -237,6 +250,7 @@ test('getRfmAnalytics groups period buyers by their RFM tag before the selected
|
||||
return {
|
||||
rows: [
|
||||
{
|
||||
customer_key: '1',
|
||||
name: 'Cliente Ontem',
|
||||
phone: '1',
|
||||
monetary: 100,
|
||||
@@ -245,12 +259,22 @@ test('getRfmAnalytics groups period buyers by their RFM tag before the selected
|
||||
last_purchase_date: '2026-06-14'
|
||||
},
|
||||
{
|
||||
customer_key: '4',
|
||||
name: 'Cliente Novo no Periodo',
|
||||
phone: '4',
|
||||
monetary: 25,
|
||||
frequency: 1,
|
||||
quantity_purchased: 1,
|
||||
last_purchase_date: '2026-06-14'
|
||||
},
|
||||
{
|
||||
customer_key: 'name:Cliente Sem Fone',
|
||||
name: 'Cliente Sem Fone',
|
||||
phone: null,
|
||||
monetary: 30,
|
||||
frequency: 1,
|
||||
quantity_purchased: 1,
|
||||
last_purchase_date: '2026-06-14'
|
||||
}
|
||||
]
|
||||
};
|
||||
@@ -261,18 +285,26 @@ test('getRfmAnalytics groups period buyers by their RFM tag before the selected
|
||||
const yesterday = await getRfmAnalytics({ start: '2026-06-14', end: '2026-06-14' });
|
||||
|
||||
assert.deepEqual(calls[0].params, ['2026-06-09', '2026-06-15']);
|
||||
assert.doesNotMatch(calls[0].sql, /cliente_fone IS NOT NULL/);
|
||||
assert.match(calls[1].sql, /\(\$1::date - MAX\(data_pedido_date\)\)::int/);
|
||||
assert.match(calls[1].sql, /data_pedido_date <= \$1::date/);
|
||||
assert.deepEqual(calls[1].params, ['2026-06-08']);
|
||||
assert.doesNotMatch(calls[1].sql, /cliente_fone IS NOT NULL/);
|
||||
assert.deepEqual(calls[1].params, ['2026-06-15']);
|
||||
assert.deepEqual(calls[2].params, ['2026-06-14', '2026-06-14']);
|
||||
assert.deepEqual(calls[3].params, ['2026-06-13']);
|
||||
assert.deepEqual(calls[3].params, ['2026-06-14']);
|
||||
assert.equal(sevenDays.clients[0].segmentKey, 'champions');
|
||||
assert.equal(yesterday.clients[0].segmentKey, 'champions');
|
||||
assert.equal(yesterday.clients[0].frequency, 1);
|
||||
assert.equal(yesterday.clients[0].monetary, 100);
|
||||
assert.equal(yesterday.clients.length, 2);
|
||||
assert.equal(yesterday.clients.length, 3);
|
||||
assert.ok(!yesterday.clients.some(client => client.phone === '2'));
|
||||
assert.ok(yesterday.clients.some(client => client.phone === '4' && client.segmentKey === 'new_customers'));
|
||||
assert.ok(yesterday.clients.some(client => (
|
||||
client.customerKey === 'name:Cliente Sem Fone' &&
|
||||
client.phone === '' &&
|
||||
client.segmentKey === 'champions' &&
|
||||
client.monetary === 30
|
||||
)));
|
||||
} finally {
|
||||
pool.query = originalQuery;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user