mirror of
https://github.com/tenrok/axios.git
synced 2026-06-05 16:42:32 +03:00
Michael Toscano
32c7bcc0f2
* fix: prevent request url override prevent request URL from overriding preconfigured base URL BREAKING CHANGE: code relying on the above will now combine the URLs instead of prefer request URL * feat: add config option for allowing absolute URLs * fix: add default value for allowAbsoluteUrls in buildFullPath * fix: typo in flow control when setting allowAbsoluteUrls * feat: update tests supporting issue #5902 functionality * feat: update README.md with allowAbsoluteUrls * fix: properly group conditions in buildFullPath.js to avoid undefined error when baseUrl undefined * Update README.md fix typo * fix: update build full path logic to address failing test case * fix: update base URL test * fix: remove problem test (works locally, will not work in the pipeline) * fix: update https test to use github.com instead of google.com * fix: revert previous commit * fix: add back problem test * chore: remove un-needed passed var to URL class instanciation --------- Co-authored-by: Austin Ryan Lawson <ryan.lawson2@gmail.com> Co-authored-by: Jay <jasonsaayman@gmail.com>
25 lines
1.1 KiB
JavaScript
25 lines
1.1 KiB
JavaScript
import buildFullPath from '../../../lib/core/buildFullPath';
|
|
|
|
describe('helpers::buildFullPath', function () {
|
|
it('should combine URLs when the requestedURL is relative', function () {
|
|
expect(buildFullPath('https://api.github.com', '/users')).toBe('https://api.github.com/users');
|
|
});
|
|
|
|
it('should not combine the URLs when the requestedURL is absolute', function () {
|
|
expect(buildFullPath('https://api.github.com', 'https://api.example.com/users')).toBe('https://api.example.com/users');
|
|
});
|
|
|
|
it('should combine the URLs when the requestedURL is absolute and allowAbsoluteUrls is false', function () {
|
|
expect(buildFullPath('https://api.github.com', 'https://api.example.com/users', false)).toBe('https://api.github.com/https://api.example.com/users');
|
|
});
|
|
|
|
it('should not combine URLs when the baseURL is not configured', function () {
|
|
expect(buildFullPath(undefined, '/users')).toBe('/users');
|
|
});
|
|
|
|
it('should combine URLs when the baseURL and requestedURL are relative', function () {
|
|
expect(buildFullPath('/api', '/users')).toBe('/api/users');
|
|
});
|
|
|
|
});
|