MMT-3915: Fixed when a user tries to publish a draft and is unsuccess…

…ful, there is no error message displayed

static/src/js/pages/DraftPage/DraftPage.jsx

@@ -68,7 +68,8 @@ const DraftPageHeader = () => {
 
  const {
  publishMutation,
- publishDraft
+ publishDraft,
+ error: publishErrors
  } = usePublishMutation(pluralize(derivedConceptType).toLowerCase())
 
  const toggleShowDeleteModal = (nextState) => {
@@ -121,6 +122,19 @@ const DraftPageHeader = () => {
  }
  }, [publishDraft])
 
+ useEffect(() => {
+ if (publishErrors) {
+ console.log('publishErrors', publishErrors)
+ const { message } = publishErrors
+ addNotification({
+ message,
+ variant: 'danger'
+ })
+
+ errorLogger(message, 'PublishMutation: publishMutation')
+ }
+ }, [publishErrors])
+
  const handleTemplate = async () => {
  const response = await createTemplate(providerId, mmtJwt, {
  TemplateName: '',

static/src/js/pages/DraftPage/__tests__/DraftPage.test.jsx

@@ -24,6 +24,7 @@ import createTemplate from '@/js/utils/createTemplate'
 import Providers from '@/js/providers/Providers/Providers'
 
 import DraftPage from '../DraftPage'
+import { GraphQLError } from 'graphql'
 
 vi.mock('@/js/components/MetadataPreview/MetadataPreview')
 vi.mock('@/js/utils/createTemplate')
@@ -341,6 +342,35 @@ describe('DraftPage', () => {
  expect(button).toHaveAttribute('disabled')
  })
 
+ test('calls the publish mutation and if unsuccesfull shows error', async () => {
+ const navigateSpy = vi.fn()
+ vi.spyOn(router, 'useNavigate').mockImplementation(() => navigateSpy)
+
+ const { user } = setup({
+ additionalMocks: [{
+ request: {
+ query: PUBLISH_DRAFT,
+ variables: {
+ draftConceptId: 'TD1000000-MMT',
+ nativeId: 'MMT_2331e312-cbbc-4e56-9d6f-fe217464be2c',
+ ummVersion: '1.2.0'
+ }
+ },
+ result: {
+ errors: [new GraphQLError('An error occurred')]
+ }
+ }]
+ })
+
+ const button = await screen.findByRole('button', { name: /Publish/ })
+ await user.click(button)
+
+ expect(navigateSpy).toHaveBeenCalledTimes(0)
+
+ expect(errorLogger).toHaveBeenCalledTimes(1)
+ expect(errorLogger).toHaveBeenCalledWith('An error occurred', 'PublishMutation: publishMutation')
+ })
+
  test('calls the publish mutation and navigates to the concept page', async () => {
  const navigateSpy = vi.fn()
  vi.spyOn(router, 'useNavigate').mockImplementation(() => navigateSpy)