forked from electron/apps
-
Notifications
You must be signed in to change notification settings - Fork 0
Expand file tree
/
Copy pathhuman-data.js
More file actions
203 lines (172 loc) · 7.53 KB
/
Copy pathhuman-data.js
File metadata and controls
203 lines (172 loc) · 7.53 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
const categories = require('../lib/app-categories');
const mocha = require('mocha');
const describe = mocha.describe;
const it = mocha.it;
const fs = require('fs');
const path = require('path');
const expect = require('chai').expect;
const yaml = require('yaml');
const { isUrl } = require('../lib/is-url');
const { URL } = require('url');
const slugify = require('slugify');
const grandfatheredDescriptions = require('../lib/grandfathered-descriptions');
const grandfatheredLinks = require('../lib/grandfathered-links.js');
const grandfatheredSlugs = require('../lib/grandfathered-small-icons');
const slugs = fs.readdirSync(path.join(__dirname, '../apps')).filter((filename) => {
return fs.statSync(path.join(__dirname, `../apps/${filename}`)).isDirectory();
});
const IGNORE_IMAGES_SQUARE = ['asana', 'splice'];
describe('human-submitted app data', () => {
it('includes lots of apps', () => {
expect(slugs.length).to.be.above(200);
});
slugs.forEach((slug) => {
describe(slug, () => {
const basedir = path.join(__dirname, `../apps/${slug}`);
const yamlFile = `${slug}.yml`;
const yamlPath = path.join(basedir, yamlFile);
const iconPath = path.join(basedir, `${slug}-icon.png`);
it('is in a directory whose name is lowercase with dashes as a delimiter', () => {
expect(slugify(slug)).to.equal(slug);
});
it(`includes a data file named ${slug}.yml`, () => {
expect(fs.existsSync(yamlPath)).to.equal(true);
});
describe(`${yamlFile}`, () => {
const app = yaml.parse(fs.readFileSync(yamlPath, 'utf-8'));
it('has a name', () => {
expect(app.name.length).to.be.above(0);
});
describe('description', () => {
it('exists', () => {
expect(app.description.length).to.be.above(0);
});
it('should not start with app name', () => {
const appName = app.name.toLowerCase();
const description = app.description.toLowerCase();
expect(description).to.satisfy((desc) => !desc.startsWith(appName));
});
const descIsGrandfathered = grandfatheredDescriptions.includes(slug);
if (!descIsGrandfathered) {
it('should start with a capital letter', () => {
const firstLetter = app.description[0];
expect(firstLetter).to.equal(firstLetter.toUpperCase());
});
it('should end with a period / full stop', () => {
expect(app.description[app.description.length - 1]).to.equal(
'.',
`Description should end in a period / full stop: '${app.description}'`,
);
});
it('should not mention Electron since Electron is already implied', () => {
const description = app.description.toLowerCase();
expect(description.indexOf('electron')).to.equal(
-1,
`Description should not mention Electron, as Electron is already implied: ${description}`,
);
});
it('should not start description with "A" or "An"', () => {
const descriptionFirstWord = app.description.toLowerCase().split(' ', 1)[0];
const badStarts = ['a', 'an'];
expect(badStarts).to.not.include(
descriptionFirstWord,
`Description should not start with 'A' or 'An': '${app.description}'`,
);
});
}
});
const linksAreGrandfathered = grandfatheredLinks.includes(slug);
if (!linksAreGrandfathered) {
// walk an object subtree looking for URLs
const getObjectUrls = (root) => {
const found = [];
const queue = [root];
while (queue.length !== 0) {
const vals = Object.values(queue.shift());
found.push(...vals.filter(isUrl).map((v) => new URL(v)));
queue.push(...vals.filter((v) => typeof v === 'object'));
}
return found;
};
it('should use ssl links', () => {
const goodProtocols = ['https:', 'sftp:'];
const urls = getObjectUrls(app);
urls.forEach((url) => expect(url.protocol, url).to.be.oneOf(goodProtocols));
});
}
it('has a website with a valid URL (or no website)', () => {
expect(!app.website || isUrl(app.website)).to.equal(true);
});
it('has a valid repository URL (or no repository)', () => {
expect(!app.repository || isUrl(app.repository)).to.equal(true);
});
describe('keywords', () => {
it('should, if present, be an array of keywords', () => {
expect(app.keywords || []).to.be.an('array');
});
it("should not include 'electron'", () => {
expect((app.keywords || []).map((key) => key.toLocaleLowerCase())).to.not.include(
'electron',
);
});
it('should not include duplicates', () => {
const keywords = app.keywords || [];
expect(keywords.sort().toString()).to.equal(
[...new Set(keywords).values()].sort().toString(),
);
});
});
it('has a valid category', () => {
expect(app.category.length).to.be.above(0);
expect(app.category).to.be.oneOf(categories);
});
describe('screenshots', () => {
const screenshots = app.screenshots || [];
it('requires imageUrl to be a fully-qualified HTTPS URL', () => {
screenshots.forEach((screenshot) => {
expect(
isUrl(screenshot.imageUrl) && screenshot.imageUrl.startsWith('https'),
).to.equal(
true,
`${app.slug} screenshot imageUrl must be a fully-qualified HTTPS URL`,
);
});
});
it('requires linkUrl to be a fully-qualified URL, if present', () => {
screenshots.forEach((screenshot) => {
expect(!screenshot.linkUrl || isUrl(screenshot.linkUrl)).to.equal(
true,
`${app.slug} screenshot linkURL must be a fully qualified URL`,
);
});
});
});
it('has a valid YouTube URL (or none)', () => {
expect(!app.youtube_video_url || isUrl(app.youtube_video_url)).to.equal(true);
});
});
// Do this without any external dependencies
describe.skip('icon', () => {
it(`exists as ${slug}-icon.png`, () => {
expect(fs.existsSync(iconPath)).to.equal(true, `${slug}-icon.png not found`);
});
it('is a square', function () {
if (IGNORE_IMAGES_SQUARE.includes(slug)) return this.skip();
if (!fs.existsSync(iconPath)) return this.skip();
const dimensions = imageSize(iconPath);
expect(dimensions.width).to.be.a('number');
expect(dimensions.width).to.equal(dimensions.height);
});
const minPixels = grandfatheredSlugs.indexOf(slug) > -1 ? 128 : 256;
const maxPixels = 1024;
it(`is between ${minPixels}px x ${minPixels}px and ${maxPixels}px x ${maxPixels}px`, function () {
if (IGNORE_IMAGES_SQUARE.includes(slug)) return this.skip();
if (!fs.existsSync(iconPath)) return this.skip();
const dimensions = imageSize(iconPath);
expect(dimensions.width).to.be.within(minPixels, maxPixels);
expect(dimensions.height).to.be.within(minPixels, maxPixels);
});
});
});
});
});