Skip to content

Conversation

@jimblanc
Copy link
Contributor

Description of changes

Adds a basic unit test to exercise custom AnalyticsProvider functionality.

Issue #, if available

Description of how you validated changes

Local run.

Checklist

  • PR description included
  • yarn test passes
  • Tests are changed or added
  • Relevant documentation is changed or added (and PR referenced)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@jimblanc jimblanc marked this pull request as ready for review July 12, 2023 22:24
@jimblanc jimblanc requested a review from a team as a code owner July 12, 2023 22:24
kvramyasri7
kvramyasri7 previously approved these changes Jul 12, 2023
Copy link
Contributor

@kvramyasri7 kvramyasri7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks @jimblanc

cshfang
cshfang previously approved these changes Jul 12, 2023
Co-authored-by: Chris F <5827964+cshfang@users.noreply.github.com>
@jimblanc jimblanc dismissed stale reviews from cshfang and kvramyasri7 via 75da4cc July 12, 2023 22:53
Co-authored-by: Chris F <5827964+cshfang@users.noreply.github.com>
@codecov-commenter
Copy link

Codecov Report

Merging #11618 (908391d) into main (28bc873) will increase coverage by 0.01%.
The diff coverage is n/a.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##             main   #11618      +/-   ##
==========================================
+ Coverage   83.67%   83.69%   +0.01%     
==========================================
  Files         340      340              
  Lines       21549    21549              
  Branches     4604     4604              
==========================================
+ Hits        18032    18036       +4     
+ Misses       3230     3226       -4     
  Partials      287      287              

see 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@jimblanc jimblanc merged commit 7d58055 into aws-amplify:main Jul 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants