fix: default value ignored when getting an empty config value

[hbugdoll]

Fixes #894.

Introduction:

Example: The default value in config.get('cli.width', 80) is ignored, if config value cli.width is empty.

Steps to reproduce:

ti config cli.width ""
ti info -t android --sdk 13.0.1.GA

Description:

• changed return logic of TiConfig.get(key, defaultValue)
  • previously, the default value was only returned if the key didn't exist
  • a specified default value is now respected if a config value is empty (get('cli.width', 80) → 80)
  • if no default value is specified, it remains as before (get('cli.width') → '')
• added unit tests for empty config settings
  • therefore, added example case from above in fixture JSON

[hbugdoll]

A Windows test failed with exit code 3221226505 in

titanium-cli/test/commands/ti-sdk.test.js

Line 262 in a29ce59

assert.strictEqual(exitCode, 1);

[cb1kenobi]

Awesome, thanks!

[cb1kenobi]

A Windows test failed with exit code 3221226505 in

titanium-cli/test/commands/ti-sdk.test.js

Line 262 in a29ce59

assert.strictEqual(exitCode, 1);

Yeah, that happens on Windows sometimes. It means there's a memory issue. I think it's coming from Vitest. Oh well.

[hbugdoll]

Yeah, that happens on Windows sometimes. It means there's a memory issue. I think it's coming from Vitest.

Thanks for the clarification.