Add information about number of tests and subtests

This modifies the scores.json file so it now provides more data:
* total_tests: The total number of tests
* total_score: The total scores for the tests
* per_mille: Per mille value of dividing total_score by total_tests
* total_subtests: The total number of subtests
* total_subtests_passed: The number of subtests that pass
* per_mille_subtests: Per mille value of dividing total_subtests by total_subtests_passed

This is based on previous work by Delan Azabani at #35.

Co-authored-by: Delan Azabani <[email protected]>
Signed-off-by: Manuel Rego Casasnovas <[email protected]>

Author: mrego

process-wpt-results.js

@@ -199,7 +199,11 @@ export function score_run (run, against_run, focus_areas_map) {
     for (const area of Object.keys(FOCUS_AREAS)) {
         scores[area] = {
             total_tests: 0,
-            total_score: 0
+            total_score: 0,
+            per_mille: 0,
+            total_subtests: 0,
+            total_subtests_passed: 0,
+            per_mille_subtests: 0
         }
     }
 
@@ -213,33 +217,47 @@ export function score_run (run, against_run, focus_areas_map) {
         const run_test = run.test_scores[test]
 
         // score new tests not present in older runs
-        if (!run_test) continue
+        if (!run_test) {
+          for (const area of areas) {
+              scores[area].total_subtests += 1
+          }
+          continue
+        }
 
         const subtest_names = Object.keys(subtests)
         if (!subtest_names.length) {
             for (const area of areas) {
                 scores[area].total_score += run_test.score
+                scores[area].total_subtests += 1
+                scores[area].total_subtests_passed += run_test.score
             }
         } else {
-            let test_score = 0
+            let subtests_count = 0
+            let subtests_passed = 0
             for (const subtest of subtest_names) {
+                subtests_count += 1
                 if (Object.hasOwn(run_test.subtests, subtest)) {
-                    test_score += run_test.subtests[subtest].score
+                    subtests_passed += run_test.subtests[subtest].score
                 }
             }
-            test_score /= subtest_names.length
+            let test_score = subtests_passed / subtests_count
             for (const area of areas) {
                 scores[area].total_score += test_score
+                scores[area].total_subtests += subtests_count
+                scores[area].total_subtests_passed += subtests_passed 
             }
         }
     }
 
     return Object.entries(scores).reduce((scores, [area, totals]) => {
-        scores[area] = 0
+        scores[area] = totals
         if (totals.total_tests !== 0) {
-            scores[area] = Math.floor(
+            scores[area].per_mille = Math.floor(
                 1000 * totals.total_score / totals.total_tests
             )
+            scores[area].per_mille_subtests = Math.floor(
+                1000 * totals.total_subtests_passed / totals.total_subtests
+            )
         }
         return scores
     }, {})

test/process-wpt-results.test.js

@@ -166,11 +166,109 @@ describe('Scoring', () => {
             test2: ['all']
         }
         let score = score_run(run, run, focus_area_map)
-        assert.deepEqual(score.all, 1000)
+        assert.deepEqual(score.all,
+            {
+                total_tests: 2,
+                total_score: 2,
+                per_mille: 1000,
+                total_subtests: 2,
+                total_subtests_passed: 2,
+                per_mille_subtests: 1000
+            })
 
         run.test_scores.test2.score = 0
         score = score_run(run, run, focus_area_map)
-        assert.deepEqual(score.all, 500)
+        assert.deepEqual(score.all,
+            {
+                total_tests: 2,
+                total_score: 1,
+                per_mille: 500,
+                total_subtests: 2,
+                total_subtests_passed: 1,
+                per_mille_subtests: 500
+            })
+    })
+    it('calculates subtests count', () => {
+        const run = {
+            test_scores: {
+                test1: {
+                    score: 1,
+                    subtests: {
+                        subtest1: { score: 1 },
+                        subtest2: { score: 1 },
+                        subtest3: { score: 1 }
+                    }
+                },
+                test2: {
+                    score: 0,
+                    subtests: {
+                        subtest1: { score: 1 },
+                        subtest2: { score: 0 },
+                    }
+                }
+            }
+        }
+
+        const focus_area_map = {
+            test1: ['all'],
+            test2: ['all']
+        }
+        let score = score_run(run, run, focus_area_map)
+        assert.deepEqual(score.all,
+            {
+                total_tests: 2,
+                total_score: 1.5,
+                per_mille: 750,
+                total_subtests: 5,
+                total_subtests_passed: 4,
+                per_mille_subtests: 800
+            })
+    })
+    it('calculates subtests counts with simple tests', () => {
+        const run = {
+            test_scores: {
+                test1: {
+                    score: 1,
+                    subtests: {
+                        subtest1: { score: 1 },
+                        subtest2: { score: 1 },
+                        subtest3: { score: 1 }
+                    }
+                },
+                test2: {
+                    score: 0,
+                    subtests: {
+                        subtest1: { score: 1 },
+                        subtest2: { score: 0 },
+                    }
+                },
+                test3: {
+                    score: 1,
+                    subtests: {}
+                },
+                test4: {
+                    score: 0,
+                    subtests: {}
+                }
+            }
+        }
+
+        const focus_area_map = {
+            test1: ['all'],
+            test2: ['all'],
+            test3: ['all'],
+            test4: ['all']
+        }
+        let score = score_run(run, run, focus_area_map)
+        assert.deepEqual(score.all,
+            {
+                total_tests: 4,
+                total_score: 2.5,
+                per_mille: 625,
+                total_subtests: 7,
+                total_subtests_passed: 5,
+                per_mille_subtests: 714
+            })
     })
     it('calculates scores for subtests', () => {
         const run = {
@@ -187,7 +285,15 @@ describe('Scoring', () => {
         }
 
         const score = score_run(run, run, { test1: ['all'] })
-        assert.equal(score.all, 1000)
+        assert.deepEqual(score.all,
+            {
+                total_tests: 1,
+                total_score: 1,
+                per_mille: 1000,
+                total_subtests: 3,
+                total_subtests_passed: 3,
+                per_mille_subtests: 1000
+            })
     })
     it('calculates scores for subtests by averaging', () => {
         const run = {
@@ -204,7 +310,15 @@ describe('Scoring', () => {
         }
 
         const score = score_run(run, run, { test1: ['all'] })
-        assert.equal(score.all, 333)
+        assert.deepEqual(score.all,
+            {
+                total_tests: 1,
+                total_score: 1/3,
+                per_mille: 333,
+                total_subtests: 3,
+                total_subtests_passed: 1,
+                per_mille_subtests: 333
+            })
     })
     it('calculates scores correctly even subtest name collides with JS builtins', () => {
         const run = {
@@ -237,7 +351,15 @@ describe('Scoring', () => {
         }
 
         const score = score_run(run, against_run, { test1: ['all'], test2: ['all'] })
-        assert.equal(score.all, 500)
+        assert.deepEqual(score.all,
+            {
+                total_tests: 2,
+                total_score: 1,
+                per_mille: 500,
+                total_subtests: 2,
+                total_subtests_passed: 1,
+                per_mille_subtests: 500
+            })
     })
     it('calculates scores based only on tests in new runs', () => {
         const old_run = {
@@ -258,11 +380,26 @@ describe('Scoring', () => {
             test1: all, test2: all, test3: all
         }
         let score = score_run(old_run, new_run, focus_map)
-        assert.equal(score.all, 0)
-
+        assert.deepEqual(score.all,
+            {
+                total_tests: 2,
+                total_score: 0,
+                per_mille: 0,
+                total_subtests: 2,
+                total_subtests_passed: 0,
+                per_mille_subtests: 0
+            })
         old_run.test_scores.test3.score = 1
         score = score_run(old_run, new_run, focus_map)
-        assert.equal(score.all, 500)
+        assert.deepEqual(score.all,
+            {
+                total_tests: 2,
+                total_score: 1,
+                per_mille: 500,
+                total_subtests: 2,
+                total_subtests_passed: 1,
+                per_mille_subtests: 500
+            })
     })
 })