mirror of
https://github.com/mauriceboe/TREK.git
synced 2026-06-19 13:21:46 +00:00
test(systemNotices): exclude v3 upgrade notices from login_count-only tests
Tests that expect an empty notice list were using first_seen_version='0.0.0'
(DB default), which matches the existingUserBeforeVersion('3.0.0') condition
now that the app is at 3.0.1. Set first_seen_version='3.0.0' so only the
firstLogin condition controls visibility in these tests.
This commit is contained in:
@@ -84,8 +84,9 @@ describe('GET /api/system-notices/active', () => {
|
|||||||
|
|
||||||
it('returns empty array for non-first-login user with no applicable notices', async () => {
|
it('returns empty array for non-first-login user with no applicable notices', async () => {
|
||||||
const { user } = createUser(testDb);
|
const { user } = createUser(testDb);
|
||||||
// login_count > 1 means firstLogin condition does not match for any notice
|
// login_count > 1 means firstLogin condition does not match for any notice;
|
||||||
testDb.prepare('UPDATE users SET login_count = 5 WHERE id = ?').run(user.id);
|
// first_seen_version >= 3.0.0 means existingUserBeforeVersion('3.0.0') also does not match
|
||||||
|
testDb.prepare('UPDATE users SET login_count = 5, first_seen_version = ? WHERE id = ?').run('3.0.0', user.id);
|
||||||
const res = await request(app)
|
const res = await request(app)
|
||||||
.get('/api/system-notices/active')
|
.get('/api/system-notices/active')
|
||||||
.set('Cookie', authCookie(user.id));
|
.set('Cookie', authCookie(user.id));
|
||||||
@@ -122,7 +123,7 @@ describe('GET /api/system-notices/active', () => {
|
|||||||
SYSTEM_NOTICES.push(TEST_NOTICE);
|
SYSTEM_NOTICES.push(TEST_NOTICE);
|
||||||
try {
|
try {
|
||||||
const { user } = createUser(testDb);
|
const { user } = createUser(testDb);
|
||||||
testDb.prepare('UPDATE users SET login_count = 5 WHERE id = ?').run(user.id);
|
testDb.prepare('UPDATE users SET login_count = 5, first_seen_version = ? WHERE id = ?').run('3.0.0', user.id);
|
||||||
|
|
||||||
const res = await request(app)
|
const res = await request(app)
|
||||||
.get('/api/system-notices/active')
|
.get('/api/system-notices/active')
|
||||||
|
|||||||
Reference in New Issue
Block a user