github_mirrors/puter
1
0
Fork 0
mirror of https://github.com/HeyPuter/puter.git synced 2025-01-23 06:00:21 +08:00
Code Issues Packages Projects Releases Wiki Activity

Revert "Created test cases for issue #878 "

Browse Source
This commit is contained in:
Nariman Jelveh 2024-12-08 11:31:47 -08:00 committed by GitHub
parent 575b4bd34c
commit 40ea2a3f02
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
6 changed files with 313 additions and 33125 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

145
eslint.config.js
View File

@ -1,12 +1,139 @@
import js from "@eslint/js";
import globals from "globals"; import globals from "globals";
import pluginJs from "@eslint/js";
import tseslint from "typescript-eslint";
/** @type {import('eslint').Linter.Config[]} */
export default [ export default [
{files: ["**/*.{js,mjs,cjs,ts}"]}, js.configs.recommended,
{languageOptions: { globals: globals.browser }},
pluginJs.configs.recommended, {
...tseslint.configs.recommended, // Global ignores
]; ignores: [
"**/*.min.js",
"**/src/lib/**",
"**/dist/",
"src/backend/src/public/assets/**",
"incubator/**"
],
},
{
// Top-level and tools use Node
files: [
"tools/**/*.js",
],
languageOptions: {
globals: {
...globals.node,
}
}
},
{
// Back end
files: [
"src/backend/**/*.js",
"mods/**/*.js",
"dev-server.js",
"utils.js",
],
languageOptions: {
globals: {
...globals.node,
"kv": true,
"def": true,
"use": true,
"ll":true,
}
}
},
{
// Front end
files: [
"src/**/*.js",
],
ignores: [
"src/backend/**/*.js",
],
languageOptions: {
globals: {
...globals.browser,
...globals.commonjs,
// Weird false positives
"Buffer": true,
// Puter Common
"puter": true,
"i18n": true,
"html_encode": true,
"html_decode": true,
"isMobile": true,
// Class Registry
"logger": true,
"def": true,
"use": true,
// Libraries
"saveAs": true, // FileSaver
"iro": true, // iro.js color picker
"$": true, // jQuery
"jQuery": true, // jQuery
"fflate": true, // fflate
"_": true, // lodash
"QRCode": true, // qrcode
"io": true, // socket.io
"timeago": true, // timeago
"SelectionArea": true, // viselect
// Puter GUI Globals
"set_menu_item_prop": true,
"determine_active_container_parent": true,
"privacy_aware_path": true,
"api_origin": true,
"auth_token": true,
"logout": true,
"is_email": true,
"select_ctxmenu_item": true,
}
}
},
{
// Mods
// NOTE: Mods have backend and frontend parts, so this just includes the globals for both.
files: [
"mods/**/*.js",
],
languageOptions: {
globals: {
...globals.node,
"use": true,
"window": true,
"puter": true,
}
}
},
{
// Tests
files: [
"**/test/**/*.js",
],
languageOptions: {
globals: {
...globals.mocha,
}
}
},
{
// Phoenix
files: [
"src/phoenix/**/*.js",
],
languageOptions: {
globals: {
...globals.node,
}
}
},
{
// Global rule settings
rules: {
"no-prototype-builtins": "off", // Complains about any use of hasOwnProperty()
"no-unused-vars": "off", // Temporary, we just have a lot of these
"no-debugger": "warn",
"no-async-promise-executor": "off", // We do this quite often and it's fine
}
},
];

31908
output.txt
View File

File diff suppressed because it is too large Load Diff

916
package-lock.json generated
View File

File diff suppressed because it is too large Load Diff

19
package.json
View File

@ -11,20 +11,18 @@
"lib": "lib" "lib": "lib"
}, },
"devDependencies": { "devDependencies": {
"@eslint/js": "^9.16.0", "@eslint/js": "^9.1.1",
"chalk": "^4.1.0", "chalk": "^4.1.0",
"clean-css": "^5.3.2", "clean-css": "^5.3.2",
"dotenv": "^16.4.5", "dotenv": "^16.4.5",
"eslint": "^9.16.0", "eslint": "^9.1.1",
"express": "^4.18.2", "express": "^4.18.2",
"globals": "^15.13.0", "globals": "^15.0.0",
"html-entities": "^2.3.3", "html-entities": "^2.3.3",
"html-webpack-plugin": "^5.6.0", "html-webpack-plugin": "^5.6.0",
"license-check-and-add": "^4.0.5", "license-check-and-add": "^4.0.5",
"mocha": "^10.6.0", "mocha": "^10.6.0",
"nodemon": "^3.1.0", "nodemon": "^3.1.0",
"nyc": "^17.1.0",
"typescript-eslint": "^8.17.0",
"uglify-js": "^3.17.4", "uglify-js": "^3.17.4",
"webpack": "^5.88.2", "webpack": "^5.88.2",
"webpack-cli": "^5.1.1" "webpack-cli": "^5.1.1"
@ -34,8 +32,7 @@
"start=gui": "nodemon --exec \"node dev-server.js\" ", "start=gui": "nodemon --exec \"node dev-server.js\" ",
"start": "node ./tools/run-selfhosted.js", "start": "node ./tools/run-selfhosted.js",
"build": "cd src/gui; node ./build.js", "build": "cd src/gui; node ./build.js",
"check-translations": "node tools/check-translations.js", "check-translations": "node tools/check-translations.js"
"lint": "eslint ."
}, },
"workspaces": [ "workspaces": [
"src/*", "src/*",
@ -56,11 +53,5 @@
"simple-git": "^3.25.0", "simple-git": "^3.25.0",
"string-template": "^1.0.0", "string-template": "^1.0.0",
"uuid": "^9.0.1" "uuid": "^9.0.1"
}, }
"nyc": {
"include": ["src/**/*.js"],
"exclude": ["test/**/*.js"],
"reporter": ["text", "html"],
"all": true
}
} }

172
src/phoenix/test/test-git-detection.js
View File

@ -1,172 +0,0 @@
/*
* Copyright (C) 2024 Puter Technologies Inc.
*
* This file is part of Puter.
*
* Puter is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as published
* by the Free Software Foundation, either version 3 of the License, or
* (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU Affero General Public License for more details.
*
* You should have received a copy of the GNU Affero General Public License
* along with this program. If not, see <https://www.gnu.org/licenses/>.
*/
import assert from 'assert';
// Function to test
async function hasGitDirectory(items) {
// Case 1: Single Puter path
if (typeof items === 'string' && (items.startsWith('/') || items.startsWith('~'))) {
const stat = await global.puter.fs.stat(items);
if (stat.is_dir) {
const files = await global.puter.fs.readdir(items);
return files.some(file => file.name === '.git' && file.is_dir);
}
return false;
}
// Case 2: Array of Puter items
if (Array.isArray(items) && items[0]?.uid) {
return items.some(item => item.name === '.git' && item.is_dir);
}
// Case 3: Local items (DataTransferItems)
if (Array.isArray(items)) {
for (let item of items) {
if (item.fullPath?.includes('/.git/') ||
item.path?.includes('/.git/') ||
item.filepath?.includes('/.git/')) {
return true;
}
}
}
return false;
}
describe('hasGitDirectory', () => {
// Mock puter.fs for testing
const mockPuterFS = {
stat: async (path) => ({
is_dir: path.endsWith('dir')
}),
readdir: async (path) => {
if (path === '/path/to/git/dir') {
return [
{ name: '.git', is_dir: true },
{ name: 'src', is_dir: true }
];
}
return [
{ name: 'src', is_dir: true },
{ name: 'test', is_dir: true }
];
}
};
beforeEach(() => {
// Set up global puter object before each test
global.puter = { fs: mockPuterFS };
});
afterEach(() => {
// Clean up global puter object after each test
delete global.puter;
});
describe('Case 1: Single Puter path', () => {
it('should return true for directory containing .git', async () => {
const result = await hasGitDirectory('/path/to/git/dir');
assert.strictEqual(result, true);
});
it('should return false for directory without .git', async () => {
const result = await hasGitDirectory('/path/to/normal/dir');
assert.strictEqual(result, false);
});
it('should return false for non-directory path', async () => {
const result = await hasGitDirectory('/path/to/file');
assert.strictEqual(result, false);
});
});
describe('Case 2: Array of Puter items', () => {
it('should return true when .git directory is present', async () => {
const items = [
{ uid: '1', name: 'src', is_dir: true },
{ uid: '2', name: '.git', is_dir: true },
{ uid: '3', name: 'test', is_dir: true }
];
const result = await hasGitDirectory(items);
assert.strictEqual(result, true);
});
it('should return false when no .git directory is present', async () => {
const items = [
{ uid: '1', name: 'src', is_dir: true },
{ uid: '2', name: 'test', is_dir: true }
];
const result = await hasGitDirectory(items);
assert.strictEqual(result, false);
});
});
describe('Case 3: Local items (DataTransferItems)', () => {
it('should return true when path contains .git directory', async () => {
const items = [
{ fullPath: '/project/.git/config' },
{ fullPath: '/project/src/index.js' }
];
const result = await hasGitDirectory(items);
assert.strictEqual(result, true);
});
it('should return true when using alternative path properties', async () => {
const items = [
{ path: '/project/.git/config' },
{ filepath: '/project/src/index.js' }
];
const result = await hasGitDirectory(items);
assert.strictEqual(result, true);
});
it('should return false when no .git directory is present', async () => {
const items = [
{ fullPath: '/project/src/index.js' },
{ fullPath: '/project/test/test.js' }
];
const result = await hasGitDirectory(items);
assert.strictEqual(result, false);
});
});
describe('Edge cases', () => {
it('should handle empty array input', async () => {
const result = await hasGitDirectory([]);
assert.strictEqual(result, false);
});
it('should handle invalid input', async () => {
const result = await hasGitDirectory(null);
assert.strictEqual(result, false);
});
it('should handle mixed path formats', async () => {
const items = [
{ fullPath: '/project/src/index.js' },
{ path: '/project/.git/config' },
{ filepath: '/project/test/test.js' }
];
const result = await hasGitDirectory(items);
assert.strictEqual(result, true);
});
});
});

278
src/phoenix/test/test-git-warning-dialog.js
View File

@ -1,278 +0,0 @@
/*
* Copyright (C) 2024 Puter Technologies Inc.
*
* This file is part of Puter.
*
* Puter is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as published
* by the Free Software Foundation, either version 3 of the License, or
* (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU Affero General Public License for more details.
*
* You should have received a copy of the GNU Affero General Public License
* along with this program. If not, see <https://www.gnu.org/licenses/>.
*/
import assert from 'assert';
import { JSDOM } from 'jsdom';
// Function to test
async function showGitWarningDialog() {
try {
// Check if the user has chosen to skip the warning
const skipWarning = await global.puter.kv.get('skip-git-warning');
// Log retrieved value for debugging
console.log('Retrieved skip-git-warning:', skipWarning);
// If the user opted to skip the warning, proceed without showing it
if (skipWarning === true) {
return true;
}
} catch (error) {
console.error('Error accessing KV store:', error);
// If KV store access fails, fall back to showing the dialog
}
// Create the modal dialog
const modal = document.createElement('div');
modal.innerHTML = `
<div style="position: fixed; top: 50%; left: 50%; transform: translate(-50%, -50%); background: white; padding: 20px; border-radius: 8px; box-shadow: 0 4px 10px rgba(0, 0, 0, 0.2); z-index: 10000;">
<h3 style="margin-top: 0;">Warning: Git Repository Detected</h3>
<p>A .git directory was found in your deployment files. Deploying .git directories may:</p>
<ul>
<li>Expose sensitive information like commit history and configuration</li>
<li>Significantly increase deployment size</li>
</ul>
<div style="margin-top: 15px; display: flex; align-items: center;">
<input type="checkbox" id="skip-git-warning" style="margin-right: 10px;">
<label for="skip-git-warning" style="margin-top:0;">Don't show this warning again</label>
</div>
<div style="margin-top: 15px; display: flex; justify-content: flex-end;">
<button id="cancel-deployment" style="margin-right: 10px; padding: 10px 15px; background: #f0f0f0; border: none; border-radius: 4px; cursor: pointer;">Cancel</button>
<button id="continue-deployment" style="padding: 10px 15px; background: #007bff; color: white; border: none; border-radius: 4px; cursor: pointer;">Continue Deployment</button>
</div>
</div>
<div style="position: fixed; top: 0; left: 0; width: 100%; height: 100%; background: rgba(0, 0, 0, 0.5); z-index: 9999;"></div>
`;
document.body.appendChild(modal);
return new Promise((resolve) => {
// Handle "Continue Deployment"
document.getElementById('continue-deployment').addEventListener('click', async () => {
try {
const skipChecked = document.getElementById('skip-git-warning')?.checked;
if (skipChecked) {
console.log("Saving 'skip-git-warning' preference as true");
await global.puter.kv.set('skip-git-warning', true);
}
} catch (error) {
console.error('Error saving user preference to KV store:', error);
} finally {
document.body.removeChild(modal);
resolve(true); // Continue deployment
}
});
// Handle "Cancel Deployment"
document.getElementById('cancel-deployment').addEventListener('click', () => {
document.body.removeChild(modal);
resolve(false); // Cancel deployment
});
});
}
describe('showGitWarningDialog', () => {
let dom;
let consoleLogs = [];
let consoleErrors = [];
// Mock console methods
const mockConsole = {
log: (msg) => consoleLogs.push(msg),
error: (msg) => consoleErrors.push(msg)
};
// Mock puter.kv
const mockPuterKV = {
get: async () => false,
set: async () => {}
};
beforeEach(() => {
// Set up JSDOM with all required features
dom = new JSDOM(`<!DOCTYPE html><html><body></body></html>`, {
url: 'http://localhost',
runScripts: 'dangerously',
resources: 'usable',
pretendToBeVisual: true
});
// Set up global objects
global.document = dom.window.document;
global.window = dom.window;
global.HTMLElement = dom.window.HTMLElement;
global.Element = dom.window.Element;
global.Node = dom.window.Node;
global.navigator = dom.window.navigator;
global.console = { ...console, ...mockConsole };
global.puter = { kv: mockPuterKV };
// Reset console logs
consoleLogs = [];
consoleErrors = [];
});
afterEach(() => {
// Clean up
dom.window.document.body.innerHTML = '';
});
describe('Skip Warning Behavior', () => {
it('should skip dialog if warning is disabled', async () => {
global.puter.kv.get = async () => true;
const result = await showGitWarningDialog();
assert.strictEqual(result, true);
assert.strictEqual(dom.window.document.body.children.length, 0);
assert.ok(consoleLogs.some(log => log.includes('Retrieved skip-git-warning')));
});
it('should show dialog if warning is not disabled', async () => {
global.puter.kv.get = async () => false;
const dialogPromise = showGitWarningDialog();
// Wait for the next tick to allow DOM updates
await new Promise(resolve => setTimeout(resolve, 0));
// Check if dialog is shown
assert.strictEqual(dom.window.document.body.children.length, 1);
assert.ok(dom.window.document.querySelector('h3').textContent.includes('Git Repository Detected'));
// Simulate clicking "Cancel" to resolve the promise
dom.window.document.getElementById('cancel-deployment').click();
const result = await dialogPromise;
assert.strictEqual(result, false);
});
it('should handle KV store error gracefully', async () => {
global.puter.kv.get = async () => {
throw new Error('KV store error');
};
const dialogPromise = showGitWarningDialog();
// Wait for the next tick to allow DOM updates
await new Promise(resolve => setTimeout(resolve, 0));
// Check if dialog is shown
assert.strictEqual(dom.window.document.body.children.length, 1);
assert.ok(consoleErrors.some(error => error.includes('Error accessing KV store')));
// Cleanup
dom.window.document.getElementById('cancel-deployment').click();
await dialogPromise;
});
});
describe('User Interaction', () => {
it('should save preference when checkbox is checked', async () => {
let kvSetCalled = false;
global.puter.kv.set = async (key, value) => {
assert.strictEqual(key, 'skip-git-warning');
assert.strictEqual(value, true);
kvSetCalled = true;
};
const dialogPromise = showGitWarningDialog();
// Wait for the next tick to allow DOM updates
await new Promise(resolve => setTimeout(resolve, 0));
const checkbox = dom.window.document.getElementById('skip-git-warning');
checkbox.checked = true;
dom.window.document.getElementById('continue-deployment').click();
const result = await dialogPromise;
assert.strictEqual(result, true);
assert.ok(kvSetCalled);
assert.ok(consoleLogs.some(log => log.includes("Saving 'skip-git-warning' preference")));
});
it('should not save preference when checkbox is unchecked', async () => {
let kvSetCalled = false;
global.puter.kv.set = async () => {
kvSetCalled = true;
};
const dialogPromise = showGitWarningDialog();
// Wait for the next tick to allow DOM updates
await new Promise(resolve => setTimeout(resolve, 0));
dom.window.document.getElementById('continue-deployment').click();
await dialogPromise;
assert.strictEqual(kvSetCalled, false);
});
it('should handle KV set error gracefully', async () => {
global.puter.kv.set = async () => {
throw new Error('KV set error');
};
const dialogPromise = showGitWarningDialog();
// Wait for the next tick to allow DOM updates
await new Promise(resolve => setTimeout(resolve, 0));
const checkbox = dom.window.document.getElementById('skip-git-warning');
checkbox.checked = true;
dom.window.document.getElementById('continue-deployment').click();
const result = await dialogPromise;
assert.strictEqual(result, true);
assert.ok(consoleErrors.some(error => error.includes('Error saving user preference')));
});
});
describe('Dialog UI', () => {
it('should create dialog with all required elements', async () => {
const dialogPromise = showGitWarningDialog();
// Wait for the next tick to allow DOM updates
await new Promise(resolve => setTimeout(resolve, 0));
assert.ok(dom.window.document.querySelector('h3'));
assert.ok(dom.window.document.querySelector('ul'));
assert.ok(dom.window.document.getElementById('skip-git-warning'));
assert.ok(dom.window.document.getElementById('cancel-deployment'));
assert.ok(dom.window.document.getElementById('continue-deployment'));
// Cleanup
dom.window.document.getElementById('cancel-deployment').click();
await dialogPromise;
});
it('should remove dialog after interaction', async () => {
const dialogPromise = showGitWarningDialog();
// Wait for the next tick to allow DOM updates
await new Promise(resolve => setTimeout(resolve, 0));
assert.strictEqual(dom.window.document.body.children.length, 1);
dom.window.document.getElementById('continue-deployment').click();
await dialogPromise;
assert.strictEqual(dom.window.document.body.children.length, 0);
});
});
});