Decaffeinate `spec/apm-cli-spec`

[GuilleW]

Merge https://github.com/pulsar-edit/ppm/pull/19 first. EDIT : ❌ decaffeinate-project is used at first, then manual refactor/clean.

From coffee

$ jasmine-focused --captureExceptions --coffee spec/apm-cli-spec.coffee
> 6 tests, 24 assertions, 0 failures, 0 skipped

to js

$ jasmine-focused --captureExceptions spec/apm-cli-spec.js
> 6 tests, 24 assertions, 0 failures, 0 skipped

[GuilleW]

Following @confused-Techie comment, I forgot to follow Decaffeination guidelines.

Since I work on many files, I edit package.json to add useful shortcut to avoid misspelt filename.

    "test-decaff": "npx coffee --compile --output spec spec/${npm_config_name}.coffee",
    "test-coffee": "jasmine-focused --captureExceptions --coffee spec/${npm_config_name}.coffee",
    "test-js": "jasmine-focused --captureExceptions spec/${npm_config_name}.js"

Steps to reproduce decaffeination:

• [x] Check package.json for coffee-script version
  • version: 1.12.7
• [x] Decaffeination with official Coffee program
  • npx coffee --compile --output spec spec/apm-cli-spec.coffee
  • (or my shortcut: ./bin/npm run test-decaff --name=apm-cli-spec)
• [x] Quality of Decaffeination
  • [x] Replace var with const and let if it's possible.
  • [x] Replace Nested Ternary Operators with if statements.
  • [x] Remove return when it's not needed.
  • [x] Remove Result Arrays when they are not needed.
  • [x] Replace for loops with for-of loops.
  • [x] Use Arrow Functions to write functions in a single line when it's possible.
  • [x] Remove unneeded ref variables.
  • [x] Remove do blocks and use let or const instead.
  • [x] Tests and assertions should be the same and success like in coffee file

Test from coffee and JS should output same successful result :

From coffee

$ jasmine-focused --captureExceptions --coffee spec/apm-cli-spec.coffee
// or my shortcut $ `./bin/npm run test-coffee --name=apm-cli-spec`
> 6 tests, 24 assertions, 0 failures, 0 skipped

to js

$ jasmine-focused --captureExceptions spec/apm-cli-spec.js
// or my shortcut $ `./bin/npm run test-js --name=apm-cli-spec`
> 6 tests, 24 assertions, 0 failures, 0 skipped

[2colours]

Supposedly superseded by https://github.com/pulsar-edit/ppm/pull/64.

[confused-Techie]

Again @2colours has very helpful refactored this PR into the modern decaf format in #64 and that has now been merged.

Thanks a ton to @2colours & @GuilleW for all the hard work here