[Feature] Support for enterprise Github repository

Add a test, and make sure to support configuration that do not have the parameter

Author: nlunets

src/spec/git/GithubURLs.ts

@@ -36,11 +36,11 @@ describe('GithubRepo / GithubURLs', () => {
 		});
 		it('should throw on bad params', () => {
 			assert.throws(() => {
-				repo = new GithubRepo({repoOwner: null, repoProject: null}, null, null);
+				repo = new GithubRepo({repoOwner: null, repoProject: null, githubHost: null}, null, null);
 			});
 		});
 		it('should be constructor', () => {
-			repo = new GithubRepo({repoOwner: 'foo', repoProject: 'bar'}, 'baz', gitTest.opts);
+			repo = new GithubRepo({repoOwner: 'foo', repoProject: 'bar', githubHost: 'github.com'}, 'baz', gitTest.opts);
 			urls = repo.urls;
 			assert.ok(urls, 'instance');
 		});
@@ -56,7 +56,7 @@ describe('GithubRepo / GithubURLs', () => {
 			});
 		});
 		it('should return replaced urls', () => {
-			urls = new GithubRepo({repoOwner: 'foo', repoProject: 'bar'}, 'baz', gitTest.opts).urls;
+			urls = new GithubRepo({repoOwner: 'foo', repoProject: 'bar', githubHost: 'github.com'}, 'baz', gitTest.opts).urls;
 			var api = 'https://api.github.com/repos/foo/bar';
 			var raw = 'https://raw.githubusercontent.com/foo/bar';
 			var base = 'https://github.com/foo/bar';
@@ -66,7 +66,7 @@ describe('GithubRepo / GithubURLs', () => {
 			assert.strictEqual(urls.rawFile('2ece23298f06d9fb45772fdb1d38086918c80f44', 'sub/folder/file.txt'), rawFile, 'rawFile');
 		});
 		it('should return correctly replaced urls if repoConfig is modified after repo creation', () => {
-			var repoConfig = {repoOwner: 'foo', repoProject: 'bar'};
+			var repoConfig = {repoOwner: 'foo', repoProject: 'bar', githubHost: 'github.com'};
 			urls = new GithubRepo(repoConfig, 'baz', gitTest.opts).urls;
 			repoConfig.repoOwner = 'correctOwner';
 			repoConfig.repoProject = 'correctProject';
@@ -77,9 +77,19 @@ describe('GithubRepo / GithubURLs', () => {
 			assert.strictEqual(urls.base(), base, 'base');
 		});
 		it('should return no trailing slash', () => {
-			urls = new GithubRepo({repoOwner: 'foo', repoProject: 'bar'}, 'baz', gitTest.opts).urls;
+			urls = new GithubRepo({repoOwner: 'foo', repoProject: 'bar', githubHost: 'github.com'}, 'baz', gitTest.opts).urls;
 			assert.notMatch(urls.apiBranches(), /\/$/, 'apiBranches');
 			assert.notMatch(urls.apiBranch('abc'), /\/$/, 'apiBranch');
 		});
+		it('should handle enterprise github urls', () => {
+			urls = new GithubRepo({repoOwner: 'foo', repoProject: 'bar', githubHost: 'github.mycompany.com'}, 'baz', gitTest.opts).urls;
+			var api = 'https://github.mycompany.com/api/v3/repos/foo/bar';
+			var raw = 'https://github.mycompany.com/foo/bar/raw';
+			var base = 'https://github.mycompany.com/foo/bar';
+			var rawFile = raw + '/2ece23298f06d9fb45772fdb1d38086918c80f44/sub/folder/file.txt';
+			assert.strictEqual(urls.api(), api, 'api');
+			assert.strictEqual(urls.base(), base, 'base');
+			assert.strictEqual(urls.rawFile('2ece23298f06d9fb45772fdb1d38086918c80f44', 'sub/folder/file.txt'), rawFile, 'rawFile');
+		});
 	});
 });

src/tsd/context/Config.ts

@@ -149,7 +149,11 @@ class Config implements GithubRepoConfig {
 		var json = this.otherFields;
 		json.version = this.version;
 		json.repo = this.repo;
-		json.githubHost = this.githubHost;
+		if (this.githubHost !== Const.githubHost) {
+			// only write to config if the host is not github.com
+			json.githubHost = this.githubHost;
+		}
+
 		json.ref = this.ref;
 		json.path = this.path;
 
@@ -195,6 +199,10 @@ class Config implements GithubRepoConfig {
 		this.version = json.version;
 		this.repo = json.repo;
 		this.githubHost = json.githubHost;
+		if (!this.githubHost) {
+			// when migrating from file that do not have this parameter make sure upgrade won't fail
+			this.githubHost = Const.githubHost;
+		}
 		this.ref = json.ref;
 		this.bundle = json.bundle;
 		this.stats = (typeOf.isBoolean(json.stats) ? json.stats : Const.statsDefault);

src/tsd/schema/config.ts

@@ -21,6 +21,11 @@ var schema = Joi.object({
 		.default(Const.definitelyRepo)
 		.required()
 		.description('github repository "owner/name"'),
+	githubHost: Joi
+		.string()
+		.default(Const.githubHost)
+		.optional()
+		.description('github url, used to specify github enteprise url'),
 	ref: Joi
 		.string().regex(/^[\w\.-]+(?:\/[\w\.-]+)*$/)
 		.default(Const.mainBranch)

test/spec/git/fixtures/config.json

@@ -2,6 +2,7 @@
 	"repo": {
 		"repoOwner": "borisyankov",
 		"repoProject": "DefinitelyTyped",
+		"githubHost": "github.com",
 		"ref": "master"
 	},
 	"data": {