ami/tl-vikunja
1
0
Fork 0
Code

fix: tests

Browse Source
This commit is contained in:
kolaente 2023-06-12 16:35:47 +02:00
parent cd8e497b24
commit 69e94e58c4
No known key found for this signature in database
GPG Key ID: F40E70337AB24C9B
2 changed files with 14 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
src/helpers/parseSubtasksViaIndention.test.ts
View File

@ -1,5 +1,6 @@
import {describe, it, expect} from 'vitest' import {describe, expect, it} from 'vitest'
import {parseSubtasksViaIndention} from '@/helpers/parseSubtasksViaIndention' import {parseSubtasksViaIndention} from '@/helpers/parseSubtasksViaIndention'
import {PrefixMode} from '@/modules/parseTaskText'
describe('Parse Subtasks via Relation', () => { describe('Parse Subtasks via Relation', () => {
it('Should not return a parent for a single task', () => { it('Should not return a parent for a single task', () => {
@ -10,7 +11,7 @@ describe('Parse Subtasks via Relation', () => {
}) })
it('Should not return a parent for multiple tasks without indention', () => { it('Should not return a parent for multiple tasks without indention', () => {
const tasks = parseSubtasksViaIndention(`task one const tasks = parseSubtasksViaIndention(`task one
task two`) task two`, PrefixMode.Default)
expect(tasks).to.have.length(2) expect(tasks).to.have.length(2)
expect(tasks[0].parent).toBeNull() expect(tasks[0].parent).toBeNull()
@ -18,7 +19,7 @@ task two`)
}) })
it('Should return a parent for two tasks with indention', () => { it('Should return a parent for two tasks with indention', () => {
const tasks = parseSubtasksViaIndention(`parent task const tasks = parseSubtasksViaIndention(`parent task
sub task`) sub task`, PrefixMode.Default)
expect(tasks).to.have.length(2) expect(tasks).to.have.length(2)
expect(tasks[0].parent).toBeNull() expect(tasks[0].parent).toBeNull()
@ -29,7 +30,7 @@ task two`)
it('Should return a parent for multiple subtasks', () => { it('Should return a parent for multiple subtasks', () => {
const tasks = parseSubtasksViaIndention(`parent task const tasks = parseSubtasksViaIndention(`parent task
sub task one sub task one
sub task two`) sub task two`, PrefixMode.Default)
expect(tasks).to.have.length(3) expect(tasks).to.have.length(3)
expect(tasks[0].parent).toBeNull() expect(tasks[0].parent).toBeNull()
@ -42,7 +43,7 @@ task two`)
it('Should work with multiple indention levels', () => { it('Should work with multiple indention levels', () => {
const tasks = parseSubtasksViaIndention(`parent task const tasks = parseSubtasksViaIndention(`parent task
sub task sub task
sub sub task`) sub sub task`, PrefixMode.Default)
expect(tasks).to.have.length(3) expect(tasks).to.have.length(3)
expect(tasks[0].parent).toBeNull() expect(tasks[0].parent).toBeNull()
@ -56,7 +57,7 @@ task two`)
const tasks = parseSubtasksViaIndention(`parent task const tasks = parseSubtasksViaIndention(`parent task
sub task sub task
sub sub task one sub sub task one
sub sub task two`) sub sub task two`, PrefixMode.Default)
expect(tasks).to.have.length(4) expect(tasks).to.have.length(4)
expect(tasks[0].parent).toBeNull() expect(tasks[0].parent).toBeNull()
@ -73,7 +74,7 @@ task two`)
sub task sub task
sub sub task one sub sub task one
sub sub sub task sub sub sub task
sub sub task two`) sub sub task two`, PrefixMode.Default)
expect(tasks).to.have.length(5) expect(tasks).to.have.length(5)
expect(tasks[0].parent).toBeNull() expect(tasks[0].parent).toBeNull()
@ -90,7 +91,7 @@ task two`)
it('Should return a parent for multiple subtasks with special stuff', () => { it('Should return a parent for multiple subtasks with special stuff', () => {
const tasks = parseSubtasksViaIndention(`* parent task const tasks = parseSubtasksViaIndention(`* parent task
* sub task one * sub task one
sub task two`) sub task two`, PrefixMode.Default)
expect(tasks).to.have.length(3) expect(tasks).to.have.length(3)
expect(tasks[0].parent).toBeNull() expect(tasks[0].parent).toBeNull()
@ -101,7 +102,7 @@ task two`)
expect(tasks[2].parent).to.eq('parent task') expect(tasks[2].parent).to.eq('parent task')
}) })
it('Should not break when the first line is indented', () => { it('Should not break when the first line is indented', () => {
const tasks = parseSubtasksViaIndention(' single task') const tasks = parseSubtasksViaIndention(' single task', PrefixMode.Default)
expect(tasks).to.have.length(1) expect(tasks).to.have.length(1)
expect(tasks[0].parent).toBeNull() expect(tasks[0].parent).toBeNull()
@ -110,7 +111,7 @@ task two`)
const tasks = parseSubtasksViaIndention( const tasks = parseSubtasksViaIndention(
`parent task +list `parent task +list
sub task 1 sub task 1
sub task 2`) sub task 2`, PrefixMode.Default)
expect(tasks).to.have.length(3) expect(tasks).to.have.length(3)
expect(tasks[0].project).to.eq('list') expect(tasks[0].project).to.eq('list')

6
src/modules/parseTaskText.ts
View File

@ -140,11 +140,11 @@ export const getProjectFromPrefix = (text: string, prefixMode: PrefixMode): stri
} }
export const getLabelsFromPrefix = (text: string, prefixMode: PrefixMode): string[] | null => { export const getLabelsFromPrefix = (text: string, prefixMode: PrefixMode): string[] | null => {
const projectPrefix = PREFIXES[prefixMode]?.project const labelsPrefix = PREFIXES[prefixMode]?.label
if(typeof projectPrefix === 'undefined') { if(typeof labelsPrefix === 'undefined') {
return null return null
} }
return getItemsFromPrefix(text, projectPrefix) return getItemsFromPrefix(text, labelsPrefix)
} }
const getPriority = (text: string, prefix: string): number | null => { const getPriority = (text: string, prefix: string): number | null => {