VSinerva/freesewing
1
0
Fork 0
Code Activity

fix: Exclude non-garments from certain pattern tests

Browse source
This commit is contained in:
joostdecock 2021-08-30 19:53:42 +02:00
parent ef3bd03e2c
commit 0ac3358350
1 changed files with 42 additions and 32 deletions
Download patch file Download diff file Expand all files Collapse all files

74
tests/patterns/config.js
View file

@ -1,3 +1,12 @@
const notGarments = [
'rendertest',
'tutorial',
'examples',
'legend',
]
const isGarment = design => (notGarments.indexOf(design) === -1) true : false
/* /*
* This runs unit tests for the pattern configuration * This runs unit tests for the pattern configuration
* It expects the following: * It expects the following:
@ -30,44 +39,45 @@ const testPatternConfig = (design, pattern, expect, models, patterns) => {
} }
}) })
} }
it(` - 'type' should be 'pattern' or 'block'`, () => { // Config tests for garments only
expect(['pattern', 'block'].indexOf(pattern.config.type)).to.not.equal(-1) if (isGarment(design)) {
}) it(` - 'type' should be 'pattern' or 'block'`, () => {
it(` - 'department' should be one of tops, bottoms, coats, swimwear, underwear, or accessories`, () => { expect(['pattern', 'block'].indexOf(pattern.config.type)).to.not.equal(-1)
expect( })
['tops', 'bottoms', 'coats', 'swimwear', 'underwear', 'accessories'].indexOf(pattern.config.department) it(` - 'department' should be one of tops, bottoms, coats, swimwear, underwear, or accessories`, () => {
).to.not.equal(-1) expect(
}) ['tops', 'bottoms', 'coats', 'swimwear', 'underwear', 'accessories'].indexOf(pattern.config.department)
it(` - 'difficulty' should be a number between 1 and 5`, () => { ).to.not.equal(-1)
expect(typeof pattern.config.difficulty).to.equal('number') })
expect(pattern.config.difficulty > 0).to.be.true it(` - 'difficulty' should be a number between 1 and 5`, () => {
expect(pattern.config.difficulty < 6).to.be.true expect(typeof pattern.config.difficulty).to.equal('number')
}) expect(pattern.config.difficulty > 0).to.be.true
expect(pattern.config.difficulty < 6).to.be.true
})
/* /*
* Ensure optiongroup structure and content * Ensure optiongroup structure and content
*/ */
it('Option groups:', () => true) it('Option groups:', () => true)
for (let group in pattern.config.optionGroups) { for (let group in pattern.config.optionGroups) {
for (let option of pattern.config.optionGroups[group]) { for (let option of pattern.config.optionGroups[group]) {
if (typeof option === 'string') { if (typeof option === 'string') {
it(` - '${option}' should be a valid option`, () => { it(` - '${option}' should be a valid option`, () => {
expect(pattern.config.options[option]).to.exist expect(pattern.config.options[option]).to.exist
}) })
} else { } else {
for (let subgroup in option) { for (let subgroup in option) {
it(` Subgroup: ${subgroup}`, () => true) it(` Subgroup: ${subgroup}`, () => true)
for (let suboption of option[subgroup]) { for (let suboption of option[subgroup]) {
it(` - '${suboption}' should be a valid option`, () => { it(` - '${suboption}' should be a valid option`, () => {
expect(pattern.config.options[suboption]).to.exist expect(pattern.config.options[suboption]).to.exist
}) })
}
} }
} }
} }
} }
}
if (['rendertest', 'tutorial', 'examples', 'legend'].indexOf(design) === -1) {
/* /*
* Ensure pattern is listed as being for breasts or not * Ensure pattern is listed as being for breasts or not
*/ */