servo case structure issue

You broke it! this was good:

  static int state = one; // initial state is one.

This, not so much:

  int state; 
  
  state = one;

The newer code sets state to one on every iteration of loop. Hence you never see state two being executed.