Skip to content
Snippets Groups Projects

Aws add schema tests

Merged Derek Hudson requested to merge aws-add-schema-tests into master
All threads resolved!

Type of change

  • Bug Fix
  • Feature

Please provide link to gitlab issue or ADR(Architecture Decision Record)

Does this introduce a change in the core logic?

  • [NO]

Does this introduce a change in the cloud provider implementation, if so which cloud?

  • AWS
  • Azure
  • Google Cloud
  • IBM

Does this introduce a breaking change?

  • [NO]

What is the current behavior?

No integration tests for the notifications emitted by Indexer service or Schema service on AWS.

What is the new/expected behavior?

Adds integration tests for the notifications emitted by Indexer service or Schema service on AWS.

Have you added/updated Unit Tests and Integration Tests?

This just adds the integration tests.

Any other useful information

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Derek Hudson
  • Derek Hudson
  • Derek Hudson added 1 commit

    added 1 commit

    Compare with previous version

  • Bruce Jin approved this merge request

    approved this merge request

  • Derek Hudson added 1 commit

    added 1 commit

    • a6415d85 - Added 2 minutes to the Indexer-notification integration tests to allow more...

    Compare with previous version

  • Derek Hudson reset approvals from @jtyself by pushing to the branch

    reset approvals from @jtyself by pushing to the branch

  • Derek Hudson resolved all threads

    resolved all threads

  • Derek Hudson added 1 commit

    added 1 commit

    • ce37a3d1 - Increased the poll time for the Indexer's notification received to 10 minutes.

    Compare with previous version

  • Yong Zeng approved this merge request

    approved this merge request

  • Bruce Jin approved this merge request

    approved this merge request

  • Derek Hudson mentioned in commit ce6285a8

    mentioned in commit ce6285a8

  • merged

  • Please register or sign in to reply
    Loading