Skip to content

Commit 87bf5bc

Browse files
committed
chore: test
1 parent 9912bca commit 87bf5bc

2 files changed

Lines changed: 8 additions & 1 deletion

File tree

packages/collector/test/integration/currencies/databases/redis/test_base.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1018,7 +1018,7 @@ module.exports = function (name, version, isLatest, mode) {
10181018
LIBRARY_VERSION: version,
10191019
LIBRARY_NAME: name,
10201020
REDIS_SETUP_TYPE: mode,
1021-
INSTANA_TRACING_DISABLE: 'redis'
1021+
INSTANA_TRACING_DISABLE_INSTRUMENTATIONS: 'redis'
10221022
}
10231023
});
10241024
await controls.startAndWaitForAgentConnection(5000, Date.now() + 1000 * 60 * 5);

packages/core/test/config/normalizeConfig_test.js

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -170,6 +170,7 @@ describe('config.normalizeConfig', () => {
170170
expect(config.tracing.enabled).to.be.true;
171171
expect(config.tracing.automaticTracingEnabled).to.be.false;
172172
});
173+
173174
it('should not enable automatic tracing when tracing is disabled in general', () => {
174175
const config = coreConfig.normalize({
175176
userConfig: {
@@ -206,6 +207,12 @@ describe('config.normalizeConfig', () => {
206207
expect(config.tracing.enabled).to.be.false;
207208
});
208209

210+
it('should enable tracing if env var conatin non-boolean value', () => {
211+
process.env.INSTANA_TRACING_DISABLE = 'redis';
212+
const config = coreConfig.normalize({});
213+
expect(config.tracing.enabled).to.be.true;
214+
});
215+
209216
it('should use default (true) for automaticTracingEnabled when neither env nor config is set', () => {
210217
const config = coreConfig.normalize({});
211218
expect(config.tracing.automaticTracingEnabled).to.be.true;

0 commit comments

Comments
 (0)