release: Amplify JS release (aws-amplify#13569)

Author: ashwinkumar6

.eslintrc.js

@@ -35,25 +35,13 @@ module.exports = {
 		'setupTests.ts',
 		'jest.setup.*',
 		'jest.config.*',
-		// 'packages/adapter-nextjs/__tests__',
-		// 'packages/analytics/__tests__',
 		'packages/api/__tests__',
 		'packages/api-graphql/__tests__',
-		// 'packages/api-rest/__tests__',
-		// 'packages/auth/__tests__',
-		// 'packages/aws-amplify/__tests__',
-		// 'packages/core/__tests__',
 		'packages/datastore/__tests__',
 		'packages/datastore-storage-adapter/__tests__',
-		// 'packages/geo/__tests__',
 		'packages/interactions/__tests__',
-		// 'packages/notifications/__tests__',
 		'packages/predictions/__tests__',
 		'packages/pubsub/__tests__',
-		'packages/react-native/__tests__',
-		'packages/rtn-push-notification/__tests__',
-		'packages/rtn-web-browser/__tests__',
-		// 'packages/storage/__tests__',
 	],
 	rules: {
 		camelcase: [

.github/PULL_REQUEST_TEMPLATE.md

@@ -24,8 +24,15 @@ the requirements below.
 
 - [ ] PR description included
 - [ ] `yarn test` passes
-- [ ] Tests are [changed or added](https://github.com/aws-amplify/amplify-js/blob/main/CONTRIBUTING.md#steps-towards-contributions)
+- [ ] Unit Tests are [changed or added](https://github.com/aws-amplify/amplify-js/blob/main/CONTRIBUTING.md#steps-towards-contributions)
 - [ ] Relevant documentation is changed or added (and PR referenced)
+
+
+
+#### Checklist for repo maintainers
+<!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->
+
+- [ ] Verify E2E tests for existing workflows are working as expected or add E2E tests for newly added workflows
 - [ ] New source file paths included in this PR have been added to CODEOWNERS, if appropriate
-      
+
 By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

.github/integ-config/integ-all.yml

@@ -822,7 +822,14 @@ tests:
     sample_name: [storage-gen2]
     spec: storage-gen2
     browser: *minimal_browser_list
-
+  - test_name: storage-guest-access
+    desc: 'Next Storage guest access'
+    framework: next
+    category: storage
+    sample_name: [guest-access]
+    spec: storage-client-server
+    browser: *minimal_browser_list
+  
   # INAPPMESSAGING
   - test_name: integ_in_app_messaging
     desc: 'React InApp Messaging'

.github/workflows/callable-npm-publish-release.yml

@@ -59,7 +59,7 @@ jobs:
         run: |
           yarn run docs
           git add ./docs/api/
-          git commit -m "chore(release): update API docs [skip release]"
+          git commit -m "chore(release): Update API docs [skip release]"
 
       - name: Push post release changes to the release branch
         working-directory: ./amplify-js

package.json

@@ -31,7 +31,7 @@
 		"publish:release": "yarn generate-metadata && lerna publish --conventional-commits --message 'chore(release): Publish [skip release]' --yes",
 		"publish:v5-stable": "lerna publish --conventional-commits --yes --dist-tag=stable-5 --message 'chore(release): Publish [ci skip]' --no-verify-access",
 		"publish:verdaccio": "lerna publish --canary --force-publish --no-push --dist-tag=unstable --preid=unstable --yes",
-		"generate-metadata": "git rev-parse --short HEAD > packages/core/metadata && git commit -am 'chore: set core metadata [skip release]'",
+		"generate-metadata": "git rev-parse --short HEAD > packages/core/metadata && git commit -am 'chore(release): Set core metadata [skip release]'",
 		"ts-coverage": "lerna run ts-coverage",
 		"prepare": "husky && ./scripts/set-preid-versions.sh"
 	},

packages/aws-amplify/package.json

@@ -485,7 +485,7 @@
 			"name": "[Storage] list (S3)",
 			"path": "./dist/esm/storage/index.mjs",
 			"import": "{ list }",
-			"limit": "14.94 kB"
+			"limit": "15.04 kB"
 		},
 		{
 			"name": "[Storage] remove (S3)",

packages/datastore/__tests__/subscription.test.ts

@@ -126,6 +126,34 @@ describe('sync engine subscription module', () => {
 			),
 		).toEqual(authInfo);
 	});
+	test('owner authorization with no token(expired)', () => {
+		const authRules = [
+			{
+				provider: 'userPools',
+				ownerField: 'owner',
+				allow: 'owner',
+				identityClaim: 'cognito:username',
+				operations: ['create', 'update', 'delete'],
+			},
+		];
+		const model = generateModelWithAuth(authRules);
+
+		const authInfo = {
+			authMode: 'userPool',
+			isOwner: false,
+		};
+
+		expect(
+			// @ts-ignore
+			SubscriptionProcessor.prototype.getAuthorizationInfo(
+				model,
+				USER_CREDENTIALS.auth,
+				'userPool',
+				undefined,
+				'userPool',
+			),
+		).toEqual(authInfo);
+	});
 	test('owner authorization with public subscription', () => {
 		const authRules = [
 			{

packages/datastore/src/sync/processors/subscription.ts

@@ -205,7 +205,7 @@ class SubscriptionProcessor {
 				: [];
 
 		oidcOwnerAuthRules.forEach(ownerAuthRule => {
-			const ownerValue = oidcTokenPayload[ownerAuthRule.identityClaim];
+			const ownerValue = oidcTokenPayload?.[ownerAuthRule.identityClaim];
 			const singleOwner =
 				model.fields[ownerAuthRule.ownerField]?.isArray !== true;
 			const isOwnerArgRequired =

packages/rtn-push-notification/__tests__/apis/getPermissionStatus.test.ts

@@ -23,7 +23,7 @@ describe('getPermissionStatus', () => {
 	beforeAll(() => {
 		mockGetPermissionStatusNative.mockResolvedValue(status);
 		mockNormalizeNativePermissionStatus.mockImplementation(
-			status => `normalized-${status}`,
+			statusParam => `normalized-${statusParam}`,
 		);
 	});
 

packages/rtn-push-notification/__tests__/apis/registerHeadlessTask.test.ts

@@ -2,6 +2,7 @@
 // SPDX-License-Identifier: Apache-2.0
 
 import { AppRegistry } from 'react-native';
+
 import { getConstants } from '../../src/apis/getConstants';
 import { registerHeadlessTask } from '../../src/apis/registerHeadlessTask';
 import { normalizeNativeMessage } from '../../src/utils';