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

Manual Testing: Console testing on google chrome (User Story 2) #41

Open
wants to merge 70 commits into
base: f24
Choose a base branch
from

Conversation

lhitmi
Copy link

@lhitmi lhitmi commented Oct 19, 2024

During this sprint, we encountered challenges that led us to scrap the initial database concept. Instead, we adapted the code to handle data directly through the controller on the page. However, we still faced issues, which prompted us to conduct manual API testing to ensure everything was working.

In Google Chrome, using the Console tab, we executed the following command to check the GET API's response:
fetch('http://localhost:4568/api/admin/get-bug-log')
.then(res => res.json())
.then(console.log)

This command allowed us to observe how the GET API behaved. We received a response containing relevant information such as the uid, username, and login status, confirming that the data was being received. The results are shown in the screenshot below:

Screenshot 2024-10-20 at 12 13 56 AM

We also verified this through the Network tab when submitting the form with the data we entered, as shown in the following screenshot:

Screenshot 2024-10-20 at 12 13 56 AM

The GET request function can be further verified by the following screenshot showing that data is being fetched upon submission:

Screenshot 2024-10-20 at 12 46 49 AM

You can clearly see in the console that the submitted data was fetched.

We can also see it when running a command on the console as shown by this screenshot

Screenshot 2024-10-20 at 1 26 51 AM

this is the error we are receiving that is blocking the post
Screenshot 2024-10-20 at 2 53 56 PM

Contributors to this pull request:

  • Aisha Al Attiyah
  • Ahmad Al Obaidan
  • Latifa Al Hitmi

This pull request resolves issue #40

@lhitmi lhitmi added frontend main code is written in frontend backend main code is written in backend labels Oct 19, 2024
@lhitmi lhitmi added this to the Sprint 2 milestone Oct 19, 2024
@aattiyah aattiyah added High impact Features that have a high impact on the application High effort Features that need high effort to implement and removed High impact Features that have a high impact on the application labels Oct 19, 2024
@coveralls
Copy link

Pull Request Test Coverage Report for Build 11420912120

Details

  • 8 of 22 (36.36%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.03%) to 82.652%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/controllers/admin/dashboard.js 6 20 30.0%
Totals Coverage Status
Change from base Build 11057241668: -0.03%
Covered Lines: 22361
Relevant Lines: 25636

💛 - Coveralls

@aattiyah aattiyah changed the title Console testing on google chrome Manual Testing: Console testing on google chrome (User Story 2 Oct 19, 2024
@aattiyah aattiyah changed the title Manual Testing: Console testing on google chrome (User Story 2 Manual Testing: Console testing on google chrome (User Story 2) Oct 19, 2024
@aattiyah aattiyah added the Test This tests features that were implemented label Oct 19, 2024
@lhitmi lhitmi requested review from aattiyah and akobaidan October 20, 2024 13:09
Copy link

@akobaidan akobaidan left a comment

Choose a reason for hiding this comment

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

the code looks good so far.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend main code is written in backend frontend main code is written in frontend High effort Features that need high effort to implement Test This tests features that were implemented
Projects
None yet
4 participants