Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

chore: updating mocks for tests to test mender-connect JWT request handling #2108

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

chore: updating mocks for tests to test mender-connect JWT request handling #2108

wants to merge 1 commit into from

Commits on May 26, 2024

  1. chore: updating tests for new way of requesting JWT tokens in mender-… 

    …connect

This PR is made to change the mocks and testflow in tests to better fit the new implementation of the PR 👉 mendersoftware/mender-connect#134. This PR fixes it so `mender-connect` requests a new JWT token on its own, so its not dependent on `mender-update` to do so. Previously `FetchJWTToken()` wasn’t called which wouldn’t let it call the JWT token independently, which made it dependent on `mender-update`. The proposed fix is using `FetchJWTToken()` to fetch the token, to then for `WaitForJwtTokenStateChange()` to retrieve it in it’s waiting. The old mocks and test flow for the tests seemed to be more tailored to test the functionality with `GetJWTToken()`. This PR proposes a change in the mocks and test flow to better test the new implementation.

Changelog: None

Ticket: MEN-6877
Signed-off-by: Sebastian Opsahl <[email protected]>
    @SebOpsahl
    SebOpsahl committed May 26, 2024
    Configuration menu
    Copy the full SHA
    5869ea2 View commit details
    Browse the repository at this point in the history