Commit Graph

10 Commits

Author SHA1 Message Date
Aaron Shafovaloff 2706bca50b restore Karma/QUnit tests and re-enable partitioning
also:
 - include tests with .jsx extension
 - automate partitioning of all QUnit tests
 - move old Ember-related tests back to their prior suffix
 - skip tests either consistently failing locally
   or in Jenkins
   - sometimes due to lack of maintenance (some weren't
     running before), or due to different order
     of running tests
 - fix some tests
 - upgrades karma-selenium-grid-launcher to fork which
   resolves longstanding bug
     cf. https://github.com/squarebracket/karma-selenium-grid-launcher/issues/4

skips three Gradebook-related test files that consistently hang
  in Jenkins, cf. EVAL-4267

Change-Id: Ib79f75963f6c4f5544e5dc1b5c31bc58d6c1110c
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/349561
Reviewed-by: Isaac Moore <isaac.moore@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>
Build-Review: James Butters <jbutters@instructure.com>
2024-06-12 17:06:12 +00:00
Aaron Shafovaloff ff826238a1 Simplify, refactor Karma test config
Also:
 - unified Karma tests
 - include Karma/QUnit specs in ui/
 - removes CoffeeScript+Karma artifacts

Change-Id: Ibfa6c8c297995d6eba046f33d532f94be9c47bd9
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/349655
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Cameron Ray <cameron.ray@instructure.com>
Reviewed-by: Jacob DeWar <jacob.dewar@instructure.com>
Reviewed-by: Isaac Moore <isaac.moore@instructure.com>
QA-Review: Jacob DeWar <jacob.dewar@instructure.com>
Product-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Build-Review: James Butters <jbutters@instructure.com>
2024-06-10 18:31:27 +00:00
Aaron Shafovaloff 08ea6a87ce increase JEST_NODE_COUNT
[skip-crystalball]
[change-merged]
[build-registry-path=jenkins/canvas-lms/shaf-jest-test]

Change-Id: I7e96ea12711f59a87ad2261ffbe5e8a529d7fb07
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/349578
QA-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Product-Review: Aaron Shafovaloff <ashafovaloff@instructure.com>
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Build-Review: James Butters <jbutters@instructure.com>
Reviewed-by: Rohan Chugh <rohan.chugh@instructure.com>
Reviewed-by: Spencer Olson <solson@instructure.com>
2024-06-10 12:42:28 +00:00
Aaron Ogata a6aae57a48 convert remaining boolean flags
refs DE-1359

[canvas-builds-refspec=1f02879d6f1b3a3ca631b542539aa0b699e64146]

Change-Id: Id0b5e194f7e7d1ba55723fcbb5c0dcede028c5de
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/313490
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Andrea Cirulli <andrea.cirulli@instructure.com>
QA-Review: Aaron Ogata <aogata@instructure.com>
Product-Review: Aaron Ogata <aogata@instructure.com>
Build-Review: Aaron Ogata <aogata@instructure.com>
2023-03-16 18:30:43 +00:00
Bobby Buten 6c56e38559 add jest coverage in js coverage report
refs DE-989
flag=none

TEST PLAN:
  confirm https://code-coverage.inseng.net/ is updated with new report

Previous versions of the JS coverage report we not including jest
tests, so this tweak adds them

Change-Id: Ic780ab9df0ab97eaf3d49cda5c6dbc8cebbb7607
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/288044
Reviewed-by: James Butters <jbutters@instructure.com>
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
QA-Review: Bobby Buten <bobby.buten@instructure.com>
Product-Review: Bobby Buten <bobby.buten@instructure.com>
2022-03-29 19:50:14 +00:00
Brian Watson f45ae395dc Dont update skipped specs when manually triggered by gerrit
flag=none

Test-plan:
- run manually via gerrit commit, should skip skipped spec updater

Change-Id: I7fda48ee1e711ca48d53de2dfa212cc0e33ab052
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/287858
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
QA-Review: Brian Watson <bwatson@instructure.com>
Product-Review: Brian Watson <bwatson@instructure.com>
Reviewed-by: James Butters <jbutters@instructure.com>
2022-03-24 15:02:38 +00:00
Bobby Buten a3e27bddac update js coverage with optimized jest builds
refs DE-989
flag=none

TEST PLAN:
  Confirm build still runs and coverage report is created

Change-Id: I800bf3a722b7bef13d2c7a42106827ce5f296d5d
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/287828
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: James Butters <jbutters@instructure.com>
QA-Review: Bobby Buten <bobby.buten@instructure.com>
Product-Review: Bobby Buten <bobby.buten@instructure.com>
2022-03-23 19:43:58 +00:00
Brian Watson cae55d4611 Identify JS and Ruby skipped specs
flag=none
Test-plan:
- manually run ruby coverage / js coverage / axe builds
- ensure artifacts are updated in s3

Change-Id: I2bd4e51eedecf5a10e5d96bdc8727abb17bac034
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/286063
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Kyle Rosenbaum <krosenbaum@instructure.com>
Reviewed-by: James Butters <jbutters@instructure.com>
QA-Review: Brian Watson <bwatson@instructure.com>
Product-Review: Brian Watson <bwatson@instructure.com>
2022-03-18 16:57:43 +00:00
Bobby Buten 64faa28bd5 allow canvas javascript coverage report to run outside of main build
refs DE-989
flag=none

TEST PLAN:
  canvas js report runs independently

Change-Id: Ifb24f8883c88a0742d1bb7d61aa8dc9e97503c79
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/286702
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Aaron Ogata <aogata@instructure.com>
QA-Review: Bobby Buten <bobby.buten@instructure.com>
Product-Review: Bobby Buten <bobby.buten@instructure.com>
2022-03-10 14:52:42 +00:00
Bobby Buten e91a88762d re-enable canvas javascript coverage report
refs DE-989
flag=none

TEST PLAN:
  confirm https://code-coverage.inseng.net/ is updated with new report

Change-Id: I596ce0df04c920b096a875004bd765979dec344b
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/285861
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Aaron Ogata <aogata@instructure.com>
QA-Review: Bobby Buten <bobby.buten@instructure.com>
Product-Review: Bobby Buten <bobby.buten@instructure.com>
2022-02-28 15:36:57 +00:00