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.

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

Fix TestTelegramConfirmedRequest #215

Merged
merged 4 commits into from
Aug 31, 2024

Commits on Jun 3, 2024

  1. Fix registration of dev provider in Service.authMiddleware.Providers

    Now it is possible to have a configuration,
    where only one single dev provider is enabled.
    
    Providers were not registered into Service.authMiddleware.Provicers slice
    in the Service.AddDevProvider() and Service.AddAppleProvider() methods before.
    cyb3r4nt committed Jun 3, 2024
    Configuration menu
    Copy the full SHA
    30656d6 View commit details
    Browse the repository at this point in the history

Commits on Jul 17, 2024

  1. Configuration menu
    Copy the full SHA
    ec4b929 View commit details
    Browse the repository at this point in the history

Commits on Aug 2, 2024

  1. Configuration menu
    Copy the full SHA
    ec38494 View commit details
    Browse the repository at this point in the history

Commits on Aug 30, 2024

  1. fix race conditions in TestTelegramConfirmedRequest

    There were two different race conditions between logic in TestTelegramConfirmedRequest
    and TelegramAPIMock.GetUpdatesFunc and TelegramAPIMock.SendFunc:
    * GetUpdatesFunc may start before token was fetched,
    then it produces empty telegramUpdate response, which causes assertions in SendFunc to fail.
    * When token becomes used and removed from wait queue after successful login completion,
    then GetUpdatesFunc may be still called and new telegram update is created for same token.
    This breaks telegram update processing logic,
    and SendFunc gets called with the error parameter, which also breaks assertions.
    cyb3r4nt committed Aug 30, 2024
    Configuration menu
    Copy the full SHA
    523974c View commit details
    Browse the repository at this point in the history