Implement delayed action queue with round countdown and automatic execution #267
Labels
No labels
Blocked
Duplicate
MoSCoW/Could Have
MoSCoW/Must Have
MoSCoW/Should Have
Points/1
Points/13
Points/2
Points/21
Points/3
Points/5
Points/8
Priority/Backlog
Priority/Critical
Priority/High
Priority/Low
Priority/Medium
State/Completed
State/In progress
State/In review
State/Paused
State/Unverified
State/Verified
State/Wont Do
Type/Bug
Type/Epic
Type/Feature
Type/Legendary
Type/Task
Type/Testing
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Blocks
#264 Epic: Round-Delay Telegraph System
aethyr/Aethyr
Reference: aethyr/Aethyr#267
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Metadata
feat(combat): implement delayed action queue with countdown executionfeature/m4-delayed-queueBackground and Context
The delayed action queue stores all committed actions that have delay>0. Each round during the Countdown phase, all queued actions have their remaining delay decremented. When an action reaches delay=0, it is moved to the Resolution queue for execution in the next Resolution phase.
Expected Behavior
DelayedActionQueue manages a priority-ordered list of pending actions. Methods:
enqueue(action),tick()(decrement all),ready_actions()(return and remove delay=0 actions),all_pending().Acceptance Criteria
tick()decrements all pending actions by 1ready_actions()returns actions that have reached delay=0Subtasks
lib/aethyr/core/combat/delayed_action_queue.rbenqueue(action)adding to pending listtick()decrementing remaining_delay on all entriesready_actions()extracting and returning delay=0 actionsall_pending()for telegraph displaytests/unit/delayed_queue.featurecovering enqueue, tick decrement, ready extraction, order preservation.tests/integration/for delayed queue across multiple combat rounds.bundle exec rake unit_profileand verify no performance regressions.bundle exec rake unit. If coverage is <97% then review the current unit test coverage report atbuild/tests/unit/coverage/and use it to write new Cucumber based unit tests to improve code coverage. Specifically, write Cucumber/Gherkin style unit tests that are descriptively named and specifically improve coverage on whichever file has the most uncovered lines by writing tests that will target the uncovered lines in the report. Once that is done rerunbundle exec rake unitto verify all tests pass and coverage is above >=97%. Only mark this as complete once coverage is >=97%, if not repeat this task as many times as is needed until coverage reaches >=97%.bundle exec rake(default task: unit tests with coverage) andbundle exec rake integration, fix any errors if needed ensuring both pass across entire code base, do not ignore any failure even if it seems unrelated to this commit, fix it.Definition of Done
This issue is complete when:
master, reviewed, and merged before this issue is marked done.