Fix/remove url required (#4426)

* Removed error when url is null as this breaks current use cases for alot of projects

* Removed associated tests that check for the for url to not be empty

lib/core/Axios.js

@@ -36,10 +36,6 @@ Axios.prototype.request = function request(configOrUrl, config) {
     config = configOrUrl || {};
   }
 
-  if (!config.url) {
-    throw new Error('Provided config url is not valid');
-  }
-
   config = mergeConfig(this.defaults, config);
 
   // Set config.method
@@ -122,9 +118,6 @@ Axios.prototype.request = function request(configOrUrl, config) {
 };
 
 Axios.prototype.getUri = function getUri(config) {
-  if (!config.url) {
-    throw new Error('Provided config url is not valid');
-  }
   config = mergeConfig(this.defaults, config);
   return buildURL(config.url, config.params, config.paramsSerializer).replace(/^\?/, '');
 };

test/specs/requests.spec.js

@@ -7,24 +7,6 @@ describe('requests', function () {
     jasmine.Ajax.uninstall();
   });
 
-  it('should throw error when missing url', function (done) {
-    expect(() => axios()).toThrowError(/Provided config url is not valid/);
-    done();
-
-    expect(() => axios('')).toThrowError(/Provided config url is not valid/);
-    done();
-
-    expect(() => axios({
-      url: undefined,
-    })).toThrowError(/Provided config url is not valid/);
-    done();
-
-    expect(() => axios({
-      method: 'POST',
-    })).toThrowError(/Provided config url is not valid/);
-    done();
-  });
-
   it('should treat single string arg as url', function (done) {
     axios('/foo');