• 
      

    Update to Jasmine 3.8.0

    Review Request #12967 — Created April 21, 2023 and submitted — Latest diff uploaded

    Information

    Review Board
    release-6.x

    Reviewers

    I want to use Spy.calls.thisFor() in an upcoming change, but that
    wasn't introduced until Jasmine 3.8.0. This change updates us to use
    that.

    I spent a while trying to get it so I could import Jasmine entirely from
    node_modules the way we do for most of our 3rdparty libs, but I
    haven't been successful on that yet, so this just updates the bundled
    copy.

    This also exposed a case where we weren't properly using
    await expectAsync, which meant that some expect calls were happening
    after the test case completed.

    Ran js-tests.

    Commits

    Files