Skip to content

Commit f95dcc4

Browse files
committed
android permissions issue fix
1 parent d67d61f commit f95dcc4

File tree

3 files changed

+31
-39
lines changed

3 files changed

+31
-39
lines changed

src/__tests__/handlePermissionsDenied.test.tsx

Lines changed: 15 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,24 +1,20 @@
11
import { requestPermissions } from '../handlePermissions';
22

3-
jest.mock('react-native/Libraries/Utilities/Platform', () => {
4-
const Platform = jest.requireActual(
5-
'react-native/Libraries/Utilities/Platform'
6-
);
7-
Platform.OS = 'android';
8-
return Platform;
9-
});
10-
11-
jest.mock(
12-
'react-native//Libraries/PermissionsAndroid/PermissionsAndroid',
13-
() => {
14-
return {
15-
...jest.requireActual(
16-
'react-native//Libraries/PermissionsAndroid/PermissionsAndroid'
17-
),
18-
request: jest.fn(() => new Promise((resolve) => resolve('denied'))),
19-
};
20-
}
21-
);
3+
jest.mock('react-native', () => ({
4+
Platform: {
5+
OS: 'android',
6+
},
7+
PermissionsAndroid: {
8+
PERMISSIONS: {
9+
WRITE_EXTERNAL_STORAGE: 'android.permission.WRITE_EXTERNAL_STORAGE',
10+
},
11+
RESULTS: {
12+
GRANTED: 'granted',
13+
DENIED: 'denied',
14+
},
15+
request: jest.fn(() => Promise.resolve(false)),
16+
},
17+
}));
2218

2319
describe('Permission Denied', () => {
2420
it('should return false if permission is not granted', async () => {

src/__tests__/handlePermissionsGranted.test.tsx

Lines changed: 15 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,24 +1,20 @@
11
import { requestPermissions } from '../handlePermissions';
22

3-
jest.mock('react-native/Libraries/Utilities/Platform', () => {
4-
const Platform = jest.requireActual(
5-
'react-native/Libraries/Utilities/Platform'
6-
);
7-
Platform.OS = 'android';
8-
return Platform;
9-
});
10-
11-
jest.mock(
12-
'react-native//Libraries/PermissionsAndroid/PermissionsAndroid',
13-
() => {
14-
return {
15-
...jest.requireActual(
16-
'react-native//Libraries/PermissionsAndroid/PermissionsAndroid'
17-
),
18-
request: jest.fn(() => new Promise((resolve) => resolve('granted'))),
19-
};
20-
}
21-
);
3+
jest.mock('react-native', () => ({
4+
Platform: {
5+
OS: 'android',
6+
},
7+
PermissionsAndroid: {
8+
PERMISSIONS: {
9+
WRITE_EXTERNAL_STORAGE: 'android.permission.WRITE_EXTERNAL_STORAGE',
10+
},
11+
RESULTS: {
12+
GRANTED: 'granted',
13+
DENIED: 'denied',
14+
},
15+
request: jest.fn(() => Promise.resolve(true)),
16+
},
17+
}));
2218

2319
describe('Permission Granted', () => {
2420
it('should return true if permission is granted', async () => {

src/handlePermissions.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ export const requestPermissions = async (
1414
}
1515
);
1616

17-
return granted === PermissionsAndroid.RESULTS.GRANTED;
17+
return granted;
1818
}
1919
return true;
2020
};

0 commit comments

Comments
 (0)