kat-kan / kawqa-gad-playwright

4 stars 1 forks source link

Explicitly state user type in all invocations of createHeaders function #138

Open Slawomir-DKl opened 2 weeks ago

Slawomir-DKl commented 2 weeks ago

✏️ Description

According to decision 2024-11-18_1, add regular user type to all createHeaders() invocations in the tests (without parameter)

🔧 Why We Need This

Now only the admin user is stated explicitly. In one of the PRs also custom user is introduced. Explicit stating of regular user will help new contributors to comprehend the tests. Additionally, if we decide to replace the regular user with a custom user in the future, it will be easier to do so.

🚀 Proposed Solution

It would be best to wait for merging the PR #128 introducing new enum for user type

✅ Issue reporting checklist