Buttons and actions test cleanup

[msonnier]

Related Issues and PRs

A bunch of fixes on button mappings and related commands as we did button and code testing.

Issue:

Related PRs:

Checklist

• [ ] Code is complete, and naming of variables, methods and classes are readable by non-coder humans.
• [ ] This code change has been tested and validated working as expected.
• [ ] This code change has been reflected in robot documentation in the docs folder.
• [ ] The questions below have all been answered.
• [ ] The title of this PR has been updated to describe the change made, for example, [Subsystem or Command Name] Short description of change.
• [x] Assignees (usually yourself) have been assigned in the "Assignees" section to the right.
• [x] Relevant reviewers have been assigned under the "Reviewers" section at the right.

What is changing and why?

Misc bugs found in code testing

How was this tested?

Tested on bot.

Is there anything the reviewer of this PR needs to know to validate that this code is correct?

What do your team members need to know about this?

Known remaining issues:

• Speeds of intake for ground pickup probably not right -- may want to regear
• Code not there yet to auto zero Elevator encoder
• Need to add code to set Elevator Leader motor to reverse limit stop on limit switch. Done in Phoenix tuner for now.
• Add slowdown in endgame
• No sequence/button yet to knock over upright cone on spindexer