Change Details

In {T406615}, we fixed the hCaptcha QUnit tests. We should now also expand the QUnit tests to cover code that was added but has not been properly tested === Background - In {T406231} we found that the hCaptcha QUnit tests were not being run in MediaWiki CI -- This meant that they did not block merges when they failed and as such several tests for the secure enclave JS code were failing - These failures were a mixture of the tests not being updated and the tests catching problems with the secure enclave handling -- For example, no error was being displayed if the hCaptcha SDK failed to load - We should expand the QUnit tests so that the secure enclave handling is better tested -- This is to make up for recent work that did not expand the QUnit tests

In {T406615}, we fixed the hCaptcha QUnit tests. We should now also expand the QUnit tests to cover code that was added but has not been properly tested === Background - In {T406231} we found that the hCaptcha QUnit tests were not being run in MediaWiki CI -- This meant that they did not block merges when they failed and as such several tests for the secure enclave JS code were failing - These failures were a mixture of the tests not being updated and the tests catching problems with the secure enclave handling -- For example, no error was being displayed if the hCaptcha SDK failed to load - We should expand the QUnit tests so that the secure enclave handling is better tested -- This is to make up for recent work that did not expand the QUnit tests === Acceptance criteria - [ ] Expand QUnit test coverage on secure enclave and related code in hCaptcha #confirmedit

In {T406615}, we fixed the hCaptcha QUnit tests. We should now also expand the QUnit tests to cover code that was added but has not been properly tested === Background - In {T406231} we found that the hCaptcha QUnit tests were not being run in MediaWiki CI -- This meant that they did not block merges when they failed and as such several tests for the secure enclave JS code were failing - These failures were a mixture of the tests not being updated and the tests catching problems with the secure enclave handling -- For example, no error was being displayed if the hCaptcha SDK failed to load - We should expand the QUnit tests so that the secure enclave handling is better tested -- This is to make up for recent work that did not expand the QUnit tests === Acceptance criteria - [ ] Expand QUnit test coverage on secure enclave and related code in hCaptcha #confirmedit