Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add App.test.tsx and Sign.test.tsx and corrected formating #54

Merged
merged 17 commits into from
Mar 15, 2023

Conversation

kirubeltadesse
Copy link
Collaborator

@kirubeltadesse kirubeltadesse commented Mar 9, 2023

Description

This includes the unit test for the frontend, and to start with I have added the test for the App.tsx and SignIn.tsx files.
Fixes #51

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • [X ] This change requires a documentation update

How Has This Been Tested?

I ran the npm test to run all the test files inside the container. They all passed.

core/settings.py Outdated
@@ -154,3 +154,5 @@
# FIXME: Admin password and username
# Password: admin
# username: admin

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was added on the last comment. I am not sure why it is showing up on this one

core/urls.py Outdated
from django.urls import path, include

urlpatterns = [
path('', include('backend.urls')), # namespace='api' is not supported in Django 4.1
path('api/', include('api.urls')), # namespace='api' is not supported in Django 4.1
path('admin/', admin.site.urls),
path('docs/', include_docs_urls(title='Sap-API', description='This is the API for the Sap-web project. You will be able to find all the endpoints here.')),
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The same goes for this too:)

@kirubeltadesse kirubeltadesse merged commit 9f6b1c5 into Swarmies:develop Mar 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant