Allow for complete messages to be captured in a subscription

[kddejong]

Description of changes

AppSync now supports unsubscribing connections . Previously this message is not available to uses of the amplify-js libraries. This PR will allow a user to capture this completion message and take the appropriate action inside their application

Issue #, if available

Description of how you validated changes

const subscription = API.graphql<Observable<{
      provider: any;
      value: GraphQLResult<any>;
    }>>(graphqlOperation(onCreateMessage))
    .subscribe({
      next: ({ provider, value }: {provider: any, value: GraphQLResult}) => {
        console.log({ provider, value });
      },
      complete: () => console.log('Complete'),
      error: ({ provider, error }: {provider: any, error: GraphQLResult}) => {
        error.errors?.forEach(e => {
          console.log(e);
        });
      }
    });

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.

[codecov-commenter]

Codecov Report

Merging #9898 (86b72d9) into main (9831d3d) will increase coverage by 5.12%.
The diff coverage is 91.69%.

@@            Coverage Diff             @@
##             main    #9898      +/-   ##
==========================================
+ Coverage   78.34%   83.47%   +5.12%     
==========================================
  Files         250      253       +3     
  Lines       18339    18201     -138     
  Branches     3955     3901      -54     
==========================================
+ Hits        14368    15193     +825     
+ Misses       3840     2918     -922     
+ Partials      131       90      -41     

Impacted Files	Coverage Δ
packages/core/src/constants.ts	100.00% <ø> (ø)
packages/datastore/src/sync/index.ts	73.16% <ø> (+59.74%)	⬆️
packages/datastore/src/sync/merger.ts	91.30% <ø> (ø)
packages/datastore/src/sync/outbox.ts	95.61% <ø> (+3.50%)	⬆️
...ages/datastore/src/sync/processors/subscription.ts	39.23% <28.57%> (+3.27%)	⬆️
packages/api-graphql/src/GraphQLAPI.ts	87.50% <50.00%> (-0.42%)	⬇️
packages/api-rest/src/RestAPI.ts	95.34% <50.00%> (-0.72%)	⬇️
...ackages/datastore/src/sync/processors/errorMaps.ts	54.83% <54.83%> (ø)
packages/geo/src/Geo.ts	81.63% <65.78%> (-13.89%)	⬇️
packages/pushnotification/src/PushNotification.ts	49.07% <75.00%> (-0.24%)	⬇️
... and 65 more

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

[kddejong]

Added tests and now have good codecov delta

[stocaaro]

This change looks like it will work as you propose. It also appears to add functionality that wouldn't impact the existing behavior. Have you linked it to a real project and exercised the behavior this change plugs into?

I think its technically an API change and it would be good to have verification that its working as expected in an end to end use case.

If you've done that validation and remove the comment, this change has my approval.

[stocaaro]

I think you can remove this comment