Skip to content

Commit e835412

Browse files
committed
feat: report ? for unknown node versions
1 parent 8e85c1e commit e835412

File tree

3 files changed

+27
-3
lines changed

3 files changed

+27
-3
lines changed

lib/travis.js

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -64,10 +64,11 @@ internals.scan = async (travisYaml) => {
6464
const nv = await Nv(version);
6565

6666
if (!nv.length) {
67-
continue;
67+
resolved.push('?');
68+
}
69+
else {
70+
resolved.push(nv[nv.length - 1].version);
6871
}
69-
70-
resolved.push(nv[nv.length - 1].version);
7172
}
7273

7374
return { raw, resolved };
Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
language: node_js
2+
node_js: "i-am-not-a-node-version"

test/index.js

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -375,6 +375,27 @@ describe('node-support', () => {
375375
});
376376
});
377377

378+
it('handles invalid node versions', async () => {
379+
380+
const path = await internals.prepareFixture({
381+
travisYml: 'testing-invalid-version.yml'
382+
});
383+
384+
const result = await NodeSupport.detect({ path });
385+
386+
internals.assertCommit(result);
387+
388+
expect(result).to.equal({
389+
name: 'test-module',
390+
version: '0.0.0-development',
391+
timestamp: 1580673602000,
392+
travis: {
393+
raw: ['i-am-not-a-node-version'],
394+
resolved: ['?']
395+
}
396+
});
397+
});
398+
378399
it('throws when path is not a git repo', async () => {
379400

380401
const path = await internals.prepareFixture({ git: false });

0 commit comments

Comments
 (0)