Skip to content
This repository was archived by the owner on Sep 25, 2026. It is now read-only.
This repository was archived by the owner on Sep 25, 2026. It is now read-only.

test: add storage.notification.* conformance tests #1381

Closed

Description

@BenWhitehead

Now that we have notifications integrated into the api, we should ensure our retries are following the prescribed best practice.

  • Add setup steps to RpcMethodMapping
  • Add mappings into RpcMethodMappings
  • Update StorageRetryStrategy and associated classes to account for notification operations
  • Use Retrying#run instead of runWithRetries

Activity

  1. added
    type: processA process-related concern. May include testing, release, or the like.
    api: storageIssues related to the googleapis/java-storage API.
    on May 10, 2022
  2. BenWhitehead commented on May 25, 2022

    @BenWhitehead
    CollaboratorAuthor

    Related test failure #1412

  3. BenWhitehead commented on Jan 31, 2023

    @BenWhitehead
    CollaboratorAuthor

    Fixed in #1734

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    api: storageIssues related to the googleapis/java-storage API.type: processA process-related concern. May include testing, release, or the like.

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions