sweep: option selection by typing the option number

Here's the PR! https://github.com/eranheres/ultra_type/pull/10.

💎 Sweep Pro: I'm using GPT-4. You have unlimited GPT-4 tickets. (tracking ID: a7eeabab96)

Actions (click)

[ ] ↻ Restart Sweep

Sandbox Execution ✓

Here are the sandbox execution logs prior to making any changes:

Sandbox logs for e844ed6

poetry run pytest 1/1 ✓

============================= test session starts ==============================
platform linux -- Python 3.10.12, pytest-7.4.3, pluggy-1.3.0
rootdir: /repo
collected 1 item
tests/test_controller.py .                                               [100%]
============================== 1 passed in 0.02s ===============================

Sandbox passed on the latest main, so sandbox checks will be enabled for this issue.

Step 1: 🔎 Searching

I found the following snippets in your repository. I will now analyze these snippets and come up with a plan.

Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description.

https://github.com/eranheres/ultra_type/blob/e844ed6e6583434e328b0d46cb21f483bf0b5132/README.md#L1-L20 https://github.com/eranheres/ultra_type/blob/e844ed6e6583434e328b0d46cb21f483bf0b5132/poetry.lock#L1-L100 https://github.com/eranheres/ultra_type/blob/e844ed6e6583434e328b0d46cb21f483bf0b5132/ultra_type/language.py#L1-L15 https://github.com/eranheres/ultra_type/blob/e844ed6e6583434e328b0d46cb21f483bf0b5132/ultra_type/controller.py#L1-L31 https://github.com/eranheres/ultra_type/blob/e844ed6e6583434e328b0d46cb21f483bf0b5132/tests/test_controller.py#L1-L2

Step 2: ⌨️ Coding

[X] Modify ultra_type/controller.py ✓ https://github.com/eranheres/ultra_type/commit/a3016dc822668be8c07351b9d63e6f6c5fc57081

Modify ultra_type/controller.py with contents:
• In the 'run' method of the 'Controller' class, modify the print statements that display the menu options to the user. Change "Menu:\n1. Practice\n2. Show Stats\n3. Quit" to "Menu:\n1. Practice\n2. Show Stats\n3. Exit". Also, update the descriptions to indicate that the user should input the option number, not the option name.
• Still in the 'run' method, modify the conditionals that check the user input. Instead of checking if the user input matches the string of the option name, check if the user input matches the integer of the option number. Change "if action == 'quit':" to "if action == '3':", "elif action == 'practice':" to "elif action == '1':", and "elif action == 'show_stats':" to "elif action == '2':".
• Ensure that the 'get_user_input' method in the 'View' class is called to get the user's input for the menu options. This method should return the user's input as a string, so you will need to convert this string to an integer before checking it against the option numbers. You can do this by adding "action = int(action)" before the conditionals.
[X] Ran sandbox for ultra_type/controller.py. ✓

Sandbox logs for

poetry run pytest 1/1 ✓

============================= test session starts ==============================
platform linux -- Python 3.10.12, pytest-7.4.3, pluggy-1.3.0
rootdir: /repo
collected 1 item
tests/test_controller.py .                                               [100%]
============================== 1 passed in 0.02s ===============================

Step 3: 🔁 Code Review

I have finished reviewing the code for completeness. I did not find errors for sweep/option-selection-by-number.

🎉 Latest improvements to Sweep:

Sweep now uses the rope library to refactor Python! Check out Large Language Models are Bad at Refactoring Code. To have Sweep refactor your code, try sweep: Refactor <your_file>.py!
Sweep finds and fixes bugs in your code by writing unit tests! Check out Having GPT-4 Iterate on Unit Tests like a Human.

💡 To recreate the pull request edit the issue title or description. To tweak the pull request, leave a comment on the pull request. ^{Join Our Discord}

eranheres / ultra_type