Address some small comments

scality · Mar 20, 2024 · 8a0fd3b · 8a0fd3b
1 parent cf8aecd
commit 8a0fd3b
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 7 deletions.
diff --git a/src/react/ui-elements/Veeam/VeeamConfiguration.test.tsx b/src/react/ui-elements/Veeam/VeeamConfiguration.test.tsx
@@ -21,9 +21,7 @@ jest.mock('../../next-architecture/domain/business/accounts', () => ({
 
 const mockUseStepper = useStepper as jest.Mock;
 const mockUseAccountsLocationsAndEndpoints =
-  useAccountsLocationsAndEndpoints as jest.MockedFunction<
-    typeof useAccountsLocationsAndEndpoints
-  >;
+  useAccountsLocationsAndEndpoints as jest.Mock;
 describe('Veeam Configuration UI', () => {
   const selectors = {
     accountNameInput: () => screen.getByLabelText(/account/i),
@@ -40,7 +38,6 @@ describe('Veeam Configuration UI', () => {
   };
 
   const mockUseAccountsImplementation = () => {
-    //@ts-ignore
     mockUseAccountsLocationsAndEndpoints.mockImplementation(() => {
       return {
         accountsLocationsAndEndpoints: {
@@ -174,7 +171,6 @@ describe('Veeam Configuration UI', () => {
 
   it('should prefilled with Veeam account is there is no account', async () => {
     //S
-    //@ts-ignore
     mockUseAccountsLocationsAndEndpoints.mockImplementation(() => {
       return {
         accountsLocationsAndEndpoints: {

diff --git a/src/react/ui-elements/Veeam/VeeamConfiguration.tsx b/src/react/ui-elements/Veeam/VeeamConfiguration.tsx
@@ -171,7 +171,7 @@ const Configuration = () => {
       status === 'success' &&
       accounts.some((account) => account.name === accountName)
     );
-  }, [accountName, status]);
+  }, [accountName, status, accounts]);
 
   const onSubmit = ({
     capacity,

diff --git a/src/react/ui-elements/Veeam/VeeamSummary.tsx b/src/react/ui-elements/Veeam/VeeamSummary.tsx
@@ -88,7 +88,7 @@ export const VeeamSummary = ({
       <Text isEmphazed>
         Your ARTESCA is now configured and ready to integrate with Veeam. <br />
         The next steps involve managing Certificates and entering specific
-        PRODUCT details within the Veeam application
+        ARTESCA details within the Veeam application
       </Text>
       {isPlatformAdmin ? (
         <Level4FormSection title={{ name: CERTIFICATE_SECTION_TITLE }}>