Skip to content

fix(tests) :- resolve commented-out POST request in test_create_tracker - #1069

Closed
Aaravanand00 wants to merge 1 commit into
alphaonelabs:mainfrom
Aaravanand00:fix/broken-tracker-test
Closed

Aaravanand00 wants to merge 1 commit into
alphaonelabs:mainfrom
Aaravanand00:fix/broken-tracker-test

Conversation

@Aaravanand00

Copy link
Copy Markdown

Related issues

Fixes #<ISSUE_NUMBER>

Summary of Changes

  • Fixed tests/test_trackers.py:
    • In ProgressTrackerTests.test_create_tracker, the self.client.post(...) call was previously commented out inside a docstring block ("""..."""), while the assertions were still active, causing immediate DoesNotExist / AssertionError failures upon execution.
    • Uncommented and restored the POST request to create_tracker endpoint.
    • Added assertion self.assertEqual(response.status_code, 302) to verify successful redirect to tracker detail view.
  • Added tests/__init__.py:
    • Created package __init__.py file in tests/ directory to ensure Python and Django test runners properly discover test modules.

Testing & Verification

Executed the test suite using Django's test runner:

python manage.py test tests.test_trackers

##Results :- 
Creating test database for alias 'default'...
....
----------------------------------------------------------------------
Ran 4 tests in 3.891s

OK
Destroying test database for alias 'default'...
System check identified no issues (0 silenced).

@github-actions

Copy link
Copy Markdown
Contributor

👀 Peer Review Required

Hi @Aaravanand00! This pull request does not yet have a peer review.

Before this PR can be merged, please request a review from one of your peers:

  • Go to the PR page and click "Reviewers" on the right sidebar.
  • Select a team member or contributor to review your changes.
  • Once they approve, this reminder will be automatically removed.

Thank you for contributing! 🎉

@github-actions github-actions Bot added the files-changed: 2 PR changes 2 files label Sep 29, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🚚 This Repository Is Moving

Hi @Aaravanand00, thank you for your contribution!

We are in the process of migrating most of the logic from this repository to our new repository: alphaonelabs/learn.

What this means for your PR

Please do not merge or continue work here. Instead:

  1. Review the alphaonelabs/learn repository and familiarize yourself with its tech stack.
  2. Adapt your changes to work with that codebase.
  3. Open a new Pull Request in alphaonelabs/learn.

This PR has been automatically closed. Once you have opened your PR in the new repository, feel free to reference it here.

Thank you for your understanding and continued support! 🙏

@github-actions github-actions Bot closed this Sep 29, 2026
@coderabbitai

coderabbitai Bot commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Repository: alphaonelabs/coderabbit/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: c942bd45-055f-496c-85ba-6482900be720

📥 Commits

Reviewing files that changed from the base of the PR and between 7a682c2 and 7c7eb2c.

📒 Files selected for processing (2)
  • tests/__init__.py
  • tests/test_trackers.py
 ___________________________________________
< Transformers: Not just a movie franchise. >
 -------------------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

files-changed: 2 PR changes 2 files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant