1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: skip tests failing when run under root

Browse Source 
PR-URL: https://github.com/nodejs/node/pull/58610
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Pietro Marchini <pietro.marchini94@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Stefan Stojanovic <stefan.stojanovic@janeasystems.com>
This commit is contained in:
Livia Medeiros 2025-06-10 20:57:27 +08:00 committed by GitHub
parent 58e1cba200
commit be2120f1cd
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
test/parallel/test-config-file.js
View File

@ -361,7 +361,10 @@ test('should override node.config.json when specificied', async () => {
strictEqual(result.code, 0); strictEqual(result.code, 0);
}); });
// Skip on windows because it doesn't support chmod changing read permissions // Skip on windows because it doesn't support chmod changing read permissions
test('should throw an error when the file is non readable', { skip: common.isWindows }, async () => { // Also skip if user is root because it would have read permissions anyway
test('should throw an error when the file is non readable', {
skip: common.isWindows || process.getuid() === 0,
}, async () => {
chmodSync(fixtures.path('rc/non-readable/node.config.json'), constants.O_RDONLY); chmodSync(fixtures.path('rc/non-readable/node.config.json'), constants.O_RDONLY);
const result = await spawnPromisified(process.execPath, [ const result = await spawnPromisified(process.execPath, [
'--no-warnings', '--no-warnings',

6
test/parallel/test-sqlite-backup.mjs
View File

@ -1,4 +1,4 @@
import { skipIfSQLiteMissing } from '../common/index.mjs'; import { isWindows, skipIfSQLiteMissing } from '../common/index.mjs';
import tmpdir from '../common/tmpdir.js'; import tmpdir from '../common/tmpdir.js';
import { join } from 'node:path'; import { join } from 'node:path';
import { describe, test } from 'node:test'; import { describe, test } from 'node:test';
@ -7,6 +7,8 @@ import { pathToFileURL } from 'node:url';
skipIfSQLiteMissing(); skipIfSQLiteMissing();
const { backup, DatabaseSync } = await import('node:sqlite'); const { backup, DatabaseSync } = await import('node:sqlite');
const isRoot = !isWindows && process.getuid() === 0;
let cnt = 0; let cnt = 0;
tmpdir.refresh(); tmpdir.refresh();
@ -252,7 +254,7 @@ test('throws if URL is not file: scheme', (t) => {
}); });
}); });
test('database backup fails when dest file is not writable', async (t) => { test('database backup fails when dest file is not writable', { skip: isRoot }, async (t) => {
const readonlyDestDb = nextDb(); const readonlyDestDb = nextDb();
writeFileSync(readonlyDestDb, '', { mode: 0o444 }); writeFileSync(readonlyDestDb, '', { mode: 0o444 });