Merge pull request #17 from maks-rafalko/feature/line-width-120

Use 120 line width

Author: maks-rafalko

.eslintrc.json

@@ -47,7 +47,13 @@
       "@typescript-eslint/semi": ["error", "always"],
       "@typescript-eslint/type-annotation-spacing": "error",
       "@typescript-eslint/unbound-method": "error",
-      "prettier/prettier": ["error", { "semi": true }]
+      "prettier/prettier": [
+        "error",
+        {
+          "semi": true,
+          "printWidth": 120
+        }
+      ]
     },
     "env": {
       "node": true,

.prettierrc.json

@@ -1,5 +1,5 @@
 {
-  "printWidth": 80,
+  "printWidth": 120,
   "tabWidth": 2,
   "useTabs": false,
   "semi": true,

__tests__/main.test.ts

@@ -1,17 +1,9 @@
 import {expect, test} from '@jest/globals';
 
-import {
-  getTouchedSourceFilesRequireTests,
-  getTouchedTestFiles,
-  File
-} from '../src/fileFilters';
+import {getTouchedSourceFilesRequireTests, getTouchedTestFiles, File} from '../src/fileFilters';
 
 test('getTouchedSourceFilesRequireTests should filter out files by file extensions', () => {
-  const files: File[] = [
-    {filename: 'README.md'},
-    {filename: 'foo.php'},
-    {filename: 'bar.ts'}
-  ];
+  const files: File[] = [{filename: 'README.md'}, {filename: 'foo.php'}, {filename: 'bar.ts'}];
   const extensions = ['.php'];
 
   const filteredFiles = getTouchedSourceFilesRequireTests(files, extensions);
@@ -37,54 +29,24 @@ test('getTouchedTestFiles should return test files from the list of different fi
   expect(filteredFilesTestDir[0].filename).toBe('lib/tests/bar1.ts');
   expect(filteredFilesTestDir[1].filename).toBe('lib/tests/foo1.ts');
 
-  const filteredFilesTestPatternSimple = getTouchedTestFiles(
-    files,
-    '',
-    '*_test.ts'
-  );
+  const filteredFilesTestPatternSimple = getTouchedTestFiles(files, '', '*_test.ts');
   expect(filteredFilesTestPatternSimple.length).toBe(1);
   expect(filteredFilesTestPatternSimple[0].filename).toBe('test_test.ts');
 
-  const filteredFilesTestPatternSubdir = getTouchedTestFiles(
-    files,
-    '',
-    '*/test/*.ts'
-  );
+  const filteredFilesTestPatternSubdir = getTouchedTestFiles(files, '', '*/test/*.ts');
   expect(filteredFilesTestPatternSubdir.length).toBe(1);
   expect(filteredFilesTestPatternSubdir[0].filename).toBe('a/test/bar3.ts');
 
-  const filteredFilesTestPatternAllInTests = getTouchedTestFiles(
-    files,
-    '',
-    '**/tests/*.ts'
-  );
+  const filteredFilesTestPatternAllInTests = getTouchedTestFiles(files, '', '**/tests/*.ts');
   expect(filteredFilesTestPatternAllInTests.length).toBe(3);
-  expect(filteredFilesTestPatternAllInTests[0].filename).toBe(
-    'lib/tests/bar1.ts'
-  );
-  expect(filteredFilesTestPatternAllInTests[1].filename).toBe(
-    'lib/tests/foo1.ts'
-  );
-  expect(filteredFilesTestPatternAllInTests[2].filename).toBe(
-    'a/tests/bar4.ts'
-  );
+  expect(filteredFilesTestPatternAllInTests[0].filename).toBe('lib/tests/bar1.ts');
+  expect(filteredFilesTestPatternAllInTests[1].filename).toBe('lib/tests/foo1.ts');
+  expect(filteredFilesTestPatternAllInTests[2].filename).toBe('a/tests/bar4.ts');
 
-  const filteredFilesTestBothDirAndPattern = getTouchedTestFiles(
-    files,
-    'lib/testsxxx',
-    '**/tests/*.ts'
-  );
+  const filteredFilesTestBothDirAndPattern = getTouchedTestFiles(files, 'lib/testsxxx', '**/tests/*.ts');
   expect(filteredFilesTestBothDirAndPattern.length).toBe(4);
-  expect(filteredFilesTestBothDirAndPattern[0].filename).toBe(
-    'lib/testsxxx/bar2.ts'
-  );
-  expect(filteredFilesTestBothDirAndPattern[1].filename).toBe(
-    'lib/tests/bar1.ts'
-  );
-  expect(filteredFilesTestBothDirAndPattern[2].filename).toBe(
-    'lib/tests/foo1.ts'
-  );
-  expect(filteredFilesTestBothDirAndPattern[3].filename).toBe(
-    'a/tests/bar4.ts'
-  );
+  expect(filteredFilesTestBothDirAndPattern[0].filename).toBe('lib/testsxxx/bar2.ts');
+  expect(filteredFilesTestBothDirAndPattern[1].filename).toBe('lib/tests/bar1.ts');
+  expect(filteredFilesTestBothDirAndPattern[2].filename).toBe('lib/tests/foo1.ts');
+  expect(filteredFilesTestBothDirAndPattern[3].filename).toBe('a/tests/bar4.ts');
 });

dist/index.js

@@ -4,22 +4,15 @@ export type File = {
   filename: string;
 };
 
-export function getTouchedSourceFilesRequireTests(
-  files: File[],
-  fileExtensions: string[]
-): File[] {
+export function getTouchedSourceFilesRequireTests(files: File[], fileExtensions: string[]): File[] {
   return files.filter(file => {
     return fileExtensions.find((fileExtension: string) => {
       return file.filename.endsWith(fileExtension);
     });
   });
 }
 
-export function getTouchedTestFiles(
-  files: File[],
-  testDir: string,
-  testPattern: string
-): File[] {
+export function getTouchedTestFiles(files: File[], testDir: string, testPattern: string): File[] {
   let filtered: File[] = [];
 
   if (testDir) {

dist/index.js.map

@@ -1,9 +1,6 @@
 import * as core from '@actions/core';
 import * as github from '@actions/github';
-import {
-  getTouchedSourceFilesRequireTests,
-  getTouchedTestFiles
-} from './fileFilters';
+import {getTouchedSourceFilesRequireTests, getTouchedTestFiles} from './fileFilters';
 
 function getFileExtensions(): string[] {
   const fileExtensionsInput = core.getInput('fileExtensions');
@@ -20,18 +17,14 @@ async function run(): Promise<void> {
     const {context} = github;
 
     const config = {
-      comment:
-        core.getInput('comment') ||
-        'Could you please add tests to make sure this change works as expected?',
+      comment: core.getInput('comment') || 'Could you please add tests to make sure this change works as expected?',
       fileExtensions: getFileExtensions(),
       testDir: core.getInput('testDir') || 'tests',
       testPattern: core.getInput('testPattern') || ''
     };
 
     if (context.payload.pull_request == null) {
-      core.debug(
-        'This action is supposed to be run on pull_request event only.'
-      );
+      core.debug('This action is supposed to be run on pull_request event only.');
     }
 
     const githubToken: string = core.getInput('GITHUB_TOKEN');
@@ -56,21 +49,14 @@ async function run(): Promise<void> {
       res => res.data
     );
 
-    const sourceFilesRequireTests = getTouchedSourceFilesRequireTests(
-      allFiles,
-      config.fileExtensions
-    );
+    const sourceFilesRequireTests = getTouchedSourceFilesRequireTests(allFiles, config.fileExtensions);
 
     if (sourceFilesRequireTests.length === 0) {
       core.debug('No source files require tests, exiting');
       return;
     }
 
-    const testFiles = getTouchedTestFiles(
-      allFiles,
-      config.testDir,
-      config.testPattern
-    );
+    const testFiles = getTouchedTestFiles(allFiles, config.testDir, config.testPattern);
 
     if (testFiles.length !== 0) {
       core.debug('Test files were touched, exiting');