fix(auth): headers for authentication are now set

Merge pull request #4 from janetechinc/francesco/auth-headers

Author: Fire-Dragon-DoL

examples/javascript/src/lib/api-service.mjs

@@ -7,13 +7,14 @@ const defaultHost = "https://api.iheartjane.com"
 const authenticateClient = async () => {
   const clientId = process.env.JANE_CLIENT_ID || ''
   const clientSecret = process.env.JANE_CLIENT_SECRET || ''
-  const apiUrl = process.env.JANE_API_URL
+  const apiUrl = process.env.JANE_API_URL || defaultHost
 
   const resp = await request({
     method: 'post',
     url: `${apiUrl}/oauth/token`,
-    grant_type: 'client_credentials'
-  }, {
+    data: {
+      grant_type: 'client_credentials',
+    },
     auth: {
       username: clientId,
       password: clientSecret
@@ -24,7 +25,7 @@ const authenticateClient = async () => {
 }
 
 const makeRequest = async (options) => {
-  const apiUrl = process.env.JANE_API_URL
+  const apiUrl = process.env.JANE_API_URL || defaultHost
 
   if (!apiUrl) {
     throw Error('No JANE_API_URL configured')

examples/javascript/src/lib/api-service.spec.mjs

@@ -16,16 +16,6 @@ describe('api service', () => {
     )
   })
 
-  describe('if no JANE_API_URL is set', () => {
-    test('it throws an error', async () => {
-      const token = 'someToken'
-
-      await expect(async () => {
-        await apiService.post('/api/path', {}, token)
-      }).rejects.toEqual(new Error('No JANE_API_URL configured'))
-    })
-  })
-
   describe('#get', () => {
     beforeEach(() => {
       process.env.JANE_API_URL = 'https://test.localhost'