Skip to content

Commit 3972b8f

Browse files
committed
Rename LGTM-specific function
1 parent 2d1707d commit 3972b8f

2 files changed

Lines changed: 9 additions & 9 deletions

File tree

extensions/ql-vscode/src/databaseFetcher.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ export async function promptImportLgtmDatabase(
9090
}
9191

9292
if (looksLikeLgtmUrl(lgtmUrl)) {
93-
const databaseUrl = await convertToDatabaseUrl(lgtmUrl, progress);
93+
const databaseUrl = await convertLgtmUrlToDatabaseUrl(lgtmUrl, progress);
9494
if (databaseUrl) {
9595
const item = await databaseArchiveFetcher(
9696
databaseUrl,
@@ -446,7 +446,7 @@ function extractProjectSlug(lgtmUrl: string): string | undefined {
446446
}
447447

448448
// exported for testing
449-
export async function convertToDatabaseUrl(
449+
export async function convertLgtmUrlToDatabaseUrl(
450450
lgtmUrl: string,
451451
progress: ProgressCallback) {
452452
try {

extensions/ql-vscode/src/vscode-tests/no-workspace/databaseFetcher.test.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -9,19 +9,19 @@ import * as chai from 'chai';
99
import { window } from 'vscode';
1010

1111
import {
12-
convertToDatabaseUrl,
12+
convertLgtmUrlToDatabaseUrl,
1313
looksLikeLgtmUrl,
1414
findDirWithFile,
1515
} from '../../databaseFetcher';
1616
import { ProgressCallback } from '../../commandRunner';
1717
chai.use(chaiAsPromised);
1818
const expect = chai.expect;
1919

20-
describe('databaseFetcher', function () {
20+
describe('databaseFetcher', function() {
2121
// These tests make API calls and may need extra time to complete.
2222
this.timeout(10000);
2323

24-
describe('convertToDatabaseUrl', () => {
24+
describe('convertLgtmUrlToDatabaseUrl', () => {
2525
let sandbox: sinon.SinonSandbox;
2626
let quickPickSpy: sinon.SinonStub;
2727
let progressSpy: ProgressCallback;
@@ -39,7 +39,7 @@ describe('databaseFetcher', function () {
3939
it('should convert a project url to a database url', async () => {
4040
quickPickSpy.resolves('javascript');
4141
const lgtmUrl = 'https://lgtm.com/projects/g/github/codeql';
42-
const dbUrl = await convertToDatabaseUrl(lgtmUrl, progressSpy);
42+
const dbUrl = await convertLgtmUrlToDatabaseUrl(lgtmUrl, progressSpy);
4343

4444
expect(dbUrl).to.equal(
4545
'https://lgtm.com/api/v1.0/snapshots/1506465042581/javascript'
@@ -52,7 +52,7 @@ describe('databaseFetcher', function () {
5252
quickPickSpy.resolves('python');
5353
const lgtmUrl =
5454
'https://lgtm.com/projects/g/github/codeql/subpage/subpage2?query=xxx';
55-
const dbUrl = await convertToDatabaseUrl(lgtmUrl, progressSpy);
55+
const dbUrl = await convertLgtmUrlToDatabaseUrl(lgtmUrl, progressSpy);
5656

5757
expect(dbUrl).to.equal(
5858
'https://lgtm.com/api/v1.0/snapshots/1506465042581/python'
@@ -64,7 +64,7 @@ describe('databaseFetcher', function () {
6464
quickPickSpy.resolves('python');
6565
const lgtmUrl =
6666
'g/github/codeql';
67-
const dbUrl = await convertToDatabaseUrl(lgtmUrl, progressSpy);
67+
const dbUrl = await convertLgtmUrlToDatabaseUrl(lgtmUrl, progressSpy);
6868

6969
expect(dbUrl).to.equal(
7070
'https://lgtm.com/api/v1.0/snapshots/1506465042581/python'
@@ -75,7 +75,7 @@ describe('databaseFetcher', function () {
7575
it('should fail on a nonexistent project', async () => {
7676
quickPickSpy.resolves('javascript');
7777
const lgtmUrl = 'https://lgtm.com/projects/g/github/hucairz';
78-
await expect(convertToDatabaseUrl(lgtmUrl, progressSpy)).to.rejectedWith(/Invalid LGTM URL/);
78+
await expect(convertLgtmUrlToDatabaseUrl(lgtmUrl, progressSpy)).to.rejectedWith(/Invalid LGTM URL/);
7979
expect(progressSpy).to.have.callCount(0);
8080
});
8181
});

0 commit comments

Comments
 (0)