[FSSDK-9987] fix: Conditional ODP instantiation

[mikechu-optimizely]

Summary

• When explicit odpOptions.disabled = true is passed, avoid ODP instantiation
• Fix Jest test setting in VSCode
• Lint updates

Test plan

• Existing unit and e2e test suite are expected to pass

Issues

• FSSDK-9987

[coveralls]

coverage: 90.39% (-0.02%) from 90.411%
when pulling 5775e15 on mike/FSSDK-9987/performance-refactors
into 961f6a1 on master.

[mikechu-optimizely]

Developer review completed.

[jaeopt]

Looks good to me. We need to double check of its side-effects.

For example -

javascript-sdk/lib/optimizely/index.ts

Line 1761 in 961f6a1

this.logger.error(ERROR_MESSAGES.ODP_FETCH_QUALIFIED_SEGMENTS_FAILED_ODP_MANAGER_MISSING);

They do not want to see many "ERROR"-level logging. Better stay silent except for "ODP disabled" at the init phase.

[mikechu-optimizely]

Looks good to me. We need to double check of its side-effects.

For example -

javascript-sdk/lib/optimizely/index.ts

Line 1761 in 961f6a1

this.logger.error(ERROR_MESSAGES.ODP_FETCH_QUALIFIED_SEGMENTS_FAILED_ODP_MANAGER_MISSING);

They do not want to see many "ERROR"-level logging. Better stay silent except for "ODP disabled" at the init phase.

This is an EXCELLENT catch @jaeopt. Thanks. Let me comb through and find these scenarios.

[mikechu-optimizely]

Temp close