Merge pull request #165 from Arquisoft/150-increase-code-coverage

Added test to navbar
Arquisoft · Apr 28, 2024 · 52ef5aa · 52ef5aa
2 parents 2bda0de + 08796c1
commit 52ef5aa
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -45,4 +45,9 @@ jobs:
     - run: npm --prefix gatewayservice install
     - run: npm --prefix webapp install
     - run: npm --prefix webapp run build
-    - run: npm --prefix webapp run test:e2e
+    - run: npm --prefix webapp run test:e2e
+    - name: Analyze with SonarCloud
+      uses: sonarsource/sonarcloud-github-action@master
+      env:
+        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
diff --git a/webapp/src/components/fragments/NavBar.test.js b/webapp/src/components/fragments/NavBar.test.js
@@ -18,12 +18,15 @@ Cookies.set('user', JSON.stringify({ username: 'dummy' }))
 
 describe('NavBar fragment', () => {
 
-    it('shows the user name',async () => {
+    it('shows the user name and can log out',async () => {
         render(<MemoryRouter><Navbar style={{ width: '100%' }} /></MemoryRouter>);
         const text2 = await screen.findByText('dummy')
         expect(text2).toBeInTheDocument();
-        // Wait for questions to load
 
+        fireEvent.click(screen.getByText('dummy'));
+        expect(screen.getByText('navBar.logout')).toBeInTheDocument();
+        fireEvent.click(screen.getByText('navBar.logout'));
+        expect(screen.queryByText('dummy')).not.toBeInTheDocument();
     });
 
     test('Navbar renders correctly', async () => {