diff --git a/app/Providers/RouteServiceProvider.php b/app/Providers/RouteServiceProvider.php index d7e3c8fa..4ebf1e55 100644 --- a/app/Providers/RouteServiceProvider.php +++ b/app/Providers/RouteServiceProvider.php @@ -27,6 +27,10 @@ class RouteServiceProvider extends ServiceProvider public function boot(): void { RateLimiter::for('api', function (Request $request) { + if (! $this->app->isProduction()) { + return Limit::none(); + } + return Limit::perMinute(60)->by($request->user()?->id ?: $request->ip()); }); diff --git a/e2e/timetracker.spec.ts b/e2e/timetracker.spec.ts index be1fd350..f7efd2ed 100644 --- a/e2e/timetracker.spec.ts +++ b/e2e/timetracker.spec.ts @@ -364,9 +364,7 @@ test('test that adding a new tag works', async ({ page }) => { ); }); await expect(page.getByTestId('tag_dropdown_search')).toHaveValue(''); - await expect(page.getByTestId('tag_dropdown_entries')).toHaveText( - newTagName - ); + await expect(page.getByRole('option', { name: newTagName })).toBeVisible(); }); test('test that adding a new tag when the timer is running', async ({ @@ -389,9 +387,8 @@ test('test that adding a new tag when the timer is running', async ({ ); }); await expect(page.getByTestId('tag_dropdown_search')).toHaveValue(''); - await expect(page.getByTestId('tag_dropdown_entries')).toHaveText( - newTagName - ); + + await expect(page.getByRole('option', { name: newTagName })).toBeVisible(); await page.waitForResponse(async (response) => { return ( diff --git a/package.json b/package.json index 9ecfa5fa..fa9f8876 100644 --- a/package.json +++ b/package.json @@ -7,7 +7,7 @@ "lint": "eslint --ext .js,.vue,.ts --ignore-path .gitignore .", "lint:fix": "eslint --fix --ext .js,.vue,.ts --ignore-path .gitignore .", "type-check": "vue-tsc --noEmit", - "test:e2e": "npx playwright test", + "test:e2e": "rm -rf test-results/.auth && npx playwright test", "generate:zod": "npx openapi-zod-client http://localhost:80/docs/api.json --output openapi.json.client.ts --base-url http://solidtime.test/api" }, "devDependencies": { diff --git a/playwright.config.ts b/playwright.config.ts index 1c552630..838b4745 100644 --- a/playwright.config.ts +++ b/playwright.config.ts @@ -20,14 +20,14 @@ export default defineConfig({ /* Opt out of parallel tests on CI. */ workers: process.env.CI ? 1 : undefined, /* Reporter to use. See https://playwright.dev/docs/test-reporters */ - reporter: process.env.CI ? 'list' : 'html', + reporter: process.env.CI ? 'github' : 'html', /* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */ use: { /* Base URL to use in actions like `await page.goto('/')`. */ // baseURL: 'http://127.0.0.1:3000', /* Collect trace when retrying the failed test. See https://playwright.dev/docs/trace-viewer */ - trace: 'on-first-retry', + trace: 'on', }, /* Configure projects for major browsers */