-
Notifications
You must be signed in to change notification settings - Fork 8.2k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Migrate base path APIs and UiSettings client to new platform (#22694)
Fixes #20697 This PR migrates the base path related methods from `ui/chrome` to `core.basePath` and the uiSettings to `core.uiSettings`. The two are not split into separate PRs because I'm kinda cramped for time right now so rather I split the changes up into two commits so that you can review them separately if you like. If you'd like I can submit them as separate PRs but the basePath PR will block the uiSettings PR either way. There shouldn't be any API changes except one thing, since the existing implementation is so close to what we'd want from the new platform API I made the one change that is inconsistent with what we've been doing so far and moved `uiSettings.subscribe()` to `uiSettings.getUpdate$().subscribe()`. This method isn't super commonly used, but it is a breaking change that will likely impact plugins so I'll notify some folks if we decide to move forward this way. I can also make a super-light wrapper for angular that just updates this method if you prefer.
- Loading branch information
Spencer
authored
Sep 8, 2018
1 parent
7e94ecc
commit 52060b8
Showing
43 changed files
with
1,799 additions
and
625 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,104 @@ | ||
/* | ||
* Licensed to Elasticsearch B.V. under one or more contributor | ||
* license agreements. See the NOTICE file distributed with | ||
* this work for additional information regarding copyright | ||
* ownership. Elasticsearch B.V. licenses this file to you under | ||
* the Apache License, Version 2.0 (the "License"); you may | ||
* not use this file except in compliance with the License. | ||
* You may obtain a copy of the License at | ||
* | ||
* http://www.apache.org/licenses/LICENSE-2.0 | ||
* | ||
* Unless required by applicable law or agreed to in writing, | ||
* software distributed under the License is distributed on an | ||
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY | ||
* KIND, either express or implied. See the License for the | ||
* specific language governing permissions and limitations | ||
* under the License. | ||
*/ | ||
|
||
import { BasePathService } from './base_path_service'; | ||
|
||
function setup(options: any = {}) { | ||
const injectedBasePath: string = | ||
options.injectedBasePath === undefined ? '/foo/bar' : options.injectedBasePath; | ||
|
||
const service = new BasePathService(); | ||
|
||
const injectedMetadata = { | ||
getBasePath: jest.fn().mockReturnValue(injectedBasePath), | ||
} as any; | ||
|
||
const startContract = service.start({ | ||
injectedMetadata, | ||
}); | ||
|
||
return { | ||
service, | ||
startContract, | ||
injectedBasePath, | ||
}; | ||
} | ||
|
||
describe('startContract.get()', () => { | ||
it('returns an empty string if no basePath is injected', () => { | ||
const { startContract } = setup({ injectedBasePath: null }); | ||
expect(startContract.get()).toBe(''); | ||
}); | ||
|
||
it('returns the injected basePath', () => { | ||
const { startContract } = setup(); | ||
expect(startContract.get()).toBe('/foo/bar'); | ||
}); | ||
}); | ||
|
||
describe('startContract.addToPath()', () => { | ||
it('adds the base path to the path if it is relative and starts with a slash', () => { | ||
const { startContract } = setup(); | ||
expect(startContract.addToPath('/a/b')).toBe('/foo/bar/a/b'); | ||
}); | ||
|
||
it('leaves the query string and hash of path unchanged', () => { | ||
const { startContract } = setup(); | ||
expect(startContract.addToPath('/a/b?x=y#c/d/e')).toBe('/foo/bar/a/b?x=y#c/d/e'); | ||
}); | ||
|
||
it('returns the path unchanged if it does not start with a slash', () => { | ||
const { startContract } = setup(); | ||
expect(startContract.addToPath('a/b')).toBe('a/b'); | ||
}); | ||
|
||
it('returns the path unchanged it it has a hostname', () => { | ||
const { startContract } = setup(); | ||
expect(startContract.addToPath('http://localhost:5601/a/b')).toBe('http://localhost:5601/a/b'); | ||
}); | ||
}); | ||
|
||
describe('startContract.removeFromPath()', () => { | ||
it('removes the basePath if relative path starts with it', () => { | ||
const { startContract } = setup(); | ||
expect(startContract.removeFromPath('/foo/bar/a/b')).toBe('/a/b'); | ||
}); | ||
|
||
it('leaves query string and hash intact', () => { | ||
const { startContract } = setup(); | ||
expect(startContract.removeFromPath('/foo/bar/a/b?c=y#1234')).toBe('/a/b?c=y#1234'); | ||
}); | ||
|
||
it('ignores urls with hostnames', () => { | ||
const { startContract } = setup(); | ||
expect(startContract.removeFromPath('http://localhost:5601/foo/bar/a/b')).toBe( | ||
'http://localhost:5601/foo/bar/a/b' | ||
); | ||
}); | ||
|
||
it('returns slash if path is just basePath', () => { | ||
const { startContract } = setup(); | ||
expect(startContract.removeFromPath('/foo/bar')).toBe('/'); | ||
}); | ||
|
||
it('returns full path if basePath is not its own segment', () => { | ||
const { startContract } = setup(); | ||
expect(startContract.removeFromPath('/foo/barhop')).toBe('/foo/barhop'); | ||
}); | ||
}); |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,74 @@ | ||
/* | ||
* Licensed to Elasticsearch B.V. under one or more contributor | ||
* license agreements. See the NOTICE file distributed with | ||
* this work for additional information regarding copyright | ||
* ownership. Elasticsearch B.V. licenses this file to you under | ||
* the Apache License, Version 2.0 (the "License"); you may | ||
* not use this file except in compliance with the License. | ||
* You may obtain a copy of the License at | ||
* | ||
* http://www.apache.org/licenses/LICENSE-2.0 | ||
* | ||
* Unless required by applicable law or agreed to in writing, | ||
* software distributed under the License is distributed on an | ||
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY | ||
* KIND, either express or implied. See the License for the | ||
* specific language governing permissions and limitations | ||
* under the License. | ||
*/ | ||
|
||
import { InjectedMetadataStartContract } from '../injected_metadata'; | ||
import { modifyUrl } from '../utils'; | ||
|
||
interface Deps { | ||
injectedMetadata: InjectedMetadataStartContract; | ||
} | ||
|
||
export class BasePathService { | ||
public start({ injectedMetadata }: Deps) { | ||
const basePath = injectedMetadata.getBasePath() || ''; | ||
|
||
return { | ||
/** | ||
* Get the current basePath as defined by the server | ||
*/ | ||
get() { | ||
return basePath; | ||
}, | ||
|
||
/** | ||
* Add the current basePath to a path string. | ||
* @param path A relative url including the leading `/`, otherwise it will be returned without modification | ||
*/ | ||
addToPath(path: string) { | ||
return modifyUrl(path, parts => { | ||
if (!parts.hostname && parts.pathname && parts.pathname.startsWith('/')) { | ||
parts.pathname = `${basePath}${parts.pathname}`; | ||
} | ||
}); | ||
}, | ||
|
||
/** | ||
* Remove the basePath from a path that starts with it | ||
* @param path A relative url that starts with the basePath, which will be stripped | ||
*/ | ||
removeFromPath(path: string) { | ||
if (!basePath) { | ||
return path; | ||
} | ||
|
||
if (path === basePath) { | ||
return '/'; | ||
} | ||
|
||
if (path.startsWith(basePath + '/')) { | ||
return path.slice(basePath.length); | ||
} | ||
|
||
return path; | ||
}, | ||
}; | ||
} | ||
} | ||
|
||
export type BasePathStartContract = ReturnType<BasePathService['start']>; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
26 changes: 26 additions & 0 deletions
26
src/core/public/legacy_platform/__snapshots__/legacy_platform_service.test.ts.snap
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.