Skip to content

[ACEM] llteacher x ruff#111

Merged
siscia merged 1 commit into
mainfrom
acem/ruff-0.14.8-20260518-013819
May 18, 2026
Merged

[ACEM] llteacher x ruff#111
siscia merged 1 commit into
mainfrom
acem/ruff-0.14.8-20260518-013819

Conversation

@automaticcodeevolutionmanagement
Copy link
Copy Markdown

llteacher x ruff

What This PR Does

Fixed 6 F841 violations in 3 files using unsafe_llm strategy.

Details

  • Policy: Ruff 0.14.8
  • Initiative ID: 25
  • Fixes Applied: 6 violations
  • Files Modified: 3

Original Violations

📄 src/homeworks/tests/test_detail_view.py
F841 Local variable `widget2` is assigned to but never used
   --> src/homeworks/tests/test_detail_view.py:258:9
    |
256 |             order=1,
257 |         )
258 |         widget2 = HomeworkProgressWidget.objects.create(
    |         ^^^^^^^
259 |             homework=self.homework,
260 |             pre_prompt="Topic 2 pre",
    |
help: Remove assignment to unused variable `widget2`

Found 1 error.
No fixes available (1 hidden fix can be enabled with the `--unsafe-fixes` option).
📄 src/homeworks/tests/test_progress_widget.py
F841 Local variable `widget1` is assigned to but never used
  --> src/homeworks/tests/test_progress_widget.py:88:9
   |
86 |     def test_widget_ordering(self):
87 |         """Test widget ordering by order field."""
88 |         widget1 = HomeworkProgressWidget.objects.create(
   |         ^^^^^^^
89 |             homework=self.homework, pre_prompt="Pre 1", post_prompt="Post 1", order=2
90 |         )
   |
help: Remove assignment to unused variable `widget1`

F841 Local variable `widget2` is assigned to but never used
  --> src/homeworks/tests/test_progress_widget.py:91:9
   |
89 |             homework=self.homework, pre_prompt="Pre 1", post_prompt="Post 1", order=2
90 |         )
91 |         widget2 = HomeworkProgressWidget.objects.create(
   |         ^^^^^^^
92 |             homework=self.homework, pre_prompt="Pre 2", post_prompt="Post 2", order=1
93 |         )
   |
help: Remove assignment to unused variable `widget2`

F841 Local variable `widget3` is assigned to but never used
  --> src/homeworks/tests/test_progress_widget.py:94:9
   |
92 |             homework=self.homework, pre_prompt="Pre 2", post_prompt="Post 2", order=1
93 |         )
94 |         widget3 = HomeworkProgressWidget.objects.create(
   |         ^^^^^^^
95 |             homework=self.homework, pre_prompt="Pre 3", post_prompt="Post 3", order=3
96 |         )
   |
help: Remove assignment to unused variable `widget3`

Found 3 errors.
No fixes available (3 hidden fixes can be enabled with the `--unsafe-fixes` option).
📄 src/homeworks/tests/test_widget_answer_view.py
F841 Local variable `messages` is assigned to but never used
   --> src/homeworks/tests/test_widget_answer_view.py:190:9
    |
188 |         )
189 |         self.assertEqual(response.status_code, 302)
190 |         messages = (
    |         ^^^^^^^^
191 |             list(response.context["messages"])
192 |             if hasattr(response, "context") and response.context
    |
help: Remove assignment to unused variable `messages`

F841 Local variable `other_student` is assigned to but never used
   --> src/homeworks/tests/test_widget_answer_view.py:206:9
    |
204 |             username="other", email="other@example.com", password="password"
205 |         )
206 |         other_student = Student.objects.create(user=new_user)
    |         ^^^^^^^^^^^^^
207 |
208 |         self.client.login(username="other", password="password")
    |
help: Remove assignment to unused variable `other_student`

Found 2 errors.
No fixes available (2 hidden fixes can be enabled with the `--unsafe-fixes` option).
---

This pull request was automatically created by ACEM.

Policy: Ruff 0.14.8
Fixed: 6 violations

Original Violations:

src/homeworks/tests/test_detail_view.py:
F841 Local variable `widget2` is assigned to but never used
   --> src/homeworks/tests/test_detail_view.py:258:9
    |
256 |             order=1,
257 |         )
258 |         widget2 = HomeworkProgressWidget.objects.create(
    |         ^^^^^^^
259 |             homework=self.homework,
260 |             pre_prompt="Topic 2 pre",
    |
help: Remove assignment to unused variable `widget2`

Found 1 error.
No fixes available (1 hidden fix can be enabled with the `--unsafe-fixes` option).

src/homeworks/tests/test_progress_widget.py:
F841 Local variable `widget1` is assigned to but never used
  --> src/homeworks/tests/test_progress_widget.py:88:9
   |
86 |     def test_widget_ordering(self):
87 |         """Test widget ordering by order field."""
88 |         widget1 = HomeworkProgressWidget.objects.create(
   |         ^^^^^^^
89 |             homework=self.homework, pre_prompt="Pre 1", post_prompt="Post 1", order=2
90 |         )
   |
help: Remove assignment to unused variable `widget1`

F841 Local variable `widget2` is assigned to but never used
  --> src/homeworks/tests/test_progress_widget.py:91:9
   |
89 |             homework=self.homework, pre_prompt="Pre 1", post_prompt="Post 1", order=2
90 |         )
91 |         widget2 = HomeworkProgressWidget.objects.create(
   |         ^^^^^^^
92 |             homework=self.homework, pre_prompt="Pre 2", post_prompt="Post 2", order=1
93 |         )
   |
help: Remove assignment to unused variable `widget2`

F841 Local variable `widget3` is assigned to but never used
  --> src/homeworks/tests/test_progress_widget.py:94:9
   |
92 |             homework=self.homework, pre_prompt="Pre 2", post_prompt="Post 2", order=1
93 |         )
94 |         widget3 = HomeworkProgressWidget.objects.create(
   |         ^^^^^^^
95 |             homework=self.homework, pre_prompt="Pre 3", post_prompt="Post 3", order=3
96 |         )
   |
help: Remove assignment to unused variable `widget3`

Found 3 errors.
No fixes available (3 hidden fixes can be enabled with the `--unsafe-fixes` option).

src/homeworks/tests/test_widget_answer_view.py:
F841 Local variable `messages` is assigned to but never used
   --> src/homeworks/tests/test_widget_answer_view.py:190:9
    |
188 |         )
189 |         self.assertEqual(response.status_code, 302)
190 |         messages = (
    |         ^^^^^^^^
191 |             list(response.context["messages"])
192 |             if hasattr(response, "context") and response.context
    |
help: Remove assignment to unused variable `messages`

F841 Local variable `other_student` is assigned to but never used
   --> src/homeworks/tests/test_widget_answer_view.py:206:9
    |
204 |             username="other", email="other@example.com", password="password"
205 |         )
206 |         other_student = Student.objects.create(user=new_user)
    |         ^^^^^^^^^^^^^
207 |
208 |         self.client.login(username="other", password="password")
    |
help: Remove assignment to unused variable `other_student`

Found 2 errors.
No fixes available (2 hidden fixes can be enabled with the `--unsafe-fixes` option).

Automated by ACEM - Initiative #25
@siscia siscia merged commit ccd4031 into main May 18, 2026
6 checks passed
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