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

Fix the tests #3376

Closed
DonnieBLT opened this issue Feb 10, 2025 · 13 comments · Fixed by #3390 · May be fixed by #3393
Closed

Fix the tests #3376

DonnieBLT opened this issue Feb 10, 2025 · 13 comments · Fixed by #3390 · May be fixed by #3393
Assignees

Comments

@DonnieBLT
Copy link
Collaborator

DonnieBLT commented Feb 10, 2025

I broke them when making some changes yesterday. Sorry about that it's something to do with the signup button and the spacing in base.html

@SahilDhillon21
Copy link
Contributor

/assign

Copy link
Contributor

Hello @SahilDhillon21! You've been assigned to OWASP-BLT/BLT issue #3376. You have 24 hours to complete a pull request.

@SahilDhillon21
Copy link
Contributor

I am not available for a few days, and since this is an urgent issue, I request someone else to take over thank you!
/unassign

@tsu-ki
Copy link
Contributor

tsu-ki commented Feb 10, 2025

@DonnieBLT sir, has this been fixed already? Some PRs are passing all CI/CD tests now

@DonnieBLT
Copy link
Collaborator Author

This has not been fixed yet

@tsu-ki
Copy link
Contributor

tsu-ki commented Feb 11, 2025

/assign

Copy link
Contributor

Hello @tsu-ki! You've been assigned to OWASP-BLT/BLT issue #3376. You have 24 hours to complete a pull request.

@tsu-ki
Copy link
Contributor

tsu-ki commented Feb 11, 2025

I broke them when making some changes yesterday. Sorry about that it's something to do with the signup button and the spacing in base.html

@DonnieBLT sir, some changes made in tests_urls.py file seem to be causing the error. Are you sure this bug was caused by changes in base.html or signup.html file?

======================================================================
FAIL: test_responses (website.tests_urls.UrlsTest.test_responses)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/BLT/BLT/website/tests_urls.py", line 151, in test_responses
    check_urls(module.urlpatterns)
  File "/home/runner/work/BLT/BLT/website/tests_urls.py", line 145, in check_urls
    self.assertNotIn(
AssertionError: 'SyntaxError' unexpectedly found in "{'level': 'SEVERE', 'message': 'http://localhost:58299/ 1722:12 Uncaught SyntaxError: Invalid or unexpected token', 'source': 'javascript', 'timestamp': 1739259850080}" : !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!the url that caused the eror is: /

----------------------------------------------------------------------
Ran 40 tests in 33.749s

FAILED (failures=1)
Destroying test database for alias 'default' ('file:memorydb_default?mode=memory&cache=shared')...

@DonnieBLT
Copy link
Collaborator Author

This looks like a different one. I wonder if the other one got fixed somehow

@DonnieBLT
Copy link
Collaborator Author

Maybe it’s this

Image

@tsu-ki
Copy link
Contributor

tsu-ki commented Feb 11, 2025

Maybe it’s this

Image

@DonnieBLT I'll check this out, can you please tell in which file is this happening? thanks for the help!

@DonnieBLT
Copy link
Collaborator Author

That is the homepage

@SahilDhillon21
Copy link
Contributor

/unassign

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
3 participants