Skip to content

Commit

Permalink
obey linter
Browse files Browse the repository at this point in the history
  • Loading branch information
pauldambra committed Dec 26, 2024
1 parent 5ba0e6a commit 9826fe3
Show file tree
Hide file tree
Showing 13 changed files with 10 additions and 16 deletions.
2 changes: 0 additions & 2 deletions cypress/e2e/dashboard-deletion.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,6 @@ import { urls } from 'scenes/urls'
import { dashboard, dashboards, insight, savedInsights } from '../productAnalytics'
import { randomString } from '../support/random'

// cypress does have three args, but jest/valid-describe-callback doesn't know

describe(
'deleting dashboards',
{
Expand Down
2 changes: 0 additions & 2 deletions cypress/e2e/dashboard-duplication.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,6 @@ import { urls } from 'scenes/urls'
import { dashboard, dashboards, duplicateDashboardFromMenu, savedInsights } from '../productAnalytics'
import { randomString } from '../support/random'

// cypress does have three args, but jest/valid-describe-callback doesn't know

describe(
'duplicating dashboards',
{
Expand Down
2 changes: 1 addition & 1 deletion cypress/e2e/dashboard-shared.cy.ts
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
import { dashboards } from '../productAnalytics'

// cypress does have three args, but jest/valid-describe-callback doesn't know
// eslint-disable-next-line jest/valid-describe-callback
describe(
// eslint-disable-next-line jest/valid-describe-callback
'Shared dashboard',
{
// default is 4000
Expand Down
2 changes: 1 addition & 1 deletion cypress/e2e/dashboard.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@ import { dashboard, dashboards, insight } from '../productAnalytics'
import { randomString } from '../support/random'

// cypress does have three args, but jest/valid-describe-callback doesn't know
// eslint-disable-next-line jest/valid-describe-callback
describe(
// eslint-disable-next-line jest/valid-describe-callback
'Dashboard',
{
// default is 4000
Expand Down
2 changes: 1 addition & 1 deletion cypress/e2e/insights-date-picker.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@ import 'cypress-network-idle'
import { urls } from 'scenes/urls'

// cypress does have three args, but jest/valid-describe-callback doesn't know
// eslint-disable-next-line jest/valid-describe-callback
describe(
// eslint-disable-next-line jest/valid-describe-callback
'insights date picker',
{
// default is 4000
Expand Down
2 changes: 1 addition & 1 deletion cypress/e2e/insights-duplication.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,8 @@ import { randomString } from '../support/random'

// For tests related to trends please check trendsElements.js
// cypress does have three args, but jest/valid-describe-callback doesn't know
// eslint-disable-next-line jest/valid-describe-callback
describe(
// eslint-disable-next-line jest/valid-describe-callback
'Insights',
{
// default is 4000
Expand Down
2 changes: 1 addition & 1 deletion cypress/e2e/insights-navigation-open-directly.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@ const hogQLQuery = `select event,

// For tests related to trends please check trendsElements.js
// cypress does have three args, but jest/valid-describe-callback doesn't know
// eslint-disable-next-line jest/valid-describe-callback
describe(
// eslint-disable-next-line jest/valid-describe-callback
'Insights',
{
// default is 4000
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@ const hogQLQuery = `select event,

// For tests related to trends please check trendsElements.js
// cypress does have three args, but jest/valid-describe-callback doesn't know
// eslint-disable-next-line jest/valid-describe-callback
describe(
// eslint-disable-next-line jest/valid-describe-callback
'Insights',
{
// default is 4000
Expand Down
2 changes: 1 addition & 1 deletion cypress/e2e/insights-navigation.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,8 @@ const hogQLQuery = `select event,

// For tests related to trends please check trendsElements.js
// cypress does have three args, but jest/valid-describe-callback doesn't know
// eslint-disable-next-line jest/valid-describe-callback
describe(
// eslint-disable-next-line jest/valid-describe-callback
'Insights',
{
// default is 4000
Expand Down
2 changes: 0 additions & 2 deletions cypress/e2e/insights-reload-query.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
// cypress does have three args, but jest/valid-describe-callback doesn't know

describe(
'ReloadInsight component',
{
Expand Down
2 changes: 1 addition & 1 deletion cypress/e2e/insights-saved.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,8 @@ import { createInsight } from '../productAnalytics'
// For tests related to trends please check trendsElements.js
// insight tests were split up because Cypress was struggling with this many tests in one file 🙈
// cypress does have three args, but jest/valid-describe-callback doesn't know
// eslint-disable-next-line jest/valid-describe-callback
describe(
// eslint-disable-next-line jest/valid-describe-callback
'Insights - saved',
{
// default is 4000
Expand Down
2 changes: 1 addition & 1 deletion cypress/e2e/insights-unsaved-confirmation.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@ import { randomString } from '../support/random'

// For tests related to trends please check trendsElements.js
// cypress does have three args, but jest/valid-describe-callback doesn't know
// eslint-disable-next-line jest/valid-describe-callback
describe(
// eslint-disable-next-line jest/valid-describe-callback
'Insights',
{
// default is 4000
Expand Down
2 changes: 1 addition & 1 deletion cypress/e2e/insights.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@ import { randomString } from '../support/random'
// For tests related to trends please check trendsElements.js
// insight tests were split up because Cypress was struggling with this many tests in one file🙈
// cypress does have three args, but jest/valid-describe-callback doesn't know
// eslint-disable-next-line jest/valid-describe-callback
describe(
// eslint-disable-next-line jest/valid-describe-callback
'Insights',
{
// default is 4000
Expand Down

0 comments on commit 9826fe3

Please sign in to comment.