Restore UI compatibility with older browsers#2
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| import json | ||
| import math | ||
| import subprocess | ||
| from pathlib import Path | ||
|
|
||
| from factory_launch import GameState | ||
| from factory_launch.constants import ( | ||
| COAL_DRILL_OUTPUT_PER_SECOND, | ||
| PLAYER_STACK_LIMIT, | ||
| ) | ||
| from factory_launch.machines import CoalDrill, StoneSmelter | ||
|
|
||
|
|
||
| def test_initial_state_has_resources_and_machines(): | ||
| state = GameState() | ||
| assert len(state.drills) == 2 | ||
| assert len(state.smelters) == 1 | ||
| chest = state.storage_chests[0] | ||
| assert chest.items == {} | ||
|
|
There was a problem hiding this comment.
The test suite references math.sqrt, math.isclose, subprocess.run, and json.loads, but only Path and project modules are imported at the top of the file. When pytest runs, the first reference to any of those modules will raise a NameError and stop the suite before exercising the new code. Add the missing import math, import subprocess, and import json statements so the tests execute.
Useful? React with 👍 / 👎.
1 participant
Summary
Testing
https://chatgpt.com/codex/tasks/task_e_68ff98ba34c4832e85873e98d38d6ef0