19 Commits

Author SHA1 Message Date
20c9e03daf remove vestiges of sinon and qunit
Change-Id: I77aedaa2a488a91b1a39e7711dc4994b237cbe65
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/381954
Reviewed-by: Aaron Shafovaloff <ashafovaloff@instructure.com>
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Michael Hulse <michael.hulse@instructure.com>
QA-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Product-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
2025-06-20 19:26:27 +00:00
8f4df849b8 remove string refs check from techdebt stats script
Change-Id: I524ace3f85c785687c56dc643d9525953fcb2e9f
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/381955
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Michael Hulse <michael.hulse@instructure.com>
QA-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Product-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
2025-06-20 18:16:33 +00:00
c74d8b5b8e improve techdebt script
Change-Id: I132e1931b4038400e12b6544a80ced738db5bef5
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/381777
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Isaac Moore <isaac.moore@instructure.com>
QA-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Product-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
2025-06-18 23:10:24 +00:00
41200966f1 Replace moxios with msw in various files
Change-Id: Idb23f81b5908d4b8f86fd2f84e983c5de8bd41f0
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/380841
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
QA-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Product-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Reviewed-by: Cameron Ray <cameron.ray@instructure.com>
QA-Review: Cameron Ray <cameron.ray@instructure.com>
Product-Review: Cameron Ray <cameron.ray@instructure.com>
2025-06-07 17:26:33 +00:00
c5a423b0f0 add moxios section to techdebt script; reorder sections
Change-Id: I9b323e7e2d50bc3764ddfa87fad9f23594367dda
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/380839
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Bence Toppanto <bence.toppanto@instructure.com>
QA-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Product-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
2025-06-07 13:54:29 +00:00
09c7f8880e remove enzyme package
Change-Id: I941e653228fcc43bcdebe10e6f4db26b063b3a3a
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/380649
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Chris Soto <christopher.soto@instructure.com>
QA-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Product-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
2025-06-05 22:19:01 +00:00
76697167ba show three example files per section in techdebt_stats
Change-Id: I2b275fbf126dee0972d0eb3a7d8635c720b4e233
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/380610
Reviewed-by: Drake Harper <drake.harper@instructure.com>
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
QA-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Product-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
2025-06-05 17:33:26 +00:00
7dcd177d68 support specifying multiple sections in techdebt script
Change-Id: I569058c49ede70073c0167e17cbc35f502ea5072
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/380293
Reviewed-by: Drake Harper <drake.harper@instructure.com>
QA-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Product-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
2025-06-04 21:32:41 +00:00
333cc62fac Add react-compiler section to Tech Debt Stats
why:
- To make it easier to chip away at these violations and follow our
  progress.

test-plan:
- `node script/techdebt_stats.js --section react-compiler --verbose`
  shows a list of all our violations.
- `node script/techdebt_stats.js` includes a section about how many
  violations we have and an example violation location.

Change-Id: I197a7f335f6e9657c80ffa46c858b0f68a4bc2e4
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/368661
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Michael Hulse <michael.hulse@instructure.com>
QA-Review: Ryan Hawkins <ryan.hawkins@instructure.com>
Product-Review: Ryan Hawkins <ryan.hawkins@instructure.com>
2025-01-28 22:59:30 +00:00
ad3f72ef0d Update techdebt status with section and verbose options
flag none

Test Plan:
- no change if no options are passed
- run node script/techdebt_stats.js -s <section>
- should show only specified section with an example file
- try running with -v with and without section
- should see all files

Change-Id: I860563285cddbffa42c466991617cfac085e02c6
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/367003
Reviewed-by: Aaron Shafovaloff <ashafovaloff@instructure.com>
QA-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Product-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
2025-01-08 21:58:39 +00:00
02e782c7a6 add stats to techdebt_stat script
Change-Id: I1639e5d35dcb889c35191881299195c2ac8e7d5f
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/366769
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Michael Hulse <michael.hulse@instructure.com>
QA-Review: Michael Hulse <michael.hulse@instructure.com>
Product-Review: Michael Hulse <michael.hulse@instructure.com>
2025-01-06 20:20:59 +00:00
d78d21ef73 spec: replace React Test Utils with Testing Library (2)
refs CFA-70

Change-Id: I717d15af2afdd45e40b8fe9d78cba548f397c13c
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/366704
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Michael Hulse <michael.hulse@instructure.com>
QA-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Product-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
2025-01-06 19:26:40 +00:00
e2fcc1dfce add sinon imports to techdebt_stats
Change-Id: I245adbb70934e9e3288460dd36894a6f67317c80
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/366699
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Michael Hulse <michael.hulse@instructure.com>
QA-Review: Michael Hulse <michael.hulse@instructure.com>
Product-Review: Michael Hulse <michael.hulse@instructure.com>
2025-01-04 01:17:44 +00:00
a1b3ac0f62 don't check for QUnit files in tech debt tracker
Change-Id: I2473066364b4184a2a42a0b94db5be770e3b06c8
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/366603
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Chris Soto <christopher.soto@instructure.com>
QA-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Product-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
2025-01-02 22:38:42 +00:00
8bfd1a0b9c add Skipped Tests section to tech debt tracker
Change-Id: I47cefd696c3bcf15ff6dd1c034272226f53e5f4a
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/366561
Reviewed-by: Michael Hulse <michael.hulse@instructure.com>
QA-Review: Michael Hulse <michael.hulse@instructure.com>
Product-Review: Michael Hulse <michael.hulse@instructure.com>
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
2025-01-02 18:53:59 +00:00
03148d9139 include ReactTestUtils and React class components in techdebt stats
Change-Id: If836998c996d878b0f4592d716194c5b4dca578c
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/366349
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Michael Hulse <michael.hulse@instructure.com>
QA-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Product-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
2024-12-22 17:28:59 +00:00
2433ccff47 fix Outdated Packages section in techdebt stats
Change-Id: I303b6756feb487fb3e87d32ee2575125d5d89a1e
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/366337
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Michael Hulse <michael.hulse@instructure.com>
QA-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Product-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
2024-12-22 15:47:39 +00:00
2c0dcff4a0 feat: Add function to count files with ReactDOM.render
Change-Id: I1fe2baf61b967620705df30d4c068d2ddd19e4c1
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/365934
Reviewed-by: Michael Hulse <michael.hulse@instructure.com>
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
QA-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Product-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
2024-12-17 19:59:43 +00:00
6eac295e9a add script to summarize frontend technical debt
test plan:
  run `node script/techdebt_stats.js`

flag=none

Change-Id: I0ad9de979d91a7ac91185d3be4f00aa14dbb408a
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/365793
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Michael Hulse <michael.hulse@instructure.com>
QA-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Product-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
2024-12-17 05:53:20 +00:00