fix(curriculum): Tests force using {} around statements in if-else (#53012)

This commit is contained in:
weilirs 2024-01-08 02:25:58 -05:00 committed by GitHub
parent 99266b388a
commit 2a5a834256
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 5 additions and 5 deletions

View File

@ -36,7 +36,7 @@ assert.match(attack.toString(), /else\s*if\s*\(\s*monsterHealth\s*<=\s*0\s*\)/);
Your `else if` statement should call the `defeatMonster` function.
```js
assert.match(attack.toString(), /else\s*if\s*\(\s*monsterHealth\s*<=\s*0\s*\)\s*\{\s*defeatMonster/);
assert.match(attack.toString(), /else\s*if\s*\(\s*monsterHealth\s*<=\s*0\s*\)\s*(\{\s*)?defeatMonster(\s*\})?/);
```
# --seed--

View File

@ -41,7 +41,7 @@ You should call the `winGame` function in your `if` block.
```js
const block = attack.toString().split("else if");
assert.match(block[1], /if\s*\(\s*fighting\s*===\s*2\s*\)\s*\{\s*winGame\(\)/);
assert.match(block[1], /if\s*\(\s*fighting\s*===\s*2\s*\)\s*(\{\s*)?winGame\(\)(\s*\})?/);
```
You should add an `else` block.
@ -55,7 +55,7 @@ Your `else` block should call the `defeatMonster` function.
```js
const block = attack.toString().split("else if");
assert.match(block[1], /else\s*\{\s*defeatMonster\(\)/);
assert.match(block[1], /else\s*(\{\s*)?defeatMonster\(\)(\s*\})?/);
```
# --seed--

View File

@ -43,13 +43,13 @@ assert.match(code, /for\s*\(\s*(let|var)\s+i\s*=\s*0\s*;\s*i\s*<\s*strArray\.len
Your `for` loop should see if `strArray[i]` is found in `["+", "-", " "]`.
```js
assert.match(code, /for\s*\(\s*(let|var)\s+i\s*=\s*0\s*;\s*i\s*<\s*strArray\.length\s*;\s*i\s*\+\+\s*\)\s*\{\s*if\s*\(!\[("|')\+\2\s*,\s*("|')-\3\s*,\s*("|')\s\4\s*\]\.includes\(\s*strArray\[i\]\s*\)\)\s*\{/);
assert.match(code, /for\s*\(\s*(let|var)\s+i\s*=\s*0\s*;\s*i\s*<\s*strArray\.length\s*;\s*i\s*\+\+\s*\)\s*\{\s*if\s*\(!\[("|')\+\2\s*,\s*("|')-\3\s*,\s*("|')\s\4\s*\]\.includes\(\s*strArray\[i\]\s*\)\)(\s*\{)?/);
```
Your `for` loop should `push` `strArray[i]` to `cleanStrArray`.
```js
assert.match(code, /for\s*\(\s*(let|var)\s+i\s*=\s*0\s*;\s*i\s*<\s*strArray\.length\s*;\s*i\s*\+\+\s*\)\s*\{\s*if\s*\(!\[("|')\+\2\s*,\s*("|')-\3\s*,\s*("|')\s\4\s*\]\.includes\(\s*strArray\[i\]\s*\)\)\s*\{\s*cleanStrArray\.push\(\s*strArray\[i\]\s*\)\s*;?\s*\}/);
assert.match(code, /for\s*\(\s*(let|var)\s+i\s*=\s*0\s*;\s*i\s*<\s*strArray\.length\s*;\s*i\s*\+\+\s*\)\s*\{\s*if\s*\(!\[("|')\+\2\s*,\s*("|')-\3\s*,\s*("|')\s\4\s*\]\.includes\(\s*strArray\[i\]\s*\)\)(\s*\{)?\s*cleanStrArray\.push\(\s*strArray\[i\]\s*\)\s*;?\s*(\s*\})?/);
```
# --seed--