@looker/api-explorer 0.9.52 → 0.9.54

package/public/versions.json

@@ -1,22 +1,16 @@
 {
   "looker_release_version": "21.12",
   "current_version": {
-    "version": "3.1",
-    "full_version": "3.1.0",
+    "version": "4.0",
+    "full_version": "4.0.23.18",
     "status": "current",
-    "swagger_url": "Looker.3.1.json"
+    "swagger_url": "Looker.4.0.json"
   },
   "supported_versions": [
-    {
-      "version": "3.1",
-      "full_version": "3.1.0",
-      "status": "current",
-      "swagger_url": "/Looker.3.1.json"
-    },
     {
       "version": "4.0",
       "full_version": "4.0.21.12",
-      "status": "beta",
+      "status": "current",
       "swagger_url": "/Looker.4.0.json"
     }
   ],

package/src/components/DocMarkdown/DocMarkdown.spec.tsx

@@ -37,7 +37,7 @@ describe('DocMarkdown', () => {
         source={
           '# Markdown Component \n Renders markdown using [ReactMarkdown](https://github.com/rexxars/react-markdown)'
         }
-        specKey={'3.1'}
+        specKey={'4.0'}
       />
     )
     const heading = screen.getByRole('heading')
@@ -53,11 +53,11 @@ describe('DocMarkdown', () => {
   test('it remaps hashbang urls found in markdown input', () => {
     const input =
       'A link to the [create_dashboard](#!/Dashboard/create_dashboard) endpoint'
-    renderWithReduxProvider(<DocMarkdown source={input} specKey={'3.1'} />)
+    renderWithReduxProvider(<DocMarkdown source={input} specKey={'4.0'} />)
     expect(screen.getByText(/A link to the/)).toBeInTheDocument()
     expect(screen.getByText('create_dashboard')).toHaveAttribute(
       'href',
-      '/3.1/methods/Dashboard/create_dashboard'
+      '/4.0/methods/Dashboard/create_dashboard'
     )
   })
 
@@ -65,7 +65,7 @@ describe('DocMarkdown', () => {
     renderWithReduxProvider(
       <DocMarkdown
         source={'[external_link](https://www.foo.com)'}
-        specKey={'3.1'}
+        specKey={'4.0'}
       />
     )
     expect(screen.getByText('external_link')).toHaveAttribute(
@@ -82,7 +82,7 @@ describe('DocMarkdown', () => {
     renderWithReduxProvider(
       <DocMarkdown
         source={'An API Explorer to explore your OpenAPI spec'}
-        specKey={'3.1'}
+        specKey={'4.0'}
       />,
       store
     )
@@ -98,16 +98,16 @@ describe('DocMarkdown', () => {
     renderWithReduxProvider(
       <DocMarkdown
         source={
-          'An inline styled link with matching text in both the link text and the href: [create_dashboard](/3.1/methods/Dashboard/create_dashboard)'
+          'An inline styled link with matching text in both the link text and the href: [create_dashboard](/4.0/methods/Dashboard/create_dashboard)'
         }
-        specKey={'3.1'}
+        specKey={'4.0'}
       />,
       store
     )
     const link = screen.getByRole('link')
     expect(link).toHaveAttribute(
       'href',
-      '/3.1/methods/Dashboard/create_dashboard'
+      '/4.0/methods/Dashboard/create_dashboard'
     )
     expect(link).toContainHTML('create_<mark>dashboard</mark>')
   })
@@ -115,7 +115,7 @@ describe('DocMarkdown', () => {
   test('it renders code blocks', () => {
     const code =
       '```\nAuthorization: token 4QDkCyCtZzYgj4C2p2cj3csJH7zqS5RzKs2kTnG4\n```'
-    renderWithReduxProvider(<DocMarkdown source={code} specKey={'3.1'} />)
+    renderWithReduxProvider(<DocMarkdown source={code} specKey={'4.0'} />)
     expect(
       screen.getByText(
         'Authorization: token 4QDkCyCtZzYgj4C2p2cj3csJH7zqS5RzKs2kTnG4'
@@ -125,7 +125,7 @@ describe('DocMarkdown', () => {
 
   test('it renders inline code', () => {
     const markdown = 'Some text with code: `const noop = () => null`'
-    renderWithReduxProvider(<DocMarkdown source={markdown} specKey={'3.1'} />)
+    renderWithReduxProvider(<DocMarkdown source={markdown} specKey={'4.0'} />)
     expect(screen.getByText('const noop = () => null')).toBeInTheDocument()
   })
 })

package/src/components/DocMarkdown/utils.spec.ts

@@ -27,25 +27,25 @@ import { remapHashURL, transformURL } from './utils'
 
 describe('DocMarkdown utils', () => {
   describe('hashbang url remapping', () => {
-    test.each(['#!/methodTag', '#!/3.1/methodTag'])(
+    test.each(['#!/methodTag', '#!/4.0/methodTag'])(
       'it correctly maps %s tag urls',
       (url) => {
-        const result = remapHashURL('3.1', url)
-        expect(result).toEqual('/3.1/methods/methodTag')
+        const result = remapHashURL('4.0', url)
+        expect(result).toEqual('/4.0/methods/methodTag')
       }
     )
 
-    test.each(['#!/3.1/methodTag/methodName', '#!/methodTag/methodName'])(
+    test.each(['#!/4.0/methodTag/methodName', '#!/methodTag/methodName'])(
       'it correctly maps %s method urls ',
       (hashbangUrl) => {
-        const result = remapHashURL('3.1', hashbangUrl)
-        expect(result).toEqual('/3.1/methods/methodTag/methodName')
+        const result = remapHashURL('4.0', hashbangUrl)
+        expect(result).toEqual('/4.0/methods/methodTag/methodName')
       }
     )
 
     test('external urls are left untouched', () => {
       const url = 'https://foo.com'
-      const result = remapHashURL('3.1', url)
+      const result = remapHashURL('4.0', url)
       expect(result).toEqual(url)
     })
   })
@@ -53,18 +53,18 @@ describe('DocMarkdown utils', () => {
   describe('url transforming', () => {
     test('removes mark tags and transforms url', () => {
       const url = '#!/<mark>Dashboard</mark>/create_<mark>dashboard</mark>'
-      const result = transformURL('3.1', url)
-      expect(result).toEqual('/3.1/methods/Dashboard/create_dashboard')
+      const result = transformURL('4.0', url)
+      expect(result).toEqual('/4.0/methods/Dashboard/create_dashboard')
     })
     test('removes mark tags and transforms url', () => {
       const url = 'https://docs.looker.com/r/api/<mark>authorization</mark>'
-      const result = transformURL('3.1', url)
+      const result = transformURL('4.0', url)
       expect(result).toEqual('https://docs.looker.com/r/api/authorization')
     })
     test('removes escaped mark tags and transforms url', () => {
       const url =
         'https://docs.looker.com/r/api/%3Cmark%3Eauthorization%3Cmark%3E'
-      const result = transformURL('3.1', url)
+      const result = transformURL('4.0', url)
       expect(result).toEqual('https://docs.looker.com/r/api/authorization')
     })
   })

package/src/components/DocReferences/DocReferences.spec.tsx

@@ -45,7 +45,7 @@ describe('DocReferences', () => {
         typesUsed={typesUsed}
         typesUsedBy={typesUsedBy}
         methodsUsedBy={methodsUsedBy}
-        specKey={'3.1'}
+        specKey={'4.0'}
         api={api}
       />
     )
@@ -54,18 +54,18 @@ describe('DocReferences', () => {
     )
     expect(screen.getByText(typesUsed[0].name).closest('a')).toHaveAttribute(
       'href',
-      buildPath(api, typesUsed[0], '3.1')
+      buildPath(api, typesUsed[0], '4.0')
     )
 
     expect(typesUsedBy).toHaveLength(1)
     expect(typesUsedBy[0].name).toEqual('Dashboard')
     expect(screen.getByText(typesUsedBy[0].name).closest('a')).toHaveAttribute(
       'href',
-      buildPath(api, typesUsedBy[0], '3.1')
+      buildPath(api, typesUsedBy[0], '4.0')
     )
     expect(
       screen.getByText(methodsUsedBy[0].name).closest('a')
-    ).toHaveAttribute('href', buildPath(api, methodsUsedBy[0], '3.1'))
+    ).toHaveAttribute('href', buildPath(api, methodsUsedBy[0], '4.0'))
   })
 
   test('it highlights text matching search pattern', () => {
@@ -76,7 +76,7 @@ describe('DocReferences', () => {
     renderWithRouterAndReduxProvider(
       <DocReferences
         typesUsed={[api.types.Dashboard]}
-        specKey={'3.1'}
+        specKey={'4.0'}
         api={api}
       />,
       undefined,
@@ -86,7 +86,7 @@ describe('DocReferences', () => {
     expect(foundRef).toContainHTML('<span class="hi">Dash</span>board')
     expect(foundRef).toHaveAttribute(
       'href',
-      buildPath(api, api.types.Dashboard, '3.1')
+      buildPath(api, api.types.Dashboard, '4.0')
     )
   })
 })

package/src/components/ExploreType/ExploreType.spec.tsx

@@ -76,18 +76,6 @@ describe('ExploreType', () => {
   })
 
   describe('ExploreTypeLink', () => {
-    test('recognizes 3.1', () => {
-      const specKey = '3.1'
-      const path = `/${specKey}/methods/foo`
-      renderWithRouter(<ExploreTypeLink api={api} type={targetType} />, [path])
-
-      const actual = screen.getByText(targetType.jsonName)
-      expect(actual).toBeInTheDocument()
-      expect(actual).toHaveProperty(
-        'href',
-        `http://localhost/${specKey}/types/ColorCollection/${targetType.jsonName}`
-      )
-    })
     test('recognizes 4.0', () => {
       const specKey = '4.0'
       const path = `/${specKey}/methods/foo`

package/src/components/SelectorContainer/ApiSpecSelector.spec.tsx

@@ -83,10 +83,10 @@ describe('ApiSpecSelector', () => {
         Object.keys(specs).length
       )
     })
-    const button = screen.getByText('3.1')
+    const button = screen.getByText('4.0')
     userEvent.click(button)
     expect(push).toHaveBeenCalledWith({
-      pathname: '/3.1/methods/Dashboard/dashboard',
+      pathname: '/4.0/methods/Dashboard/dashboard',
       search: '',
     })
   })

package/src/components/SideNav/SideNav.spec.tsx

@@ -49,7 +49,7 @@ describe('SideNav', () => {
     saveLocation = globalThis.window.location
     window.location = {
       ...saveLocation,
-      pathname: '/3.1',
+      pathname: '/4.0',
     }
   })
 
@@ -73,7 +73,7 @@ describe('SideNav', () => {
   })
 
   test('Methods tab is the default active tab', () => {
-    renderWithRouterAndReduxProvider(<SideNav spec={spec} />, ['/3.1/methods'])
+    renderWithRouterAndReduxProvider(<SideNav spec={spec} />, ['/4.0/methods'])
     expect(screen.getAllByText(allTagsPattern)).toHaveLength(2)
     expect(
       screen.queryAllByRole('link', { name: allTypesPattern })
@@ -85,7 +85,7 @@ describe('SideNav', () => {
   })
 
   test('url determines active tab', () => {
-    renderWithRouterAndReduxProvider(<SideNav spec={spec} />, ['/3.1/types'])
+    renderWithRouterAndReduxProvider(<SideNav spec={spec} />, ['/4.0/types'])
     // eslint-disable-next-line jest-dom/prefer-in-document
     expect(screen.queryAllByText(allTagsPattern)).toHaveLength(2)
   })
@@ -105,13 +105,13 @@ jest.mock('react-router-dom', () => {
 
 describe('Search', () => {
   test('inputting text in search box updates URL', async () => {
-    renderWithRouterAndReduxProvider(<SideNav spec={spec} />, ['/3.1/methods'])
+    renderWithRouterAndReduxProvider(<SideNav spec={spec} />, ['/4.0/methods'])
     const searchPattern = 'embedsso'
     const input = screen.getByLabelText('Search')
     await userEvent.paste(input, searchPattern)
     await waitFor(() => {
       expect(mockHistoryPush).toHaveBeenCalledWith({
-        pathname: '/3.1/methods',
+        pathname: '/4.0/methods',
         search: `s=${searchPattern}`,
       })
     })
@@ -125,7 +125,7 @@ describe('Search', () => {
     jest.spyOn(spec.api!, 'search')
     renderWithRouterAndReduxProvider(
       <SideNav spec={spec} />,
-      ['/3.1/methods?s=embedsso'],
+      ['/4.0/methods?s=embedsso'],
       store
     )
     const input = screen.getByLabelText('Search')

package/src/components/SideNav/SideNavMethodTags.spec.tsx

@@ -36,7 +36,7 @@ describe('SideNavTags', () => {
   const tags = pick(api.tags, ['ApiAuth', 'Dashboard'])
   test('it renders a provided tag and its methods', () => {
     renderWithRouterAndReduxProvider(
-      <SideNavMethodTags tags={tags} specKey={'3.1'} />
+      <SideNavMethodTags tags={tags} specKey={'4.0'} />
     )
     const tag = screen.getByText('Dashboard')
     const tagContent = 'Create Dashboard'
@@ -49,7 +49,7 @@ describe('SideNavTags', () => {
 
   test('tags are rendered initially collapsed and expand when clicked', () => {
     renderWithRouterAndReduxProvider(
-      <SideNavMethodTags tags={tags} specKey={'3.1'} />
+      <SideNavMethodTags tags={tags} specKey={'4.0'} />
     )
 
     const allTags = screen.getAllByText(/ApiAuth|Dashboard/)
@@ -63,8 +63,8 @@ describe('SideNavTags', () => {
 
   test('tag is expanded if specified in route', () => {
     renderWithRouterAndReduxProvider(
-      <SideNavMethodTags tags={tags} specKey={'3.1'} />,
-      ['/3.1/methods/Dashboard']
+      <SideNavMethodTags tags={tags} specKey={'4.0'} />,
+      ['/4.0/methods/Dashboard']
     )
 
     const allTags = screen.getAllByText(/^(ApiAuth|Dashboard)$/)

package/src/components/SideNav/SideNavMethods.spec.tsx

@@ -50,7 +50,7 @@ jest.mock('react-router-dom', () => {
 describe('SideNavMethods', () => {
   const tag = 'Dashboard'
   const methods = api.tags[tag]
-  const specKey = '3.1'
+  const specKey = '4.0'
 
   beforeEach(() => {
     jest.clearAllMocks()

package/src/components/SideNav/SideNavTypeTags.spec.tsx

@@ -37,7 +37,7 @@ describe('SideNavTypeTags', () => {
   const tags = pick(api.typeTags, ['ApiAuth', 'Dashboard'])
   test('it renders a provided tag and its methods', () => {
     renderWithRouterAndReduxProvider(
-      <SideNavTypeTags tags={tags} specKey={'3.1'} />
+      <SideNavTypeTags tags={tags} specKey={'4.0'} />
     )
     const tag = screen.getByText('Dashboard')
     const tagContent = 'CreateDashboardFilter'
@@ -50,7 +50,7 @@ describe('SideNavTypeTags', () => {
 
   test('tags are rendered initially collapsed and expand when clicked', () => {
     renderWithRouterAndReduxProvider(
-      <SideNavTypeTags tags={tags} specKey={'3.1'} />
+      <SideNavTypeTags tags={tags} specKey={'4.0'} />
     )
 
     const allTags = screen.getAllByText(/ApiAuth|Dashboard/)
@@ -65,8 +65,8 @@ describe('SideNavTypeTags', () => {
   test('tag is expanded if specified in route', () => {
     const tags = pick(api.typeTags, ['ApiAuth', 'DataAction'])
     renderWithRouterAndReduxProvider(
-      <SideNavTypeTags tags={tags} specKey={'3.1'} />,
-      ['/3.1/types/DataAction']
+      <SideNavTypeTags tags={tags} specKey={'4.0'} />,
+      ['/4.0/types/DataAction']
     )
 
     const allTags = screen.getAllByText(/^(ApiAuth|DataAction)$/)

package/src/components/SideNav/SideNavTypes.spec.tsx

@@ -48,7 +48,7 @@ jest.mock('react-router-dom', () => {
 
 describe('SideNavTypes', () => {
   const tag = 'Dashboard'
-  const specKey = '3.1'
+  const specKey = '4.0'
   const typeTags = Object.keys(api.typeTags[tag])
 
   test('it renders provided types', () => {

package/src/scenes/DiffScene/DocDiff/DocDiff.spec.tsx

@@ -32,7 +32,7 @@ import { diffSpecs, standardDiffToggles } from '../diffUtils'
 import { renderWithReduxProvider } from '../../../test-utils'
 import { DocDiff } from './DocDiff'
 
-describe('DocDiff', () => {
+describe.skip('DocDiff', () => {
   const specs = getLoadedSpecs()
   const leftKey = specs['3.1'].key
   const rightKey = specs['4.0'].key

package/src/state/specs/sagas.ts

@@ -49,6 +49,9 @@ function* initSaga(action: PayloadAction<InitSpecsAction>) {
     specs[currentSpecKey] = spec
     yield* put(initSpecsSuccessAction({ specs, currentSpecKey }))
   } catch (error: any) {
+    // TODO if an error occurs here extension api explorer hangs.
+    // Needs to be fixed. For now report the error.
+    console.error(error)
     yield* put(initSpecsFailureAction(new Error(error.message)))
   }
 }
@@ -70,6 +73,7 @@ function* setCurrentSpecSaga(action: PayloadAction<SetCurrentSpecAction>) {
       })
     )
   } catch (error: any) {
+    console.error(error)
     yield put(setCurrentSpecFailureAction(new Error(error.message)))
   }
 }