Skip to content

Commit e8e253d

Browse files
authoredNov 26, 2024
remove obsolete [CratesSize] test for null size (#10688)
* remove obsolete test for version without size changes upstream removed all null crate_size see also rust-lang/crates.io#9926 * remove null handling in CratesSize service
1 parent 60aa530 commit e8e253d

File tree

3 files changed

+2
-12
lines changed

3 files changed

+2
-12
lines changed
 

‎services/crates/crates-base.js

+1-2
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,7 @@ import { BaseJsonService, InvalidResponse } from '../index.js'
44

55
const versionSchema = Joi.object({
66
downloads: nonNegativeInteger,
7-
// Crate size is not available for all versions.
8-
crate_size: nonNegativeInteger.allow(null),
7+
crate_size: nonNegativeInteger,
98
num: Joi.string().required(),
109
license: Joi.string().required().allow(null),
1110
rust_version: Joi.string().allow(null),

‎services/crates/crates-size.service.js

+1-6
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import prettyBytes from 'pretty-bytes'
2-
import { InvalidResponse, pathParams } from '../index.js'
2+
import { pathParams } from '../index.js'
33
import { BaseCratesService, description } from './crates-base.js'
44

55
export default class CratesSize extends BaseCratesService {
@@ -49,11 +49,6 @@ export default class CratesSize extends BaseCratesService {
4949
async handle({ crate, version }) {
5050
const json = await this.fetch({ crate, version })
5151
const size = this.constructor.getVersionObj(json).crate_size
52-
53-
if (size == null) {
54-
throw new InvalidResponse({ prettyMessage: 'unknown' })
55-
}
56-
5752
return this.render({ size })
5853
}
5954
}

‎services/crates/crates-size.tester.js

-4
Original file line numberDiff line numberDiff line change
@@ -10,10 +10,6 @@ t.create('size (with version)')
1010
.get('/tokio/1.32.0.json')
1111
.expectBadge({ label: 'size', message: '725 kB' })
1212

13-
t.create('size (with version where version doesnt have size)')
14-
.get('/tokio/0.1.6.json')
15-
.expectBadge({ label: 'crates.io', message: 'unknown' })
16-
1713
t.create('size (not found)')
1814
.get('/not-a-crate.json')
1915
.expectBadge({ label: 'crates.io', message: 'not found' })

0 commit comments

Comments
 (0)