If you wrote the code by hand (rather than RobotBuilder generated) make sure that you are calling Scheduler.run() in the TeleopPeriodic method like this:
Code:
public void teleopPeriodic() {
Scheduler.getInstance().run();
}
That's what actually causes the commands to run each iteration.