From 64871dd9b0f4a5390b4adbae6d77618dde8c7b56 Mon Sep 17 00:00:00 2001 From: MSilva95 Date: Tue, 16 Jun 2020 15:36:18 +0100 Subject: [PATCH] adding comments to the tests --- js/tests/using-filter_test.js | 10 ++++----- js/tests/using-map_test.js | 34 +++++++++++++++++----------- js/tests/using-reduce_test.js | 42 ++++++++++++++++++++--------------- 3 files changed, 50 insertions(+), 36 deletions(-) diff --git a/js/tests/using-filter_test.js b/js/tests/using-filter_test.js index 3afafd00..c74007c4 100644 --- a/js/tests/using-filter_test.js +++ b/js/tests/using-filter_test.js @@ -8,8 +8,8 @@ const check = ({ filterCalls }, eq, a, b) => { return len ? result : false } -//check that the code did use filter properly t(({ eq, ctx }) => +// check that the code did use filter properly check(ctx, eq, filterShortStateName(ctx.arr1), [ 'Alaska', 'Hawaii', @@ -25,8 +25,8 @@ t(({ eq, ctx }) => ]), ) -//check that the code did use filter properly t(({ eq, ctx }) => +// check that the code did use filter properly check(ctx, eq, filterStartVowel(ctx.arr1), [ 'Alabama', 'Alaska', @@ -43,8 +43,8 @@ t(({ eq, ctx }) => ]), ) -//check that the code did use filter properly t(({ eq, ctx }) => +// check that the code did use filter properly check(ctx, eq, filter5Vowels(ctx.arr1), [ 'California', 'Louisiana', @@ -55,8 +55,8 @@ t(({ eq, ctx }) => ]), ) -//check that the code did use filter properly t(({ eq, ctx }) => +// check that the code did use filter properly check(ctx, eq, filter1DistinctVowel(ctx.arr1), [ 'Alabama', 'Alaska', @@ -69,8 +69,8 @@ t(({ eq, ctx }) => ]), ) -//check that the code did use filter properly t(({ eq, ctx }) => +// check that the code did use filter properly check(ctx, eq, multiFilter(ctx.arr2), [ { tag: 'CA', name: 'California', capital: 'Sacramento', region: 'West' }, { tag: 'HI', name: 'Hawaii', capital: 'Honolulu', region: 'West' }, diff --git a/js/tests/using-map_test.js b/js/tests/using-map_test.js index 9539eff5..2bf3871a 100644 --- a/js/tests/using-map_test.js +++ b/js/tests/using-map_test.js @@ -13,11 +13,13 @@ t(({ eq, ctx }) => 'Jackson', 'Utqiagvik', 'Albuquerque', - ]), + ]) ) -//check that the code did use map properly -t(({ eq, ctx }) => eq(ctx.mapCalls[0], ctx.states)) +t(({ eq, ctx }) => + // check that the code did use map properly + eq(ctx.mapCalls[0], ctx.states) +) // upperCasingStates t(({ eq, ctx }) => @@ -31,11 +33,13 @@ t(({ eq, ctx }) => 'Ohio', 'Texas', 'West Virginia', - ]), + ]) ) -//check that the code did use map properly -t(({ eq, ctx }) => eq(ctx.mapCalls.includes(ctx.cities), true)) +t(({ eq, ctx }) => + // check that the code did use map properly + eq(ctx.mapCalls.includes(ctx.cities), true) +) // fahrenheitToCelsius t(({ eq, ctx }) => @@ -47,11 +51,13 @@ t(({ eq, ctx }) => '-13°C', '21°C', '-19°C', - ]), + ]) ) -//check that the code did use map properly -t(({ eq, ctx }) => eq(ctx.mapCalls.includes(ctx.temps), true)) +t(({ eq, ctx }) => + // check that the code did use map properly + eq(ctx.mapCalls.includes(ctx.temps), true) +) // trimTemp t(({ eq, ctx }) => @@ -95,11 +101,13 @@ t(({ eq, ctx }) => region: 'West', temperature: '95°F', }, - ]), + ]) ) -//check that the code did use map properly -t(({ eq, ctx }) => eq(ctx.mapCalls.includes(ctx.states), true)) +t(({ eq, ctx }) => + // check that the code did use map properly + eq(ctx.mapCalls.includes(ctx.states), true) +) // tempForecasts t(({ eq, ctx }) => @@ -113,7 +121,7 @@ t(({ eq, ctx }) => '21°Celsius in Jackson, Mississippi', '-19°Celsius in Utqiagvik, Alaska', '35°Celsius in Albuquerque, New Mexico', - ]), + ]) ) export const setup = () => { diff --git a/js/tests/using-reduce_test.js b/js/tests/using-reduce_test.js index 55834e69..cb286b03 100644 --- a/js/tests/using-reduce_test.js +++ b/js/tests/using-reduce_test.js @@ -2,47 +2,53 @@ export const tests = [] const t = (f) => tests.push(f) t(({ eq }) => eq(adder([1, 2, 3, 4]), 10)) -//check that the code did use reduce properly t(({ eq, ctx }) => - eq(ctx.reduceCalls[ctx.reduceCalls.length - 1], [1, 2, 3, 4]), + // check that the code did use reduce properly + eq(ctx.reduceCalls[ctx.reduceCalls.length - 1], [1, 2, 3, 4]) ) t(({ eq }) => eq(adder([9, 24, 7, 11, 3], 10), 64)) -//check that the code did use reduce properly t(({ eq, ctx }) => - eq(ctx.reduceCalls[ctx.reduceCalls.length - 1], [9, 24, 7, 11, 3]), + // check that the code did use reduce properly + eq(ctx.reduceCalls[ctx.reduceCalls.length - 1], [9, 24, 7, 11, 3]) ) t(({ eq }) => eq(adder([]), 0)) -//check that the code did use reduce properly -t(({ eq, ctx }) => eq(ctx.reduceCalls[ctx.reduceCalls.length - 1], [])) +t(({ eq, ctx }) => + // check that the code did use reduce properly + eq(ctx.reduceCalls[ctx.reduceCalls.length - 1], []) +) t(({ eq }) => eq(sumOrMul([29, 23, 3, 2, 25]), 135)) -//check that the code did use reduce properly t(({ eq, ctx }) => - eq(ctx.reduceCalls[ctx.reduceCalls.length - 1], [29, 23, 3, 2, 25]), + // check that the code did use reduce properly + eq(ctx.reduceCalls[ctx.reduceCalls.length - 1], [29, 23, 3, 2, 25]) ) t(({ eq }) => eq(sumOrMul([18, 17, 7, 13, 25], 12), 278)) -//check that the code did use reduce properly t(({ eq, ctx }) => - eq(ctx.reduceCalls[ctx.reduceCalls.length - 1], [18, 17, 7, 13, 25]), + // check that the code did use reduce properly + eq(ctx.reduceCalls[ctx.reduceCalls.length - 1], [18, 17, 7, 13, 25]) ) t(({ eq }) => eq(sumOrMul([8, 16, 7, 0, 32]), 0)) -//check that the code did use reduce properly t(({ eq, ctx }) => - eq(ctx.reduceCalls[ctx.reduceCalls.length - 1], [8, 16, 7, 0, 32]), + // check that the code did use reduce properly + eq(ctx.reduceCalls[ctx.reduceCalls.length - 1], [8, 16, 7, 0, 32]) ) t(({ eq }) => eq(sumOrMul([8, 16, 7, 0, 31]), 31)) -//check that the code did use reduce properly t(({ eq, ctx }) => - eq(ctx.reduceCalls[ctx.reduceCalls.length - 1], [8, 16, 7, 0, 31]), + // check that the code did use reduce properly + eq(ctx.reduceCalls[ctx.reduceCalls.length - 1], [8, 16, 7, 0, 31]) ) t(({ eq, ctx }) => eq(funcExec(ctx.fArr1, 10), `result: [137]`)) -//check that the code did use reduce properly -t(({ eq, ctx }) => eq(ctx.reduceCalls[ctx.reduceCalls.length - 1], ctx.fArr1)) +t(({ eq, ctx }) => + // check that the code did use reduce properly + eq(ctx.reduceCalls[ctx.reduceCalls.length - 1], ctx.fArr1) +) t(({ eq, ctx }) => eq(funcExec(ctx.fArr2, 4), { result: 72, isOdd: true })) -//check that the code did use reduce properly -t(({ eq, ctx }) => eq(ctx.reduceCalls[ctx.reduceCalls.length - 1], ctx.fArr2)) +t(({ eq, ctx }) => + // check that the code did use reduce properly + eq(ctx.reduceCalls[ctx.reduceCalls.length - 1], ctx.fArr2) +) Object.freeze(tests)