test: follow mike request - replace data-testid for querySelector by …

…classes on file ProjeCard.jsx
Codeminer42 · Sep 30, 2024 · 69e6613 · 69e6613
1 parent c9bddce
commit 69e6613
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/app/assets/javascripts/components/projects/ProjectCard.jsx b/app/assets/javascripts/components/projects/ProjectCard.jsx
@@ -80,7 +80,6 @@ export default class ProjectCard extends React.Component {
               <ul
                 className="dropdown-menu dropdown-menu-right"
                 aria-labelledby="dLabel"
-                data-testid="dropdown-menu-container"
               >
                 <li className="dropdown-header">{I18n.t('options')}</li>
 

diff --git a/spec/javascripts/components/projects/project_card_spec.js b/spec/javascripts/components/projects/project_card_spec.js
@@ -104,9 +104,9 @@ describe('<ProjectCard />', () => {
 
       describe('.dropdown-menu', () => {
         it('should contain dropdown-menu', () => {
-          const { getByTestId } = render(<ProjectCard {...defaultProps} />);
+          const { container } = render(<ProjectCard {...defaultProps} />);
 
-          expect(getByTestId('dropdown-menu-container')).toHaveClass(
+          expect(container.querySelector('.dropdown-menu')).toHaveClass(
             'dropdown-menu'
           );
         });
@@ -186,10 +186,10 @@ describe('<ProjectCard />', () => {
       describe('.dropdown-menu', () => {
         it('should not contain dropdown-menu', () => {
           defaultProps.joined = false;
-          const { queryByTestId } = render(<ProjectCard {...defaultProps} />);
+          const { container } = render(<ProjectCard {...defaultProps} />);
 
           expect(
-            queryByTestId('dropdown-menu-container')
+            container.querySelector('.dropdown-menu')
           ).not.toBeInTheDocument();
         });
       });