feature/fix-macos-cypress-and-align-with-create-vite #2898
No reviewers
Labels
No Label
area/internal-code
changes requested
confirmed
dependencies
duplicate
good first issue
help wanted
hosting
invalid
kind/bug
kind/feature
question
wontfix
No Milestone
No project
No Assignees
3 Participants
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: vikunja/frontend#2898
Loading…
Reference in New Issue
No description provided.
Delete Branch "dpschen/frontend:feature/fix-macos-cypress-and-align-with-create-vite"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
create-vue
template and improve cypress integrationHi dpschen!
Thank you for creating a PR!
I've deployed the changes of this PR on a preview environment under this URL: https://2898-feature-fix-macos-cypress-and-al--vikunja-frontend-preview.netlify.app
You can use this url to view the changes live and test them out.
You will need to manually connect this to an api running somehwere. The easiest to use is https://try.vikunja.io/.
Have a nice day!
@ -15,1 +14,3 @@
specPattern: 'cypress/e2e/**/*.{js,jsx,ts,tsx}',
specPattern: 'cypress/e2e/**/*.{cy,spec}.{js,jsx,ts,tsx}',
baseUrl: 'http://127.0.0.1:4173',
experimentalRunAllSpecs: true,
What does this do?
Cyress removed the "run all" button in the UI in cypress 12 (i think). This readds it
@ -11,3 +10,1 @@
function seedTasks(numberOfTasks = 100, startDueDate = new Date()) {
UserFactory.create(1)
function seedTasks(numberOfTasks = 50, startDueDate = new Date()) {
I think the changes to this function have broken the tests: https://cloud.cypress.io/projects/181c7x/runs/3562/overview
will check
See https://kolaente.dev/vikunja/frontend/pulls/2898/files#issuecomment-41985
150e03b65a
toa04272846a
@ -45,3 +45,3 @@
cy.get('[data-cy="showTasks"] .card .task')
.each(([task], index) => {
expect(task.innerText).to.contain(tasks[index].title)
expect(task.innerText).to.contain(tasks[taskCount - 1 - index].title)
Somehow the order of the tasks has turned around. I think the reason is in line 20. But I have no clue why. All of those tasks should have 100% the same
due_date
. So I'm also do not understand why we expect the order to be like we do in the first place.The line you picked is correct. Only now this is
Before the last due date was increased by 2, now it's always the value of
now
. Hence all tasks get the same due date.I don't get where in the loop dueDate gets modified.
Regardless I replaced
now
withdueDate
and it works now as intended.Inside of
setDate
. It gets saved back to the variable in the same line.Okay… took me a bit :) 👍
a04272846a
tod850d5b98f