Refactor step function code by flipping if-condition

[lym953]

What

Refactors some code by changing:

if (<condition>) {
  // do a lot of things
}
return false;

to

if (!<condition>) {
  return false;
}
// do a lot of things

Why

Make code more readable in that:

• the "if" block is shorter so the reader won't need to take time to find where the } is
• it's more clear that we don't cover the case when <condition> is false

Testing

This is a simple change, so just npm test

Review checklist

• [ ] Feature or bugfix MUST have appropriate tests (unit, integration)

[datadog-datadog-prod-us1[bot]]

Datadog Report

Branch report: yiming.luo/fix-step-func-5 Commit report: 1306a7c Test service: datadog-ci-tests

:white_check_mark: 0 Failed, 376 Passed, 0 Skipped, 1m 26.67s Total duration (2m 1.01s time saved)