fix: client: dom: toggleSelectedFile: return DOM

Author: coderaiser

.madrun.js

@@ -29,7 +29,7 @@ export default {
     'fix:lint': async () => `putout --fix . && redlint fix`,
     'lint:stream': () => run('lint', '-f stream'),
     'test': () => [testEnv, `tape '{test}/**/*.js' '{bin,client,static,common,server}/**/*.spec.js' -f fail`],
-    'test:e2e': () => `tape 'test-e2e/ratelimit.js'`,
+    'test:e2e': () => `tape 'test-e2e/server/**/*.js'`,
     'test:e2e:client': () => 'playwright test',
     'test:client': () => `tape 'test/client/**/*.js'`,
     'test:server': () => `tape 'test/**/*.js' 'server/**/*.spec.js' 'common/**/*.spec.js'`,

client/dom/cmd.js

@@ -1,14 +1,5 @@
 /* global DOM */
 const SELECTED_FILE = 'selected-file';
-const Cmd = {
-    getSelectedFiles,
-    isSelected,
-    unselectFile,
-    selectFile,
-    selectAllFiles,
-    toggleSelectedFile,
-    toggleAllSelectedFiles,
-};
 
 /**
  * selected file check
@@ -31,43 +22,43 @@ export function selectFile(currentFile) {
 
     current.classList.add(SELECTED_FILE);
 
-    return Cmd;
+    return DOM;
 }
 
 export function unselectFile(currentFile) {
     const current = currentFile || DOM.getCurrentFile();
 
     current.classList.remove(SELECTED_FILE);
 
-    return Cmd;
+    return DOM;
 }
 
 export function toggleSelectedFile(currentFile) {
     const current = currentFile || DOM.getCurrentFile();
     const name = DOM.getCurrentName(current);
 
     if (name === '..')
-        return Cmd;
+        return DOM;
 
     current.classList.toggle(SELECTED_FILE);
 
-    return Cmd;
+    return DOM;
 }
 
 export function toggleAllSelectedFiles() {
     DOM
         .getAllFiles()
         .map(DOM.toggleSelectedFile);
 
-    return Cmd;
+    return DOM;
 }
 
 export function selectAllFiles() {
     DOM
         .getAllFiles()
         .map(DOM.selectFile);
 
-    return Cmd;
+    return DOM;
 }
 
 /**

client/key/index.js

@@ -161,13 +161,13 @@ export async function _switchKey(event) {
         break;
 
     case KEY.INSERT:
-        DOM.toggleSelectedFile(current);
-        DOM.setCurrentFile(next);
+        DOM.toggleSelectedFile(current)
+            .setCurrentFile(next);
         break;
 
     case KEY.INSERT_MAC:
-        DOM.toggleSelectedFile(current);
-        DOM.setCurrentFile(next);
+        DOM.toggleSelectedFile(current)
+            .setCurrentFile(next);
         break;
 
     case KEY.DELETE:

playwright.config.js

@@ -2,14 +2,14 @@ import {defineConfig} from '@playwright/test';
 
 export default defineConfig({
     testDir: './test-e2e',
-    testMatch: '**/*.spec.js',
+    testMatch: 'client/**/*.js',
     timeout: 30_000,
     use: {
         baseURL: 'http://localhost:3002',
         headless: true,
     },
     webServer: {
-        command: 'node bin/cloudcmd.js --no-auth --port 3002 --root test-e2e/fixture --no-open',
+        command: 'node bin/cloudcmd.js --no-auth --port 3002 --root test-e2e/client/fixture --no-open',
         url: 'http://localhost:3002',
         reuseExistingServer: false,
         timeout: 15_000,

test-e2e/fixture/copy.txt test-e2e/client/fixture/copy.txttest-e2e/fixture/copy.txt renamed to test-e2e/client/fixture/copy.txt

@@ -8,7 +8,6 @@ test('Insert selects the current file', async ({page}) => {
     const rows = getRows(page);
     const first = rows.nth(1);
 
-    // nth(0) is '..'
     await first.click();
     await page.keyboard.press('Insert');