@verdaccio/ui-components 3.0.0-next-7.7 → 3.0.0-next-7.9

package/build/store/models/manifest.js

@@ -6,7 +6,9 @@ Object.defineProperty(exports, "__esModule", {
 exports.manifest = void 0;
 var _core = require("@rematch/core");
 var _api = _interopRequireDefault(require("../api"));
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+var _routes = require("./routes");
+var _utils = require("./utils");
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 function isPackageVersionValid(packageMeta, packageVersion) {
   if (!packageVersion || typeof packageVersion === 'undefined') {
     // if is undefined, that means versions does not exist, we continue
@@ -87,16 +89,16 @@ const manifest = exports.manifest = (0, _core.createModel)()({
       packageName,
       packageVersion
     }, state) {
-      const basePath = state.configuration.config.base;
+      const basePath = (0, _utils.stripTrailingSlash)(state.configuration.config.base);
       try {
-        const manifest = await _api.default.request(`${basePath}-/verdaccio/data/sidebar/${packageName}${packageVersion ? `?v=${packageVersion}` : ''}`);
+        const manifest = await _api.default.request(`${basePath}${_routes.APIRoute.SIDEBAR}${packageName}${packageVersion ? `?v=${packageVersion}` : ''}`);
 
         // FIXME: update types accordingly
         // @ts-ignore
         if (!isPackageVersionValid(manifest, packageVersion)) {
           throw new Error('not found');
         }
-        const readme = await _api.default.request(`${basePath}-/verdaccio/data/package/readme/${packageName}${packageVersion ? `?v=${packageVersion}` : ''}`, 'GET');
+        const readme = await _api.default.request(`${basePath}${_routes.APIRoute.README}${packageName}${packageVersion ? `?v=${packageVersion}` : ''}`);
         dispatch.manifest.saveManifest({
           packageName,
           packageVersion,

package/build/store/models/packages.js

@@ -6,7 +6,9 @@ Object.defineProperty(exports, "__esModule", {
 exports.packages = void 0;
 var _core = require("@rematch/core");
 var _api = _interopRequireDefault(require("../api"));
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+var _routes = require("./routes");
+var _utils = require("./utils");
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 /**
  *
  * @category Model
@@ -25,9 +27,9 @@ const packages = exports.packages = (0, _core.createModel)()({
   },
   effects: dispatch => ({
     async getPackages(_payload, state) {
-      const basePath = state.configuration.config.base;
+      const basePath = (0, _utils.stripTrailingSlash)(state.configuration.config.base);
       try {
-        const payload = await _api.default.request(`${basePath}-/verdaccio/data/packages`, 'GET');
+        const payload = await _api.default.request(`${basePath}${_routes.APIRoute.PACKAGES}`);
         dispatch.packages.savePackages(payload);
       } catch (error) {
         // eslint-disable-next-line no-console

package/build/store/models/routes.js

@@ -0,0 +1,18 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.APIRoute = void 0;
+// Example API request:
+// http://localhost:8000/-/verdaccio/data/package/readme/jquery
+let APIRoute = exports.APIRoute = /*#__PURE__*/function (APIRoute) {
+  APIRoute["LOGIN"] = "/-/verdaccio/sec/login";
+  APIRoute["CONFIG"] = "/-/verdaccio/packages";
+  APIRoute["PACKAGES"] = "/-/verdaccio/data/packages";
+  APIRoute["SEARCH"] = "/-/verdaccio/data/search/";
+  APIRoute["SIDEBAR"] = "/-/verdaccio/data/sidebar/";
+  APIRoute["README"] = "/-/verdaccio/data/package/readme/";
+  return APIRoute;
+}({}); // :packageName?v=version
+//# sourceMappingURL=routes.js.map

package/build/store/models/search.js

@@ -7,7 +7,9 @@ exports.search = void 0;
 var _core = require("@rematch/core");
 var _orderBy = _interopRequireDefault(require("lodash/orderBy"));
 var _api = _interopRequireDefault(require("../api"));
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+var _routes = require("./routes");
+var _utils = require("./utils");
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 const CONSTANTS = {
   API_DELAY: 300,
   ABORT_ERROR: 'AbortError'
@@ -21,7 +23,7 @@ const search = exports.search = (0, _core.createModel)()({
     clearRequestQueue(state) {
       const controllers = state.controller;
       controllers.forEach(request => {
-        request === null || request === void 0 ? void 0 : request.abort();
+        request === null || request === void 0 || request.abort();
       });
       return {
         ...state,
@@ -57,7 +59,7 @@ const search = exports.search = (0, _core.createModel)()({
     async getSuggestions({
       value
     }, state) {
-      const basePath = state.configuration.config.base;
+      const basePath = (0, _utils.stripTrailingSlash)(state.configuration.config.base);
       try {
         const controller = new window.AbortController();
         dispatch.search.addControllerToQueue({
@@ -66,7 +68,7 @@ const search = exports.search = (0, _core.createModel)()({
         const signal = controller.signal;
         // https://developer.mozilla.org/en-US/docs/Web/API/Fetch_API#Browser_compatibility
         // FUTURE: signal is not well supported for IE and Samsung Browser
-        const suggestions = await _api.default.request(`${basePath}-/verdaccio/data/search/${encodeURIComponent(value)}`, 'GET', {
+        const suggestions = await _api.default.request(`${basePath}${_routes.APIRoute.SEARCH}${encodeURIComponent(value)}`, 'GET', {
           signal,
           headers: {}
         });

package/build/store/models/utils.js

@@ -0,0 +1,10 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.stripTrailingSlash = stripTrailingSlash;
+function stripTrailingSlash(url) {
+  return url.replace(/\/$/, '');
+}
+//# sourceMappingURL=utils.js.map

package/build/store/storage.js

@@ -5,7 +5,7 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 var _localstorageMemory = _interopRequireDefault(require("localstorage-memory"));
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 let storage;
 try {
   localStorage.setItem('__TEST__', '');

package/build/store/store.js

@@ -7,7 +7,7 @@ exports.store = void 0;
 var _core = require("@rematch/core");
 var _loading = _interopRequireDefault(require("@rematch/loading"));
 var _models = require("./models");
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 const store = exports.store = (0, _core.init)({
   models: _models.models,
   plugins: [(0, _loading.default)()]

package/build/test/i18n-config.js

@@ -6,7 +6,7 @@ Object.defineProperty(exports, "__esModule", {
 exports.default = void 0;
 var _i18next = _interopRequireDefault(require("i18next"));
 var _reactI18next = require("react-i18next");
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 _i18next.default.use(_reactI18next.initReactI18next).init({
   lng: 'en-US',
   whitelist: ['en-US'],

package/build/test/test-react-testing-library.js

@@ -29,7 +29,7 @@ var _Theme = require("../Theme");
 var _AppConfigurationProvider = _interopRequireDefault(require("../providers/AppConfigurationProvider"));
 var _PersistenceSettingProvider = _interopRequireDefault(require("../providers/PersistenceSettingProvider"));
 var _i18nConfig = _interopRequireDefault(require("./i18n-config"));
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 const renderWithStore = (ui, store) => (0, _react.render)(ui, {
   wrapper: ({
     children

package/build/utils/index.js

@@ -69,7 +69,7 @@ Object.keys(_token).forEach(function (key) {
     }
   });
 });
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 function _getRequireWildcardCache(e) { if ("function" != typeof WeakMap) return null; var r = new WeakMap(), t = new WeakMap(); return (_getRequireWildcardCache = function (e) { return e ? t : r; })(e); }
-function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != typeof e && "function" != typeof e) return { default: e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && Object.prototype.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n.default = e, t && t.set(e, n), n; }
+function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != typeof e && "function" != typeof e) return { default: e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && {}.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n.default = e, t && t.set(e, n), n; }
 //# sourceMappingURL=index.js.map

package/build/utils/loadable.js

@@ -6,7 +6,7 @@ Object.defineProperty(exports, "__esModule", {
 exports.default = void 0;
 var _react = _interopRequireWildcard(require("react"));
 function _getRequireWildcardCache(e) { if ("function" != typeof WeakMap) return null; var r = new WeakMap(), t = new WeakMap(); return (_getRequireWildcardCache = function (e) { return e ? t : r; })(e); }
-function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != typeof e && "function" != typeof e) return { default: e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && Object.prototype.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n.default = e, t && t.set(e, n), n; }
+function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != typeof e && "function" != typeof e) return { default: e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && {}.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n.default = e, t && t.set(e, n), n; }
 /* eslint-disable verdaccio/jsx-spread */
 /**
  *  With the combination of a bundler, enable to code split a package.

package/build/utils/routes.js

@@ -6,6 +6,7 @@ Object.defineProperty(exports, "__esModule", {
 exports.Route = void 0;
 let Route = exports.Route = /*#__PURE__*/function (Route) {
   Route["ROOT"] = "/";
+  Route["DETAIL"] = "/-/web/detail/";
   Route["SCOPE_PACKAGE"] = "/-/web/detail/@:scope/:package";
   Route["SCOPE_PACKAGE_VERSION"] = "/-/web/detail/@:scope/:package/v/:version";
   Route["PACKAGE"] = "/-/web/detail/:package";

package/build/utils/token.js

@@ -7,7 +7,7 @@ exports.isTokenExpire = isTokenExpire;
 var _jsBase = require("js-base64");
 var _isNumber = _interopRequireDefault(require("lodash/isNumber"));
 var _isString = _interopRequireDefault(require("lodash/isString"));
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 function isTokenExpire(token) {
   if (!(0, _isString.default)(token)) {
     return true;

package/build/utils/url.js

@@ -9,7 +9,7 @@ exports.isEmail = isEmail;
 exports.isURL = isURL;
 var _isEmail = _interopRequireDefault(require("validator/lib/isEmail"));
 var _isURL = _interopRequireDefault(require("validator/lib/isURL"));
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 function isURL(url) {
   return (0, _isURL.default)(url || '', {
     protocols: ['http', 'https', 'git+https', 'git'],

package/build/utils/utils.js

@@ -12,12 +12,13 @@ exports.formatRepository = formatRepository;
 exports.getAuthorName = getAuthorName;
 exports.getLastUpdatedPackageTime = getLastUpdatedPackageTime;
 exports.getRecentReleases = getRecentReleases;
+exports.getUplink = getUplink;
 var _dayjs = _interopRequireDefault(require("dayjs"));
 var _localizedFormat = _interopRequireDefault(require("dayjs/plugin/localizedFormat"));
 var _relativeTime = _interopRequireDefault(require("dayjs/plugin/relativeTime"));
 var _i18next = _interopRequireDefault(require("i18next"));
 var _isString = _interopRequireDefault(require("lodash/isString"));
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 const TIMEFORMAT = exports.TIMEFORMAT = 'L LTS';
 _dayjs.default.extend(_relativeTime.default);
 _dayjs.default.extend(_localizedFormat.default);
@@ -96,7 +97,25 @@ function getAuthorName(authorName) {
   }
   return authorName;
 }
+function getUplink(upLinkName, packageName) {
+  // TODO: make this a config like "uplinks: npmjs: web: https://www.npmjs.com/package/"
+  switch (upLinkName) {
+    case 'npmjs':
+      return `https://www.npmjs.com/package/${packageName}`;
+  }
+  return null;
+}
 function fileSizeSI(a, b, c, d, e) {
-  return (b = Math, c = b.log, d = 1e3, e = c(a) / c(d) | 0, a / b.pow(d, e)).toFixed(2) + ' ' + (e ? 'kMGTPEZY'[--e] + 'B' : 'Bytes');
+  b = Math;
+  c = b.log;
+  d = 1e3;
+  e = c(a) / c(d) | 0;
+  let size = a / b.pow(d, e);
+  // no decimals for Bytes
+  if (e === 0) {
+    return Math.floor(size) + ' Bytes';
+  } else {
+    return size.toFixed(1) + ' ' + 'kMGTPEZY'[--e] + 'B';
+  }
 }
 //# sourceMappingURL=utils.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@verdaccio/ui-components",
-  "version": "3.0.0-next-7.7",
+  "version": "3.0.0-next-7.9",
   "description": "theme ui component",
   "author": "Juan Picado <juanpicado19@gmail.com>",
   "license": "MIT",
@@ -8,8 +8,8 @@
   "main": "./build/index.js",
   "types": "build/index.d.ts",
   "dependencies": {
-    "@emotion/react": "^11.10.6",
-    "@emotion/styled": "^11.10.6",
+    "@emotion/react": "11.10.6",
+    "@emotion/styled": "11.10.6",
     "@fontsource/material-icons": "^4.5.4",
     "@fontsource/roboto": "^4.5.8",
     "@mui/icons-material": "5.15.6",
@@ -33,7 +33,7 @@
     "normalize.css": "8.0.1",
     "react": "18.2.0",
     "react-dom": "18.2.0",
-    "react-hook-form": "7.49.3",
+    "react-hook-form": "7.50.1",
     "react-i18next": "13.5.0",
     "react-json-view": "1.21.3",
     "react-markdown": "9.0.1",
@@ -46,8 +46,8 @@
     "validator": "13.11.0"
   },
   "devDependencies": {
-    "@babel/core": "^7.23.0",
-    "@emotion/babel-plugin": "11.10.6",
+    "@babel/core": "7.24.9",
+    "@emotion/babel-plugin": "11.11.0",
     "@emotion/jest": "11.11.0",
     "@storybook/addon-actions": "^6.5.15",
     "@storybook/addon-essentials": "^6.5.15",
@@ -63,8 +63,8 @@
     "@types/hast": "^2.0.0",
     "@types/react-router": "^5.1.20",
     "@types/unist": "^2.0.0",
-    "@verdaccio/types": "12.0.0-next-7.4",
-    "babel-loader": "^8.3.0",
+    "@verdaccio/types": "12.0.0-next-7.5",
+    "babel-loader": "8.3.0",
     "mockdate": "3.0.5",
     "msw": "0.49.3",
     "typescript": "4.9.5"

package/src/AppTest/AppRoute.test.tsx

@@ -0,0 +1,65 @@
+import React from 'react';
+import { MemoryRouter } from 'react-router';
+
+import { store } from '../';
+import { act, renderWithStore, screen, waitFor } from '../test/test-react-testing-library';
+import AppRoute from './AppRoute';
+
+// force the windows to expand to display items
+// https://github.com/bvaughn/react-virtualized/issues/493#issuecomment-640084107
+jest.spyOn(HTMLElement.prototype, 'offsetHeight', 'get').mockReturnValue(600);
+jest.spyOn(HTMLElement.prototype, 'offsetWidth', 'get').mockReturnValue(600);
+
+function appTest(path: string) {
+  renderWithStore(
+    <MemoryRouter initialEntries={[path]}>
+      <AppRoute />
+    </MemoryRouter>,
+    store
+  );
+}
+
+// See jest/server-handlers.ts for test routes
+describe('AppRoute', () => {
+  afterEach(() => {
+    jest.clearAllMocks();
+  });
+
+  test('renders Front component for ROOT path', async () => {
+    act(() => appTest('/'));
+    await waitFor(() => expect(screen.getByTestId('loading')).toBeInTheDocument());
+    await waitFor(() => expect(screen.getAllByTestId('package-item-list')).toHaveLength(5));
+  });
+
+  test('renders VersionPage component for PACKAGE path', async () => {
+    act(() => appTest('/-/web/detail/jquery'));
+    await waitFor(() => screen.getByTestId('readme-tab'));
+    expect(screen.getByTestId('sidebar')).toBeInTheDocument();
+  });
+
+  test('renders VersionPage component for PACKAGE VERSION path', async () => {
+    act(() => appTest('/-/web/detail/jquery/v/3.6.3'));
+    await waitFor(() => screen.getByTestId('readme-tab'));
+    expect(screen.getByTestId('sidebar')).toBeInTheDocument();
+  });
+
+  test('renders Forbidden component for not allowed PACKAGE', async () => {
+    act(() => appTest('/-/web/detail/JSONstream'));
+    await waitFor(() => screen.getByTestId('not-found-go-to-home-button'));
+  });
+
+  test('renders NotFound component for missing PACKAGE', async () => {
+    act(() => appTest('/-/web/detail/kleur'));
+    await waitFor(() => screen.getByTestId('not-found-go-to-home-button'));
+  });
+
+  test('renders NotFound component for non-existing PACKAGE VERSION', async () => {
+    act(() => appTest('/-/web/detail/jquery/v/0.9.9'));
+    await waitFor(() => screen.getByTestId('not-found-go-to-home-button'));
+  });
+
+  test('renders NotFound component for non-matching path', async () => {
+    act(() => appTest('/oiccadrev'));
+    await waitFor(() => screen.getByTestId('not-found-go-to-home-button'));
+  });
+});

package/src/Theme/ThemeProvider.test.tsx

@@ -0,0 +1,27 @@
+import { render, screen } from '@testing-library/react';
+import React from 'react';
+
+import {
+  AppConfigurationProvider,
+  PersistenceSettingProvider,
+  StyleBaseline,
+  ThemeProvider,
+} from '../';
+
+const AppContainer = () => (
+  <AppConfigurationProvider>
+    <ThemeProvider>
+      <StyleBaseline />
+      <PersistenceSettingProvider>
+        <div>{'Theme'}</div>
+      </PersistenceSettingProvider>
+    </ThemeProvider>
+  </AppConfigurationProvider>
+);
+
+describe('ThemeProvider', () => {
+  test('should render with theme', async () => {
+    render(<AppContainer />);
+    await screen.findByText('Theme');
+  });
+});

package/src/Theme/theme.ts

@@ -6,11 +6,11 @@ const colors = {
   black: '#000',
   white: '#fff',
   red: '#d32f2f',
-  orange: '#CD4000',
+  orange: '#cd4000',
   greySuperLight: '#f5f5f5',
   greyLight: '#d3d3d3',
   greyLight2: '#908ba1',
-  greyLight3: '#f3f4f240',
+  greyLight3: '#f3f4f2',
   greyDark: '#a9a9a9',
   greyDark2: '#586069',
   greyChateau: '#95989a',
@@ -38,7 +38,7 @@ const themeModes = {
     ...colors,
     primary: '#ffffff',
     secondary: '#424242',
-    background: '#1A202C',
+    background: '#1a202c',
   },
 };
 

package/src/components/ActionBar/ActionBar.test.tsx

@@ -1,28 +1,22 @@
 import React from 'react';
 
 import { store } from '../../store/store';
-import {
-  cleanup,
-  fireEvent,
-  renderWithStore,
-  screen,
-  waitFor,
-} from '../../test/test-react-testing-library';
+import { cleanup, fireEvent, renderWithStore, screen } from '../../test/test-react-testing-library';
 import ActionBar from './ActionBar';
 
 const defaultPackageMeta = {
   _uplinks: {},
   latest: {
-    name: 'verdaccio-ui/local-storage',
-    version: '8.0.1-next.1',
+    name: 'verdaccio',
+    version: '1.0.0',
     dist: {
-      fileCount: 0,
-      unpackedSize: 0,
-      tarball: 'http://localhost:8080/bootstrap/-/bootstrap-4.3.1.tgz',
+      fileCount: 1,
+      unpackedSize: 171,
+      tarball: 'http://localhost:9000/verdaccio/-/verdaccio-1.0.0.tgz',
     },
     homepage: 'https://verdaccio.org',
     bugs: {
-      url: 'https://github.com/verdaccio/monorepo/issues',
+      url: 'https://github.com/verdaccio/verdaccio/issues',
     },
   },
 };
@@ -39,6 +33,12 @@ describe('<ActionBar /> component', () => {
     expect(screen.getByTestId('HomeIcon')).toBeInTheDocument();
   });
 
+  test('should not render if data is missing', () => {
+    // @ts-ignore - testing with missing data
+    renderWithStore(<ActionBar packageMeta={undefined} />, store);
+    expect(screen.queryByTestId('HomeIcon')).toBeNull();
+  });
+
   test('when there is no action bar data', () => {
     const packageMeta = {
       ...defaultPackageMeta,
@@ -64,6 +64,14 @@ describe('<ActionBar /> component', () => {
     expect(screen.getByLabelText('action-bar-action.download-tarball')).toBeTruthy();
   });
 
+  test('when button to download is disabled', () => {
+    renderWithStore(
+      <ActionBar packageMeta={defaultPackageMeta} showDownloadTarball={false} />,
+      store
+    );
+    expect(screen.queryByTestId('download-tarball-btn')).not.toBeInTheDocument();
+  });
+
   test('when there is a button to raw manifest', () => {
     renderWithStore(<ActionBar packageMeta={defaultPackageMeta} showRaw={true} />, store);
     expect(screen.getByLabelText('action-bar-action.raw')).toBeTruthy();
@@ -71,15 +79,28 @@ describe('<ActionBar /> component', () => {
 
   test('when click button to raw manifest open a dialog with viewer', async () => {
     renderWithStore(<ActionBar packageMeta={defaultPackageMeta} showRaw={true} />, store);
+    expect(screen.queryByTestId('rawViewer--dialog')).toBeFalsy();
+
     fireEvent.click(screen.getByLabelText('action-bar-action.raw'));
-    await waitFor(() => expect(screen.getByTestId('rawViewer--dialog')).toBeInTheDocument());
+    await screen.findByTestId('rawViewer--dialog');
+
+    fireEvent.click(screen.getByTestId('close-raw-viewer'));
+    await screen.getByLabelText('action-bar-action.raw');
+
+    expect(screen.queryByTestId('rawViewer--dialog')).toBeFalsy();
   });
 
   test('should not display download tarball button', () => {
-    renderWithStore(<ActionBar packageMeta={defaultPackageMeta} showRaw={true} />, store);
+    renderWithStore(<ActionBar packageMeta={defaultPackageMeta} showRaw={false} />, store);
     expect(screen.queryByLabelText('Download tarball')).toBeFalsy();
   });
 
+  test('when click button to download ', async () => {
+    renderWithStore(<ActionBar packageMeta={defaultPackageMeta} showRaw={false} />, store);
+    fireEvent.click(screen.getByTestId('download-tarball-btn'));
+    await store.getState().loading.models.download;
+  });
+
   test('should not display show raw button', () => {
     renderWithStore(<ActionBar packageMeta={defaultPackageMeta} showRaw={false} />, store);
     expect(screen.queryByLabelText('action-bar-action.raw')).toBeFalsy();

package/src/components/ActionBar/ActionBar.tsx

@@ -41,7 +41,7 @@ const ActionBar: React.FC<Props> = ({ showRaw, showDownloadTarball = true, packa
   }
 
   return (
-    <Box alignItems="center" display="flex" marginBottom="14px">
+    <Box alignItems="center" display="flex" sx={{ my: 2 }}>
       <Stack direction="row" spacing={1}>
         {actions.map((action) => (
           <ActionBarAction key={action.type} {...action} />

package/src/components/ActionBar/ActionBarAction.tsx

@@ -12,11 +12,14 @@ import { useDispatch, useSelector } from 'react-redux';
 
 import { Theme } from '../../Theme';
 import { Dispatch, RootState } from '../../store/store';
-import { Link } from '../Link';
+import LinkExternal from '../LinkExternal';
 
 export const Fab = styled(FabMUI)<{ theme?: Theme }>(({ theme }) => ({
   backgroundColor:
     theme?.palette.mode === 'light' ? theme?.palette.primary.main : theme?.palette.cyanBlue,
+  '&:hover': {
+    color: theme?.palette.mode === 'light' ? theme?.palette.primary.main : theme?.palette.cyanBlue,
+  },
   color: theme?.palette.white,
 }));
 
@@ -42,21 +45,21 @@ const ActionBarAction: React.FC<ActionBarActionProps> = ({ type, link, action })
     case 'VISIT_HOMEPAGE':
       return (
         <Tooltip title={t('action-bar-action.visit-home-page') as string}>
-          <Link external={true} to={link} variant="button">
+          <LinkExternal to={link} variant="button">
             <Fab size="small">
               <HomeIcon />
             </Fab>
-          </Link>
+          </LinkExternal>
         </Tooltip>
       );
     case 'OPEN_AN_ISSUE':
       return (
         <Tooltip title={t('action-bar-action.open-an-issue') as string}>
-          <Link external={true} to={link} variant="button">
+          <LinkExternal to={link} variant="button">
             <Fab size="small">
               <BugReportIcon />
             </Fab>
-          </Link>
+          </LinkExternal>
         </Tooltip>
       );
     case 'DOWNLOAD_TARBALL':

package/src/components/Author/Author.test.tsx

@@ -1,6 +1,6 @@
 import React from 'react';
 
-import { cleanup, render } from '../../test/test-react-testing-library';
+import { cleanup, render, screen } from '../../test/test-react-testing-library';
 import { PackageMetaInterface } from '../../types/packageMeta';
 import Authors from './Author';
 
@@ -66,4 +66,10 @@ describe('<Author /> component', () => {
     const wrapper = render(withAuthorComponent(packageMeta));
     expect(wrapper).toMatchSnapshot();
   });
+
+  test('should not render if data is missing', () => {
+    // @ts-ignore - testing with missing data
+    render(withAuthorComponent(undefined));
+    expect(screen.queryByText('sidebar.author.title')).toBeNull();
+  });
 });