Release v1.143.0 - release → staging #12291
reviewdog [eslint] report
reported by reviewdog 🐶
Findings (201)
packages/manager/cypress/e2e/core/account/quotas.spec.ts|6 col 8| Complete the task associated to this "TODO" comment.
packages/manager/cypress/e2e/core/account/quotas.spec.ts|15 col 13| Define a constant instead of duplicating this literal 6 times.
packages/manager/cypress/e2e/core/account/quotas.spec.ts|19 col 14| Define a constant instead of duplicating this literal 6 times.
packages/manager/cypress/e2e/core/account/quotas.spec.ts|28 col 14| Define a constant instead of duplicating this literal 3 times.
packages/manager/cypress/e2e/core/account/quotas.spec.ts|39 col 16| Define a constant instead of duplicating this literal 4 times.
packages/manager/cypress/e2e/core/firewalls/create-firewall.spec.ts|168 col 15| Define a constant instead of duplicating this literal 3 times.
packages/manager/cypress/e2e/core/kubernetes/lke-create.spec.ts|949 col 53| Refactor this code to not nest functions more than 4 levels deep.
packages/manager/cypress/e2e/core/kubernetes/lke-create.spec.ts|1172 col 53| Refactor this code to not nest functions more than 4 levels deep.
packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts|485 col 33| Define a constant instead of duplicating this literal 3 times.
packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts|866 col 12| Complete the task associated to this "TODO" comment.
packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts|873 col 12| Complete the task associated to this "TODO" comment.
packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts|915 col 22| Refactor this code to not nest functions more than 4 levels deep.
packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts|927 col 22| Refactor this code to not nest functions more than 4 levels deep.
packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts|954 col 22| Refactor this code to not nest functions more than 4 levels deep.
packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts|1032 col 24| Define a constant instead of duplicating this literal 3 times.
packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts|1034 col 22| Refactor this code to not nest functions more than 4 levels deep.
packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts|1063 col 22| Refactor this code to not nest functions more than 4 levels deep.
packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts|1122 col 22| Refactor this code to not nest functions more than 4 levels deep.
packages/manager/cypress/e2e/core/linodes/upgrade-linode-interface.spec.ts|43 col 8| Complete the task associated to this "TODO" comment.
packages/manager/cypress/e2e/core/linodes/upgrade-linode-interface.spec.ts|50 col 8| Complete the task associated to this "TODO" comment.
packages/manager/cypress/e2e/core/linodes/upgrade-linode-interface.spec.ts|82 col 7| Define a constant instead of duplicating this literal 49 times.
packages/manager/cypress/e2e/core/linodes/upgrade-linode-interface.spec.ts|87 col 19| Define a constant instead of duplicating this literal 4 times.
packages/manager/cypress/e2e/core/linodes/upgrade-linode-interface.spec.ts|98 col 19| Define a constant instead of duplicating this literal 14 times.
packages/manager/cypress/e2e/core/linodes/upgrade-linode-interface.spec.ts|191 col 20| Define a constant instead of duplicating this literal 5 times.
packages/manager/cypress/e2e/core/nodebalancers/nodebalancer-create-with-vpc.spec.ts|84 col 48| Define a constant instead of duplicating this literal 6 times.
packages/manager/cypress/e2e/core/vpc/vpc-create.spec.ts|370 col 15| Define a constant instead of duplicating this literal 3 times.
packages/manager/cypress/support/constants/linode-interfaces.ts|5 col 35| Insert ⏎·
packages/manager/cypress/support/constants/linode-interfaces.ts|7 col 35| Insert ⏎·
packages/manager/cypress/support/constants/linode-interfaces.ts|9 col 40| Insert ⏎·
packages/manager/cypress/support/constants/linode-interfaces.ts|11 col 40| Insert ⏎·
packages/manager/cypress/support/constants/linode-interfaces.ts|13 col 51| Insert ⏎·
packages/manager/cypress/support/util/linodes.ts|233 col 13| Define a constant instead of duplicating this literal 26 times.
packages/manager/eslint.config.js|156 col 59| Complete the task associated to this "TODO" comment.
packages/manager/src/GoTo.tsx|120 col 5| React Hook React.useMemo has a missing dependency: 'isDatabasesEnabled'. Either include it or remove the dependency array.
packages/manager/src/components/PlatformMaintenanceBanner/LinodePlatformMaintenanceBanner.test.tsx|48 col 19| Define a constant instead of duplicating this literal 3 times.
packages/manager/src/components/PlatformMaintenanceBanner/LinodePlatformMaintenanceBanner.tsx|67 col 25| ** No font weight declarations in our code in order to avoid faux-bold **.
We manage font weights through font family declarations.
For example, instead of specifying fontWeight: "bold",
use: fontFamily: theme.font.bold.
packages/manager/src/components/PlatformMaintenanceBanner/LinodePlatformMaintenanceBanner.tsx|75 col 25| ** No font weight declarations in our code in order to avoid faux-bold .
We manage font weights through font family declarations.
For example, instead of specifying fontWeight: "bold",
use: fontFamily: theme.font.bold.
packages/manager/src/dev-tools/DevTools.tsx|41 col 13| Define a constant instead of duplicating this literal 3 times.
packages/manager/src/dev-tools/DevTools.tsx|49 col 13| Define a constant instead of duplicating this literal 3 times.
packages/manager/src/dev-tools/DevTools.tsx|90 col 6| React Hook React.useEffect has a missing dependency: 'isDraggable'. Either include it or remove the dependency array.
packages/manager/src/features/DataStream/Streams/StreamCreate/StreamCreateDataSet.tsx|12 col 14| Complete the task associated to this "TODO" comment.
packages/manager/src/features/DataStream/Streams/StreamCreate/StreamCreateDelivery.tsx|12 col 14| Complete the task associated to this "TODO" comment.
packages/manager/src/features/DataStream/Streams/StreamsLandingEmptyStateData.ts|26 col 10| Complete the task associated to this "TODO" comment.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseCreate.test.tsx|55 col 37| Prefer using queryBy when waiting for disappearance
packages/manager/src/features/Databases/DatabaseCreate/DatabaseCreate.test.tsx|55 col 37| Avoid destructuring queries from render result, use screen.getAllByTestId instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseCreate.test.tsx|56 col 5| Avoid destructuring queries from render result, use screen.getAllByText instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseCreate.tsx|140 col 8| Complete the task associated to this "TODO" comment.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseCreateNetworkingConfiguration.test.tsx|41 col 7| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseCreateNetworkingConfiguration.test.tsx|41 col 7| Don't wrap getBy* query with expect & presence matchers like toBeInTheDocument or not.toBeNull as getBy* queries fail implicitly when element is not found
packages/manager/src/features/Databases/DatabaseCreate/DatabaseCreateNetworkingConfiguration.test.tsx|46 col 7| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseCreateNetworkingConfiguration.test.tsx|46 col 7| Don't wrap getBy* query with expect & presence matchers like toBeInTheDocument or not.toBeNull as getBy* queries fail implicitly when element is not found
packages/manager/src/features/Databases/DatabaseCreate/DatabaseCreateNetworkingConfiguration.test.tsx|56 col 12| Avoid destructuring queries from render result, use screen.getByTestId instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseCreateNetworkingConfiguration.test.tsx|56 col 12| Don't wrap getBy* query with expect & presence matchers like toBeInTheDocument or not.toBeNull as getBy* queries fail implicitly when element is not found
packages/manager/src/features/Databases/DatabaseCreate/DatabaseCreateNetworkingConfiguration.test.tsx|57 col 12| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseCreateNetworkingConfiguration.test.tsx|57 col 12| Don't wrap getBy* query with expect & presence matchers like toBeInTheDocument or not.toBeNull as getBy* queries fail implicitly when element is not found
packages/manager/src/features/Databases/DatabaseCreate/DatabaseSummarySection.test.tsx|67 col 26| Avoid destructuring queries from render result, use screen.getByTestId instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseSummarySection.test.tsx|67 col 55| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseSummarySection.test.tsx|67 col 55| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseSummarySection.test.tsx|74 col 32| Avoid destructuring queries from render result, use screen.findByText instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseSummarySection.test.tsx|78 col 25| Avoid destructuring queries from render result, use screen.getByTestId instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseSummarySection.test.tsx|78 col 62| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseSummarySection.test.tsx|78 col 62| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseSummarySection.test.tsx|82 col 26| Avoid destructuring queries from render result, use screen.findByText instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|57 col 36| This function has 357 lines, which is greater than the 200 lines authorized. Split it into smaller functions.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|87 col 12| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|87 col 12| Don't wrap getBy* query with expect & presence matchers like toBeInTheDocument or not.toBeNull as getBy* queries fail implicitly when element is not found
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|94 col 25| Avoid destructuring queries from render result, use screen.getByTestId instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|94 col 37| Define a constant instead of duplicating this literal 7 times.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|96 col 24| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|98 col 7| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|98 col 7| Don't wrap getBy* query with expect & presence matchers like toBeInTheDocument or not.toBeNull as getBy* queries fail implicitly when element is not found
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|99 col 9| Define a constant instead of duplicating this literal 3 times.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|127 col 25| Avoid destructuring queries from render result, use screen.getByTestId instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|129 col 24| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|157 col 25| Avoid destructuring queries from render result, use screen.getByTestId instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|158 col 42| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|158 col 42| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|162 col 28| Avoid destructuring queries from render result, use screen.getByTestId instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|162 col 40| Define a constant instead of duplicating this literal 4 times.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|164 col 27| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|192 col 25| Avoid destructuring queries from render result, use screen.getByTestId instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|193 col 42| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|193 col 42| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|196 col 28| Avoid destructuring queries from render result, use screen.getByTestId instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|198 col 34| Avoid destructuring queries from render result, use screen.getByTestId instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|202 col 27| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|206 col 33| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|264 col 25| Avoid destructuring queries from render result, use screen.getByTestId instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|265 col 25| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|309 col 7| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|309 col 7| Don't wrap getBy* query with expect & presence matchers like toBeInTheDocument or not.toBeNull as getBy* queries fail implicitly when element is not found
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|324 col 7| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|324 col 7| Don't wrap getBy* query with expect & presence matchers like toBeInTheDocument or not.toBeNull as getBy* queries fail implicitly when element is not found
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|343 col 7| Avoid destructuring queries from render result, use screen.queryByText instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|346 col 7| Avoid destructuring queries from render result, use screen.queryByText instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|375 col 28| Avoid destructuring queries from render result, use screen.getByTestId instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|377 col 12| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|377 col 12| Don't wrap getBy* query with expect & presence matchers like toBeInTheDocument or not.toBeNull as getBy* queries fail implicitly when element is not found
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|401 col 25| Avoid destructuring queries from render result, use screen.getByTestId instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|402 col 37| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|402 col 37| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|436 col 25| Avoid destructuring queries from render result, use screen.getByTestId instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|436 col 62| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|436 col 62| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|443 col 26| Avoid destructuring queries from render result, use screen.findByText instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|473 col 28| Avoid destructuring queries from render result, use screen.getByTestId instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|475 col 7| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|475 col 7| Avoid direct Node access. Prefer using the methods from Testing Library.
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.test.tsx|481 col 29| Avoid destructuring queries from render result, use screen.findByText instead
packages/manager/src/features/Databases/DatabaseCreate/DatabaseVPCSelector.tsx|82 col 6| React Hook React.useEffect has missing dependencies: 'onConfigurationChange' and 'resetVPCConfiguration'. Either include them or remove the dependency array.
packages/manager/src/features/Help/Panels/AlgoliaSearchBar.tsx|24 col 7| Fast refresh only works when a file only exports components. Move your component(s) to a separate file.
packages/manager/src/features/Help/Panels/AlgoliaSearchBar.tsx|124 col 16| Fast refresh can't handle anonymous components. Add a name to your export.
packages/manager/src/features/IAM/Roles/RolesTable/RolesTable.tsx|73 col 5| React Hook React.useMemo has a missing dependency: 'getFilteredRows'. Either include it or remove the dependency array.
packages/manager/src/features/Kubernetes/KubernetesClusterDetail/KubeControlPaneACLDrawer.tsx|106 col 37| Define a constant instead of duplicating this literal 4 times.
packages/manager/src/features/Kubernetes/KubernetesClusterDetail/NodePoolsDisplay/NodeTable.test.tsx|111 col 13| Avoid destructuring queries from render result, use screen.findByText instead
packages/manager/src/features/Kubernetes/kubeUtils.test.ts|93 col 9| Define a constant instead of duplicating this literal 3 times.
packages/manager/src/features/Kubernetes/kubeUtils.test.ts|270 col 18| Define a constant instead of duplicating this literal 5 times.
packages/manager/src/features/Kubernetes/kubeUtils.test.ts|271 col 18| Define a constant instead of duplicating this literal 6 times.
packages/manager/src/features/Kubernetes/kubeUtils.test.ts|272 col 18| Define a constant instead of duplicating this literal 6 times.
packages/manager/src/features/Kubernetes/kubeUtils.test.ts|273 col 18| Define a constant instead of duplicating this literal 9 times.
packages/manager/src/features/Linodes/LinodeEntityDetail.test.tsx|322 col 14| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/Linodes/LinodeEntityDetail.test.tsx|323 col 7| Avoid using multiple assertions within waitFor callback
packages/manager/src/features/Linodes/LinodeEntityDetail.test.tsx|323 col 14| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/Linodes/LinodeEntityDetail.test.tsx|324 col 7| Avoid using multiple assertions within waitFor callback
packages/manager/src/features/Linodes/LinodeEntityDetail.test.tsx|324 col 14| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/Linodes/LinodesDetail/LinodeSettings/AlertSection.tsx|58 col 23| Use of theme.spacing() method is deprecated. Use theme.spacingFunction instead. See: https://linode.github.io/manager/development-guide/16-design-tokens.html#spacing
packages/manager/src/features/Longview/LongviewLanding/LongviewClientRow.tsx|208 col 16| Fast refresh can't handle anonymous components. Add a name to your export.
packages/manager/src/features/Longview/LongviewLanding/LongviewClients.tsx|296 col 16| Fast refresh can't handle anonymous components. Add a name to your export.
packages/manager/src/features/NodeBalancers/NodeBalancerCreate.test.tsx|53 col 12| Avoid destructuring queries from render result, use screen.getByLabelText instead
packages/manager/src/features/NodeBalancers/NodeBalancerCreate.tsx|309 col 58| Refactor this code to not nest functions more than 4 levels deep.
packages/manager/src/features/NodeBalancers/NodeBalancerCreate.tsx|534 col 42| Parameter name subnet_id must match one of the following formats: camelCase
packages/manager/src/features/NodeBalancers/NodeBalancerDetail/NodeBalancerSummary/SummaryPanel.test.tsx|21 col 30| Use null instead.
packages/manager/src/features/NodeBalancers/NodeBalancerDetail/NodeBalancerSummary/SummaryPanel.test.tsx|118 col 12| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/NodeBalancers/NodeBalancerDetail/NodeBalancerSummary/SummaryPanel.test.tsx|119 col 12| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/NodeBalancers/NodeBalancerDetail/NodeBalancerSummary/SummaryPanel.test.tsx|120 col 12| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/NodeBalancers/VPCPanel.test.tsx|27 col 45| Define a constant instead of duplicating this literal 4 times.
packages/manager/src/features/NodeBalancers/VPCPanel.test.tsx|41 col 14| Define a constant instead of duplicating this literal 4 times.
packages/manager/src/features/NodeBalancers/VPCPanel.test.tsx|68 col 56| Define a constant instead of duplicating this literal 3 times.
packages/manager/src/features/Support/SupportTickets/SupportTicketDialog.tsx|142 col 50| Unexpected any. Specify a different type.
packages/manager/src/features/Support/SupportTickets/SupportTicketDialog.tsx|387 col 34| Refactor this code to not nest functions more than 4 levels deep.
packages/manager/src/features/TopMenu/UserMenu/UserMenuPopover.tsx|132 col 5| React Hook React.useMemo has a missing dependency: 'flags.limitsEvolution?.enabled'. Either include it or remove the dependency array.
packages/manager/src/features/VPCs/VPCDetail/SubnetNodebalancerRow.test.tsx|55 col 12| Avoid destructuring queries from render result, use screen.getByTestId instead
packages/manager/src/features/VPCs/VPCDetail/SubnetNodebalancerRow.test.tsx|55 col 12| Don't wrap getBy* query with expect & presence matchers like toBeInTheDocument or not.toBeNull as getBy* queries fail implicitly when element is not found
packages/manager/src/features/VPCs/VPCDetail/SubnetNodebalancerRow.test.tsx|56 col 43| Prefer using queryBy when waiting for disappearance
packages/manager/src/features/VPCs/VPCDetail/SubnetNodebalancerRow.test.tsx|56 col 43| Avoid destructuring queries from render result, use screen.getByTestId instead
packages/manager/src/features/VPCs/VPCDetail/SubnetNodebalancerRow.test.tsx|82 col 14| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCDetail/SubnetNodebalancerRow.test.tsx|82 col 14| Don't wrap getBy* query with expect & presence matchers like toBeInTheDocument or not.toBeNull as getBy* queries fail implicitly when element is not found
packages/manager/src/features/VPCs/VPCDetail/SubnetNodebalancerRow.test.tsx|85 col 12| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCDetail/SubnetNodebalancerRow.test.tsx|85 col 12| Don't wrap getBy* query with expect & presence matchers like toBeInTheDocument or not.toBeNull as getBy* queries fail implicitly when element is not found
packages/manager/src/features/VPCs/VPCDetail/SubnetNodebalancerRow.test.tsx|86 col 12| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCDetail/SubnetNodebalancerRow.test.tsx|86 col 12| Don't wrap getBy* query with expect & presence matchers like toBeInTheDocument or not.toBeNull as getBy* queries fail implicitly when element is not found
packages/manager/src/features/VPCs/VPCDetail/SubnetNodebalancerRow.test.tsx|88 col 30| Avoid destructuring queries from render result, use screen.getByRole instead
packages/manager/src/features/VPCs/VPCDetail/VPCDetail.test.tsx|94 col 26| Avoid destructuring queries from render result, use screen.queryByTestId instead
packages/manager/src/features/VPCs/VPCDetail/VPCDetail.test.tsx|99 col 5| Avoid destructuring queries from render result, use screen.getAllByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCDetail.test.tsx|100 col 5| Avoid destructuring queries from render result, use screen.getAllByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCDetail.test.tsx|101 col 5| Avoid destructuring queries from render result, use screen.getAllByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCDetail.test.tsx|103 col 5| Avoid destructuring queries from render result, use screen.getAllByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCDetail.test.tsx|104 col 5| Avoid destructuring queries from render result, use screen.getAllByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCDetail.test.tsx|106 col 5| Avoid destructuring queries from render result, use screen.getAllByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCDetail.test.tsx|107 col 5| Avoid destructuring queries from render result, use screen.getAllByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCDetail.test.tsx|109 col 5| Avoid destructuring queries from render result, use screen.getAllByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCDetail.test.tsx|110 col 5| Avoid destructuring queries from render result, use screen.getAllByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCDetail.test.tsx|112 col 5| Avoid destructuring queries from render result, use screen.getAllByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCDetail.test.tsx|113 col 5| Avoid destructuring queries from render result, use screen.getAllByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|73 col 5| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|127 col 26| Avoid destructuring queries from render result, use screen.queryByTestId instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|132 col 5| Avoid destructuring queries from render result, use screen.getByPlaceholderText instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|133 col 5| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|134 col 5| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|135 col 5| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|136 col 5| Avoid destructuring queries from render result, use screen.getAllByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|138 col 5| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|139 col 5| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|141 col 5| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|142 col 5| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|144 col 30| Avoid destructuring queries from render result, use screen.getAllByRole instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|147 col 5| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|148 col 5| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|149 col 5| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|150 col 5| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|212 col 5| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|240 col 26| Avoid destructuring queries from render result, use screen.queryByTestId instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|245 col 31| Avoid destructuring queries from render result, use screen.getAllByRole instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|247 col 5| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|270 col 26| Avoid destructuring queries from render result, use screen.queryByTestId instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|275 col 31| Avoid destructuring queries from render result, use screen.getAllByRole instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|278 col 5| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|279 col 5| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCDetail/VPCSubnetsTable.test.tsx|280 col 5| Avoid destructuring queries from render result, use screen.getByText instead
packages/manager/src/features/VPCs/VPCLanding/VPCLanding.test.tsx|64 col 26| Avoid destructuring queries from render result, use screen.queryByTestId instead
packages/manager/src/features/VPCs/VPCLanding/VPCLanding.test.tsx|70 col 5| Avoid destructuring queries from render result, use screen.getAllByText instead
packages/manager/src/features/VPCs/VPCLanding/VPCLanding.test.tsx|71 col 5| Avoid destructuring queries from render result, use screen.getAllByText instead
packages/manager/src/features/VPCs/VPCLanding/VPCLanding.test.tsx|72 col 5| Avoid destructuring queries from render result, use screen.getAllByText instead
packages/manager/src/features/VPCs/VPCLanding/VPCLanding.test.tsx|73 col 5| Avoid destructuring queries from render result, use screen.getAllByText instead
packages/manager/src/features/VPCs/VPCLanding/VPCLanding.test.tsx|74 col 5| Avoid destructuring queries from render result, use screen.getAllByText instead
packages/manager/src/mocks/presets/crud/handlers/quotas.ts|172 col 12| Parameter name quota_id must match one of the following formats: camelCase
packages/manager/src/mocks/presets/crud/handlers/quotas.ts|190 col 12| Parameter name quota_id must match one of the following formats: camelCase
... (Too many findings. Dropped some findings)
Filtered Findings (0)
Annotations
Check warning on line 6 in packages/manager/cypress/e2e/core/account/quotas.spec.ts
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/account/quotas.spec.ts#L6 <sonarjs/todo-tag>(https://sonarsource.github.io/rspec/#/rspec/S1135/javascript)
Complete the task associated to this "TODO" comment.
Raw output
{"ruleId":"sonarjs/todo-tag","severity":1,"message":"Complete the task associated to this \"TODO\" comment.","line":6,"column":8,"nodeType":null,"messageId":"completeTODO","endLine":6,"endColumn":12}
Check warning on line 15 in packages/manager/cypress/e2e/core/account/quotas.spec.ts
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/account/quotas.spec.ts#L15 <sonarjs/no-duplicate-string>(https://sonarsource.github.io/rspec/#/rspec/S1192/javascript)
Define a constant instead of duplicating this literal 6 times.
Raw output
{"ruleId":"sonarjs/no-duplicate-string","severity":1,"message":"Define a constant instead of duplicating this literal 6 times.","line":15,"column":13,"nodeType":"Literal","endLine":15,"endColumn":31}
Check warning on line 19 in packages/manager/cypress/e2e/core/account/quotas.spec.ts
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/account/quotas.spec.ts#L19 <sonarjs/no-duplicate-string>(https://sonarsource.github.io/rspec/#/rspec/S1192/javascript)
Define a constant instead of duplicating this literal 6 times.
Raw output
{"ruleId":"sonarjs/no-duplicate-string","severity":1,"message":"Define a constant instead of duplicating this literal 6 times.","line":19,"column":14,"nodeType":"Literal","endLine":19,"endColumn":26}
Check warning on line 28 in packages/manager/cypress/e2e/core/account/quotas.spec.ts
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/account/quotas.spec.ts#L28 <sonarjs/no-duplicate-string>(https://sonarsource.github.io/rspec/#/rspec/S1192/javascript)
Define a constant instead of duplicating this literal 3 times.
Raw output
{"ruleId":"sonarjs/no-duplicate-string","severity":1,"message":"Define a constant instead of duplicating this literal 3 times.","line":28,"column":14,"nodeType":"Literal","endLine":28,"endColumn":48}
Check warning on line 39 in packages/manager/cypress/e2e/core/account/quotas.spec.ts
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/account/quotas.spec.ts#L39 <sonarjs/no-duplicate-string>(https://sonarsource.github.io/rspec/#/rspec/S1192/javascript)
Define a constant instead of duplicating this literal 4 times.
Raw output
{"ruleId":"sonarjs/no-duplicate-string","severity":1,"message":"Define a constant instead of duplicating this literal 4 times.","line":39,"column":16,"nodeType":"Literal","endLine":39,"endColumn":40}
Check warning on line 168 in packages/manager/cypress/e2e/core/firewalls/create-firewall.spec.ts
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/firewalls/create-firewall.spec.ts#L168 <sonarjs/no-duplicate-string>(https://sonarsource.github.io/rspec/#/rspec/S1192/javascript)
Define a constant instead of duplicating this literal 3 times.
Raw output
{"ruleId":"sonarjs/no-duplicate-string","severity":1,"message":"Define a constant instead of duplicating this literal 3 times.","line":168,"column":15,"nodeType":"Literal","endLine":168,"endColumn":28}
Check warning on line 949 in packages/manager/cypress/e2e/core/kubernetes/lke-create.spec.ts
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/kubernetes/lke-create.spec.ts#L949 <sonarjs/no-nested-functions>(https://sonarsource.github.io/rspec/#/rspec/S2004/javascript)
Refactor this code to not nest functions more than 4 levels deep.
Raw output
{"ruleId":"sonarjs/no-nested-functions","severity":1,"message":"Refactor this code to not nest functions more than 4 levels deep.","line":949,"column":53,"nodeType":null,"endLine":949,"endColumn":55}
Check warning on line 1172 in packages/manager/cypress/e2e/core/kubernetes/lke-create.spec.ts
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/kubernetes/lke-create.spec.ts#L1172 <sonarjs/no-nested-functions>(https://sonarsource.github.io/rspec/#/rspec/S2004/javascript)
Refactor this code to not nest functions more than 4 levels deep.
Raw output
{"ruleId":"sonarjs/no-nested-functions","severity":1,"message":"Refactor this code to not nest functions more than 4 levels deep.","line":1172,"column":53,"nodeType":null,"endLine":1172,"endColumn":55}
Check warning on line 485 in packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts#L485 <sonarjs/no-duplicate-string>(https://sonarsource.github.io/rspec/#/rspec/S1192/javascript)
Define a constant instead of duplicating this literal 3 times.
Raw output
{"ruleId":"sonarjs/no-duplicate-string","severity":1,"message":"Define a constant instead of duplicating this literal 3 times.","line":485,"column":33,"nodeType":"Literal","endLine":485,"endColumn":52}
Check warning on line 866 in packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts#L866 <sonarjs/todo-tag>(https://sonarsource.github.io/rspec/#/rspec/S1135/javascript)
Complete the task associated to this "TODO" comment.
Raw output
{"ruleId":"sonarjs/todo-tag","severity":1,"message":"Complete the task associated to this \"TODO\" comment.","line":866,"column":12,"nodeType":null,"messageId":"completeTODO","endLine":866,"endColumn":16}
Check warning on line 873 in packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts#L873 <sonarjs/todo-tag>(https://sonarsource.github.io/rspec/#/rspec/S1135/javascript)
Complete the task associated to this "TODO" comment.
Raw output
{"ruleId":"sonarjs/todo-tag","severity":1,"message":"Complete the task associated to this \"TODO\" comment.","line":873,"column":12,"nodeType":null,"messageId":"completeTODO","endLine":873,"endColumn":16}
Check warning on line 915 in packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts#L915 <sonarjs/no-nested-functions>(https://sonarsource.github.io/rspec/#/rspec/S2004/javascript)
Refactor this code to not nest functions more than 4 levels deep.
Raw output
{"ruleId":"sonarjs/no-nested-functions","severity":1,"message":"Refactor this code to not nest functions more than 4 levels deep.","line":915,"column":22,"nodeType":null,"endLine":915,"endColumn":24}
Check warning on line 927 in packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts#L927 <sonarjs/no-nested-functions>(https://sonarsource.github.io/rspec/#/rspec/S2004/javascript)
Refactor this code to not nest functions more than 4 levels deep.
Raw output
{"ruleId":"sonarjs/no-nested-functions","severity":1,"message":"Refactor this code to not nest functions more than 4 levels deep.","line":927,"column":22,"nodeType":null,"endLine":927,"endColumn":24}
Check warning on line 954 in packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts#L954 <sonarjs/no-nested-functions>(https://sonarsource.github.io/rspec/#/rspec/S2004/javascript)
Refactor this code to not nest functions more than 4 levels deep.
Raw output
{"ruleId":"sonarjs/no-nested-functions","severity":1,"message":"Refactor this code to not nest functions more than 4 levels deep.","line":954,"column":22,"nodeType":null,"endLine":954,"endColumn":24}
Check warning on line 1032 in packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts#L1032 <sonarjs/no-duplicate-string>(https://sonarsource.github.io/rspec/#/rspec/S1192/javascript)
Define a constant instead of duplicating this literal 3 times.
Raw output
{"ruleId":"sonarjs/no-duplicate-string","severity":1,"message":"Define a constant instead of duplicating this literal 3 times.","line":1032,"column":24,"nodeType":"Literal","endLine":1032,"endColumn":54}
Check warning on line 1034 in packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts#L1034 <sonarjs/no-nested-functions>(https://sonarsource.github.io/rspec/#/rspec/S2004/javascript)
Refactor this code to not nest functions more than 4 levels deep.
Raw output
{"ruleId":"sonarjs/no-nested-functions","severity":1,"message":"Refactor this code to not nest functions more than 4 levels deep.","line":1034,"column":22,"nodeType":null,"endLine":1034,"endColumn":24}
Check warning on line 1063 in packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts#L1063 <sonarjs/no-nested-functions>(https://sonarsource.github.io/rspec/#/rspec/S2004/javascript)
Refactor this code to not nest functions more than 4 levels deep.
Raw output
{"ruleId":"sonarjs/no-nested-functions","severity":1,"message":"Refactor this code to not nest functions more than 4 levels deep.","line":1063,"column":22,"nodeType":null,"endLine":1063,"endColumn":24}
Check warning on line 1122 in packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/linodes/linode-config.spec.ts#L1122 <sonarjs/no-nested-functions>(https://sonarsource.github.io/rspec/#/rspec/S2004/javascript)
Refactor this code to not nest functions more than 4 levels deep.
Raw output
{"ruleId":"sonarjs/no-nested-functions","severity":1,"message":"Refactor this code to not nest functions more than 4 levels deep.","line":1122,"column":22,"nodeType":null,"endLine":1122,"endColumn":24}
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/linodes/upgrade-linode-interface.spec.ts#L43 <sonarjs/todo-tag>(https://sonarsource.github.io/rspec/#/rspec/S1135/javascript)
Complete the task associated to this "TODO" comment.
Raw output
{"ruleId":"sonarjs/todo-tag","severity":1,"message":"Complete the task associated to this \"TODO\" comment.","line":43,"column":8,"nodeType":null,"messageId":"completeTODO","endLine":43,"endColumn":12}
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/linodes/upgrade-linode-interface.spec.ts#L50 <sonarjs/todo-tag>(https://sonarsource.github.io/rspec/#/rspec/S1135/javascript)
Complete the task associated to this "TODO" comment.
Raw output
{"ruleId":"sonarjs/todo-tag","severity":1,"message":"Complete the task associated to this \"TODO\" comment.","line":50,"column":8,"nodeType":null,"messageId":"completeTODO","endLine":50,"endColumn":12}
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/linodes/upgrade-linode-interface.spec.ts#L82 <sonarjs/no-duplicate-string>(https://sonarsource.github.io/rspec/#/rspec/S1192/javascript)
Define a constant instead of duplicating this literal 49 times.
Raw output
{"ruleId":"sonarjs/no-duplicate-string","severity":1,"message":"Define a constant instead of duplicating this literal 49 times.","line":82,"column":7,"nodeType":"Literal","endLine":82,"endColumn":19}
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/linodes/upgrade-linode-interface.spec.ts#L87 <sonarjs/no-duplicate-string>(https://sonarsource.github.io/rspec/#/rspec/S1192/javascript)
Define a constant instead of duplicating this literal 4 times.
Raw output
{"ruleId":"sonarjs/no-duplicate-string","severity":1,"message":"Define a constant instead of duplicating this literal 4 times.","line":87,"column":19,"nodeType":"Literal","endLine":87,"endColumn":42}
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/linodes/upgrade-linode-interface.spec.ts#L98 <sonarjs/no-duplicate-string>(https://sonarsource.github.io/rspec/#/rspec/S1192/javascript)
Define a constant instead of duplicating this literal 14 times.
Raw output
{"ruleId":"sonarjs/no-duplicate-string","severity":1,"message":"Define a constant instead of duplicating this literal 14 times.","line":98,"column":19,"nodeType":"Literal","endLine":98,"endColumn":31}
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/linodes/upgrade-linode-interface.spec.ts#L191 <sonarjs/no-duplicate-string>(https://sonarsource.github.io/rspec/#/rspec/S1192/javascript)
Define a constant instead of duplicating this literal 5 times.
Raw output
{"ruleId":"sonarjs/no-duplicate-string","severity":1,"message":"Define a constant instead of duplicating this literal 5 times.","line":191,"column":20,"nodeType":"Literal","endLine":191,"endColumn":50}
github-actions / eslint
[eslint] packages/manager/cypress/e2e/core/nodebalancers/nodebalancer-create-with-vpc.spec.ts#L84 <sonarjs/no-duplicate-string>(https://sonarsource.github.io/rspec/#/rspec/S1192/javascript)
Define a constant instead of duplicating this literal 6 times.
Raw output
{"ruleId":"sonarjs/no-duplicate-string","severity":1,"message":"Define a constant instead of duplicating this literal 6 times.","line":84,"column":48,"nodeType":"Literal","endLine":84,"endColumn":60}