[COOK-3410] eliminate redundancy

* Ensure idempotence for ntp::undo by removing, only if conflicting state exists in run_list
* Remove redundant, and confusingly named disable recipe, ntp::undo serves this purpose.
* Remove ntpdate recipe.
  - This is not a service, it is _NOT_ a thing to be managed.  This
    recipe only affects the redundant ntpdate-debian command.  There is
    little purpose in disabling a command which is not called from the
    host recipe.
[COOK-3410] (rebase)

Fixed two foodcritic warnings for the TravisCI green light, post-rebase

* FC019: Access node attributes in a consistent manner: ./recipes/windows_client.rb:44
* FC023: Prefer conditional attributes: ./recipes/windows_client.rb:33