Skip to content

Commit 460797e

Browse files
technote-spacetechnote-space
committed
feat: supress error (#26)
fix: eslint (#31)
1 parent 6ef2a6f commit 460797e

File tree

12 files changed

+496
-393
lines changed

12 files changed

+496
-393
lines changed

‎__tests__/fixtures/repos.issues.assignees.403.json‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
{
2+
"message": "Resource not accessible by integration",
3+
"documentation_url": "https://developer.github.com/v3"
4+
}

‎__tests__/util.ts‎

Lines changed: 37 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -1,36 +1,37 @@
1-
const fs = require('fs');
2-
const path = require('path');
3-
4-
export const getContext = (override: object) => Object.assign({
5-
payload: {
6-
action: '',
7-
},
8-
eventName: '',
9-
sha: '',
10-
ref: '',
11-
workflow: '',
12-
action: '',
13-
actor: '',
14-
issue: {
15-
owner: '',
16-
repo: '',
17-
number: 1,
18-
},
19-
repo: {
20-
owner: '',
21-
repo: '',
22-
},
23-
}, override);
24-
25-
export const getApiFixture = (name: string) => JSON.parse(fs.readFileSync(path.resolve(__dirname, `./fixtures/${name}.json`)));
26-
27-
export const disableNetConnect = nock => {
28-
beforeEach(() => {
29-
nock.disableNetConnect();
30-
});
31-
32-
afterEach(() => {
33-
nock.cleanAll();
34-
nock.enableNetConnect();
35-
});
36-
};
1+
import { Context } from '@actions/github/lib/context';
2+
import * as fs from 'fs';
3+
import * as path from 'path';
4+
5+
export const getContext = (override: object): Context => Object.assign({
6+
payload: {
7+
action: '',
8+
},
9+
eventName: '',
10+
sha: '',
11+
ref: '',
12+
workflow: '',
13+
action: '',
14+
actor: '',
15+
issue: {
16+
owner: '',
17+
repo: '',
18+
number: 1,
19+
},
20+
repo: {
21+
owner: '',
22+
repo: '',
23+
},
24+
}, override);
25+
26+
export const getApiFixture = (name: string): object => JSON.parse(fs.readFileSync(path.resolve(__dirname, `./fixtures/${name}.json`)).toString());
27+
28+
export const disableNetConnect = (nock): void => {
29+
beforeEach(() => {
30+
nock.disableNetConnect();
31+
});
32+
33+
afterEach(() => {
34+
nock.cleanAll();
35+
nock.enableNetConnect();
36+
});
37+
};

‎__tests__/utils/context.test.ts‎

Lines changed: 123 additions & 122 deletions
Original file line numberDiff line numberDiff line change
@@ -1,122 +1,123 @@
1-
import {getAssignees} from '../../src/utils/context';
2-
import {getContext} from '../util';
3-
4-
describe('getAssignees', () => {
5-
it('should get issue assignees', () => {
6-
expect(getAssignees(getContext({
7-
eventName: 'issues',
8-
payload: {
9-
sender: {
10-
type: 'User',
11-
login: 'test',
12-
},
13-
issue: {
14-
assignees: [],
15-
},
16-
},
17-
}))).toEqual(['test']);
18-
});
19-
20-
it('should get PR assignees', () => {
21-
expect(getAssignees(getContext({
22-
eventName: 'pull_request',
23-
payload: {
24-
sender: {
25-
type: 'User',
26-
login: 'test',
27-
},
28-
pull_request: {
29-
assignees: [],
30-
},
31-
},
32-
}))).toEqual(['test']);
33-
});
34-
35-
it('should empty 1', () => {
36-
expect(getAssignees(getContext({
37-
eventName: 'issues',
38-
payload: {
39-
sender: {
40-
type: 'User',
41-
login: 'test',
42-
},
43-
issue: {
44-
assignees: [
45-
{login: 'test'},
46-
{login: 'test2'},
47-
],
48-
},
49-
},
50-
}))).toEqual([]);
51-
});
52-
53-
it('should empty 2', () => {
54-
expect(getAssignees(getContext({
55-
eventName: 'pull_request',
56-
payload: {
57-
sender: {
58-
type: 'User',
59-
login: 'test',
60-
},
61-
pull_request: {
62-
assignees: [
63-
{login: 'test'},
64-
{login: 'test2'},
65-
],
66-
},
67-
},
68-
}))).toEqual([]);
69-
});
70-
71-
it('should return false 1', () => {
72-
expect(getAssignees(getContext({
73-
eventName: 'issues',
74-
payload: {
75-
issue: {
76-
assignees: [],
77-
},
78-
},
79-
}))).toBeFalsy();
80-
});
81-
82-
it('should return false 2', () => {
83-
expect(getAssignees(getContext({
84-
eventName: 'issues',
85-
payload: {
86-
sender: {
87-
type: 'User',
88-
login: 'test',
89-
},
90-
issue: {},
91-
},
92-
}))).toBeFalsy();
93-
});
94-
95-
it('should return false 3', () => {
96-
expect(getAssignees(getContext({
97-
eventName: 'pull_request',
98-
payload: {
99-
sender: {
100-
type: 'User',
101-
login: 'test',
102-
},
103-
pull_request: {},
104-
},
105-
}))).toBeFalsy();
106-
});
107-
108-
it('should return false 4', () => {
109-
expect(getAssignees(getContext({
110-
eventName: 'push',
111-
payload: {
112-
sender: {
113-
type: 'User',
114-
login: 'test',
115-
},
116-
issue: {
117-
assignees: [],
118-
},
119-
},
120-
}))).toBeFalsy();
121-
});
122-
});
1+
/* eslint-disable no-magic-numbers */
2+
import { getAssignees } from '../../src/utils/context';
3+
import { getContext } from '../util';
4+
5+
describe('getAssignees', () => {
6+
it('should get issue assignees', () => {
7+
expect(getAssignees(getContext({
8+
eventName: 'issues',
9+
payload: {
10+
sender: {
11+
type: 'User',
12+
login: 'test',
13+
},
14+
issue: {
15+
assignees: [],
16+
},
17+
},
18+
}))).toEqual(['test']);
19+
});
20+
21+
it('should get PR assignees', () => {
22+
expect(getAssignees(getContext({
23+
eventName: 'pull_request',
24+
payload: {
25+
sender: {
26+
type: 'User',
27+
login: 'test',
28+
},
29+
'pull_request': {
30+
assignees: [],
31+
},
32+
},
33+
}))).toEqual(['test']);
34+
});
35+
36+
it('should empty 1', () => {
37+
expect(getAssignees(getContext({
38+
eventName: 'issues',
39+
payload: {
40+
sender: {
41+
type: 'User',
42+
login: 'test',
43+
},
44+
issue: {
45+
assignees: [
46+
{login: 'test'},
47+
{login: 'test2'},
48+
],
49+
},
50+
},
51+
}))).toEqual([]);
52+
});
53+
54+
it('should empty 2', () => {
55+
expect(getAssignees(getContext({
56+
eventName: 'pull_request',
57+
payload: {
58+
sender: {
59+
type: 'User',
60+
login: 'test',
61+
},
62+
'pull_request': {
63+
assignees: [
64+
{login: 'test'},
65+
{login: 'test2'},
66+
],
67+
},
68+
},
69+
}))).toEqual([]);
70+
});
71+
72+
it('should return false 1', () => {
73+
expect(getAssignees(getContext({
74+
eventName: 'issues',
75+
payload: {
76+
issue: {
77+
assignees: [],
78+
},
79+
},
80+
}))).toBeFalsy();
81+
});
82+
83+
it('should return false 2', () => {
84+
expect(getAssignees(getContext({
85+
eventName: 'issues',
86+
payload: {
87+
sender: {
88+
type: 'User',
89+
login: 'test',
90+
},
91+
issue: {},
92+
},
93+
}))).toBeFalsy();
94+
});
95+
96+
it('should return false 3', () => {
97+
expect(getAssignees(getContext({
98+
eventName: 'pull_request',
99+
payload: {
100+
sender: {
101+
type: 'User',
102+
login: 'test',
103+
},
104+
'pull_request': {},
105+
},
106+
}))).toBeFalsy();
107+
});
108+
109+
it('should return false 4', () => {
110+
expect(getAssignees(getContext({
111+
eventName: 'push',
112+
payload: {
113+
sender: {
114+
type: 'User',
115+
login: 'test',
116+
},
117+
issue: {
118+
assignees: [],
119+
},
120+
},
121+
}))).toBeFalsy();
122+
});
123+
});

0 commit comments

Comments
 (0)