NCIOCPL / drug-dictionary-app

NCI Drug Dictionary Application
2 stars 0 forks source link

Ensure ES-Lint is config is standardized, enabled, and unneccessary modules uninstalled #142

Open adriancofie opened 1 year ago

adriancofie commented 1 year ago
adriancofie commented 1 year ago

/drug-dictionary-app/src/__tests__/App.test.js
  1:15  error  `cleanup` is performed automatically by your test runner, you don't need manual cleanups  testing-library/no-manual-cleanup

/drug-dictionary-app/src/components/atomic/Autocomplete/__tests__/Autocomplete.tests.js
    1:10  error  `cleanup` is performed automatically by your test runner, you don't need manual cleanups             testing-library/no-manual-cleanup
  108:23  error  Avoid using container methods. Prefer using the methods from Testing Library, such as "getByRole()"  testing-library/no-container
  108:33  error  Avoid direct Node access. Prefer using the methods from Testing Library                              testing-library/no-node-access
  108:33  error  Avoid direct Node access. Prefer using the methods from Testing Library                              testing-library/no-node-access

/drug-dictionary-app/src/components/atomic/InputLabel/__tests__/InputLabel.tests.js
   9:1   error  describe should not have an empty title                                                                                                        jest/valid-title
  10:8   error  `wrapper` is not a recommended name for `render` returned value. Instead, you should destructure it, or name it using one of: `view`, `utils`  testing-library/render-result-naming-convention
  23:17  error  Avoid destructuring queries from `render` result, use `screen.getByTestId` instead                                                             testing-library/prefer-screen-queries
  25:10  error  Avoid destructuring queries from `render` result, use `screen.getByText` instead                                                               testing-library/prefer-screen-queries

/drug-dictionary-app/src/components/atomic/RemovableTag/__tests__/RemovableTag.tests.js
  15:20  error  Avoid using container methods. Prefer using the methods from Testing Library, such as "getByRole()"  testing-library/no-container
  15:30  error  Avoid direct Node access. Prefer using the methods from Testing Library                              testing-library/no-node-access
  15:30  error  Avoid direct Node access. Prefer using the methods from Testing Library                              testing-library/no-node-access
  16:10  error  Avoid using container methods. Prefer using the methods from Testing Library, such as "getByRole()"  testing-library/no-container
  16:20  error  Avoid direct Node access. Prefer using the methods from Testing Library                              testing-library/no-node-access
  18:4   error  Avoid using container methods. Prefer using the methods from Testing Library, such as "getByRole()"  testing-library/no-container
  18:14  error  Avoid direct Node access. Prefer using the methods from Testing Library                              testing-library/no-node-access

/drug-dictionary-app/src/components/atomic/SynonymTable/__tests__/synonym-table.tests.js
  43:9   error  `wrapper` is not a recommended name for `render` returned value. Instead, you should destructure it, or name it using one of: `view`, `utils`  testing-library/render-result-naming-convention
  45:18  error  Avoid destructuring queries from `render` result, use `screen.getByText` instead                                                               testing-library/prefer-screen-queries
  47:18  error  Avoid destructuring queries from `render` result, use `screen.getByText` instead                                                               testing-library/prefer-screen-queries
  48:18  error  Avoid destructuring queries from `render` result, use `screen.getByText` instead                                                               testing-library/prefer-screen-queries
  49:18  error  Avoid destructuring queries from `render` result, use `screen.getByText` instead                                                               testing-library/prefer-screen-queries
  51:18  error  Avoid destructuring queries from `render` result, use `screen.getByText` instead                                                               testing-library/prefer-screen-queries
  53:18  error  Avoid destructuring queries from `render` result, use `screen.queryByText` instead                                                             testing-library/prefer-screen-queries
  54:18  error  Avoid destructuring queries from `render` result, use `screen.queryByText` instead                                                             testing-library/prefer-screen-queries

/drug-dictionary-app/src/components/atomic/TextInput/__tests__/TextInput.test.js
    1:10  error  `cleanup` is performed automatically by your test runner, you don't need manual cleanups     testing-library/no-manual-cleanup
   20:13  error  Forbidden usage of `render` within testing framework `beforeEach` setup                      testing-library/no-render-in-setup
   27:10  error  Avoid destructuring queries from `render` result, use `screen.getByLabelText` instead        testing-library/prefer-screen-queries
   28:10  error  Avoid destructuring queries from `render` result, use `screen.getByPlaceholderText` instead  testing-library/prefer-screen-queries
   29:10  error  Avoid destructuring queries from `render` result, use `screen.getByText` instead             testing-library/prefer-screen-queries
   35:21  error  Avoid destructuring queries from `render` result, use `screen.getByPlaceholderText` instead  testing-library/prefer-screen-queries
   37:10  error  Avoid destructuring queries from `render` result, use `screen.getByDisplayValue` instead     testing-library/prefer-screen-queries
   80:22  error  Avoid destructuring queries from `render` result, use `screen.getByPlaceholderText` instead  testing-library/prefer-screen-queries
  114:22  error  Avoid destructuring queries from `render` result, use `screen.getByPlaceholderText` instead  testing-library/prefer-screen-queries
  116:18  error  Avoid destructuring queries from `render` result, use `screen.getByTestId` instead           testing-library/prefer-screen-queries
  119:11  error  Avoid destructuring queries from `render` result, use `screen.getByText` instead             testing-library/prefer-screen-queries

/drug-dictionary-app/src/components/atomic/az-list/__tests__/AZList.tests.js
  27:9   error  `wrapper` is not a recommended name for `render` returned value. Instead, you should destructure it, or name it using one of: `view`, `utils`  testing-library/render-result-naming-convention
  36:35  error  Avoid direct Node access. Prefer using the methods from Testing Library                                                                        testing-library/no-node-access
  36:35  error  Avoid direct Node access. Prefer using the methods from Testing Library                                                                        testing-library/no-node-access
  38:24  error  Avoid direct Node access. Prefer using the methods from Testing Library                                                                        testing-library/no-node-access
  39:3   error  Expect must have a corresponding matcher call                                                                                                  jest/valid-expect
  39:10  error  Avoid destructuring queries from `render` result, use `screen.getByTestId` instead                                                             testing-library/prefer-screen-queries
  43:9   error  `wrapper` is not a recommended name for `render` returned value. Instead, you should destructure it, or name it using one of: `view`, `utils`  testing-library/render-result-naming-convention
  51:35  error  Avoid direct Node access. Prefer using the methods from Testing Library                                                                        testing-library/no-node-access
  51:35  error  Avoid direct Node access. Prefer using the methods from Testing Library                                                                        testing-library/no-node-access
  53:30  error  Avoid direct Node access. Prefer using the methods from Testing Library                                                                        testing-library/no-node-access
  53:30  error  Avoid direct Node access. Prefer using the methods from Testing Library                                                                        testing-library/no-node-access
  53:41  error  Avoid direct Node access. Prefer using the methods from Testing Library                                                                        testing-library/no-node-access
  53:41  error  Avoid direct Node access. Prefer using the methods from Testing Library                                                                        testing-library/no-node-access

/drug-dictionary-app/src/components/atomic/spinner/__tests__/Spinner.tests.js
   9:10  error  Avoid using container methods. Prefer using the methods from Testing Library, such as "getByRole()"  testing-library/no-container
   9:20  error  Avoid direct Node access. Prefer using the methods from Testing Library                              testing-library/no-node-access
  10:10  error  Avoid using container methods. Prefer using the methods from Testing Library, such as "getByRole()"  testing-library/no-container
  10:20  error  Avoid direct Node access. Prefer using the methods from Testing Library                              testing-library/no-node-access
  11:10  error  Avoid using container methods. Prefer using the methods from Testing Library, such as "getByRole()"  testing-library/no-container
  11:20  error  Avoid direct Node access. Prefer using the methods from Testing Library                              testing-library/no-node-access
  12:10  error  Avoid using container methods. Prefer using the methods from Testing Library, such as "getByRole()"  testing-library/no-container
  12:20  error  Avoid direct Node access. Prefer using the methods from Testing Library                              testing-library/no-node-access

/drug-dictionary-app/src/components/molecules/SearchTermList/__tests__/search-term-list-test.js
  108:5  error  Unexpected focused test  jest/no-focused-tests

/drug-dictionary-app/src/components/molecules/SearchTermListItem/__tests__/search-term-list-item.test.js
  32:10  error  Avoid using container methods. Prefer using the methods from Testing Library, such as "getByRole()"  testing-library/no-container
  32:20  error  Avoid direct Node access. Prefer using the methods from Testing Library                              testing-library/no-node-access
  36:10  error  Avoid using container methods. Prefer using the methods from Testing Library, such as "getByRole()"  testing-library/no-container
  36:20  error  Avoid direct Node access. Prefer using the methods from Testing Library                              testing-library/no-node-access
  62:10  error  Avoid using container methods. Prefer using the methods from Testing Library, such as "getByRole()"  testing-library/no-container
  62:20  error  Avoid direct Node access. Prefer using the methods from Testing Library                              testing-library/no-node-access

/drug-dictionary-app/src/components/molecules/TermList/__tests__/term-list-test.js
  108:5  error  Unexpected focused test  jest/no-focused-tests

/drug-dictionary-app/src/components/molecules/TermListItem/__tests__/term-list-item.test.js
  32:10  error  Avoid using container methods. Prefer using the methods from Testing Library, such as "getByRole()"  testing-library/no-container
  32:20  error  Avoid direct Node access. Prefer using the methods from Testing Library                              testing-library/no-node-access
  36:10  error  Avoid using container methods. Prefer using the methods from Testing Library, such as "getByRole()"  testing-library/no-container
  36:20  error  Avoid direct Node access. Prefer using the methods from Testing Library                              testing-library/no-node-access
  62:10  error  Avoid using container methods. Prefer using the methods from Testing Library, such as "getByRole()"  testing-library/no-container
  62:20  error  Avoid direct Node access. Prefer using the methods from Testing Library                              testing-library/no-node-access

/drug-dictionary-app/src/components/molecules/definitionitem/__tests__/DefinitionItem.tests.js
  176:8   error  `wrapper` is not a recommended name for `render` returned value. Instead, you should destructure it, or name it using one of: `view`, `utils`  testing-library/render-result-naming-convention
  194:14  error  Avoid direct Node access. Prefer using the methods from Testing Library                                                                        testing-library/no-node-access
  194:14  error  Avoid direct Node access. Prefer using the methods from Testing Library                                                                        testing-library/no-node-access
  198:14  error  Avoid direct Node access. Prefer using the methods from Testing Library                                                                        testing-library/no-node-access
  203:9   error  `wrapper` is not a recommended name for `render` returned value. Instead, you should destructure it, or name it using one of: `view`, `utils`  testing-library/render-result-naming-convention
  218:34  error  Avoid direct Node access. Prefer using the methods from Testing Library                                                                        testing-library/no-node-access
  218:34  error  Avoid direct Node access. Prefer using the methods from Testing Library                                                                        testing-library/no-node-access

/drug-dictionary-app/src/components/molecules/search-box/__tests__/SearchBox.tests.js
  23:8   error  `wrapper` is not a recommended name for `render` returned value. Instead, you should destructure it, or name it using one of: `view`, `utils`  testing-library/render-result-naming-convention
  34:3   error  Expect must have a corresponding matcher call                                                                                                  jest/valid-expect
  34:10  error  Avoid destructuring queries from `render` result, use `screen.getByTestId` instead                                                             testing-library/prefer-screen-queries
  36:3   error  Expect must have a corresponding matcher call                                                                                                  jest/valid-expect
  36:10  error  Avoid destructuring queries from `render` result, use `screen.getByTestId` instead                                                             testing-library/prefer-screen-queries

/drug-dictionary-app/src/components/molecules/search/__tests__/Search.test.js
    3:2   error  `cleanup` is performed automatically by your test runner, you don't need manual cleanups     testing-library/no-manual-cleanup
   61:20  error  Forbidden usage of `render` within testing framework `beforeEach` setup                      testing-library/no-render-in-setup
   79:10  error  Avoid destructuring queries from `render` result, use `screen.getByTestId` instead           testing-library/prefer-screen-queries
   84:10  error  Avoid destructuring queries from `render` result, use `screen.getByDisplayValue` instead     testing-library/prefer-screen-queries
   85:10  error  Avoid destructuring queries from `render` result, use `screen.getByDisplayValue` instead     testing-library/prefer-screen-queries
   90:27  error  Avoid destructuring queries from `render` result, use `screen.getByDisplayValue` instead     testing-library/prefer-screen-queries
  104:24  error  Avoid destructuring queries from `render` result, use `screen.getByText` instead             testing-library/prefer-screen-queries
  121:21  error  Avoid destructuring queries from `render` result, use `screen.getByPlaceholderText` instead  testing-library/prefer-screen-queries
  122:24  error  Avoid destructuring queries from `render` result, use `screen.getByText` instead             testing-library/prefer-screen-queries
  142:25  error  Avoid destructuring queries from `render` result, use `screen.getByDisplayValue` instead     testing-library/prefer-screen-queries
  144:21  error  Avoid destructuring queries from `render` result, use `screen.getByPlaceholderText` instead  testing-library/prefer-screen-queries
  148:24  error  Avoid destructuring queries from `render` result, use `screen.getByText` instead             testing-library/prefer-screen-queries
  156:21  error  Avoid destructuring queries from `render` result, use `screen.getByPlaceholderText` instead  testing-library/prefer-screen-queries
  160:24  error  Avoid destructuring queries from `render` result, use `screen.getByText` instead             testing-library/prefer-screen-queries
  168:21  error  Avoid destructuring queries from `render` result, use `screen.getByPlaceholderText` instead  testing-library/prefer-screen-queries
  171:11  error  Wrap stand-alone `getBy*` query with `expect` function for better explicit assertion         testing-library/prefer-explicit-assert
  232:31  error  Avoid direct Node access. Prefer using the methods from Testing Library                      testing-library/no-node-access
  235:15  error  Avoid direct Node access. Prefer using the methods from Testing Library                      testing-library/no-node-access
  238:31  error  Avoid direct Node access. Prefer using the methods from Testing Library                      testing-library/no-node-access

/drug-dictionary-app/src/hooks/__tests__/autoSuggestQuery.test.js
   68:10  error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
   70:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  161:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  164:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  167:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  170:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  173:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  176:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  179:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  182:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  185:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  188:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  270:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  273:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  276:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  279:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  281:10  error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  283:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  286:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  289:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  291:10  error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  293:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  296:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries
  302:4   error  Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries

/drug-dictionary-app/src/hooks/__tests__/customFetch.test.js
   13:1   error    describe should not have an empty title                                           jest/valid-title
  101:2   warning  Some tests seem to be commented                                                   jest/no-commented-out-tests
  180:10  error    Avoid destructuring queries from `render` result, use `screen.getByText` instead  testing-library/prefer-screen-queries

/drug-dictionary-app/src/services/__tests__/buildAxiosRequest.test.js
  31:2  warning  Some tests seem to be commented       jest/no-commented-out-tests
  45:2  warning  Some tests seem to be commented       jest/no-commented-out-tests
  60:2  warning  Some tests seem to be commented       jest/no-commented-out-tests
  87:3  error    Jest tests should not return a value  jest/no-test-return-statement

/drug-dictionary-app/src/utils/__tests__/url.test.js
   3:1  error  All test cases must be wrapped in a describe block  jest/require-top-level-describe
  11:1  error  All test cases must be wrapped in a describe block  jest/require-top-level-describe

/drug-dictionary-app/src/views/Definition/__tests__/Definition.PrettyURLRedirect.test.js
  30:1  error  All test cases must be wrapped in a describe block  jest/require-top-level-describe

/drug-dictionary-app/src/views/ErrorBoundary/__tests__/ErrorPage.test.js
  1:15  error  `cleanup` is performed automatically by your test runner, you don't need manual cleanups  testing-library/no-manual-cleanup

/drug-dictionary-app/src/views/ErrorBoundary/__tests__/PageNotFound.test.js
   3:2  error  `cleanup` is performed automatically by your test runner, you don't need manual cleanups  testing-library/no-manual-cleanup
  49:5  error  Test title is used multiple times in the same describe block                              jest/no-identical-title

✖ 138 problems (134 errors, 4 warnings)
adriancofie commented 7 months ago

Valid