Skip to content

Commit c6958f1

Browse files
committed
Rename RC test folder + fix tests
1 parent 9b16014 commit c6958f1

File tree

6 files changed

+8
-14
lines changed

6 files changed

+8
-14
lines changed

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
"test:debugger": "mocha -r 'packages/dd-trace/test/setup/mocha.js' 'packages/dd-trace/test/debugger/**/*.spec.js'",
2626
"test:debugger:ci": "nyc --no-clean --include 'packages/dd-trace/src/debugger/**/*.js' -- npm run test:debugger",
2727
"test:eslint-rules": "node eslint-rules/*.test.mjs",
28-
"test:trace:core": "tap packages/dd-trace/test/*.spec.js \"packages/dd-trace/test/{ci-visibility,datastreams,encode,exporters,opentelemetry,opentracing,plugins,remote_config,service-naming,standalone,telemetry}/**/*.spec.js\"",
28+
"test:trace:core": "tap packages/dd-trace/test/*.spec.js \"packages/dd-trace/test/{ci-visibility,datastreams,encode,exporters,opentelemetry,opentracing,plugins,remote-config,service-naming,standalone,telemetry}/**/*.spec.js\"",
2929
"test:trace:core:ci": "npm run test:trace:core -- --coverage --nyc-arg=--include=\"packages/dd-trace/src/**/*.js\"",
3030
"test:trace:guardrails": "mocha -r \"packages/dd-trace/test/setup/mocha.js\" \"packages/dd-trace/test/guardrails/**/*.spec.js\"",
3131
"test:trace:guardrails:ci": "nyc --no-clean --include \"packages/dd-trace/src/guardrails/**/*.js\" -- npm run test:trace:guardrails",

packages/dd-trace/test/remote_config/index.spec.js renamed to packages/dd-trace/test/remote-config/index.spec.js

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
'use strict'
22

3+
const RuleManager = require('../../src/appsec/rule_manager')
34
const RemoteConfigCapabilities = require('../../src/remote_config/capabilities')
45
const { kPreUpdate } = require('../../src/remote_config/manager')
56

67
let config
78
let rc
89
let RemoteConfigManager
9-
let RuleManager
1010
let UserTracking
1111
let log
1212
let appsec
@@ -33,10 +33,6 @@ describe('Remote Config index', () => {
3333

3434
RemoteConfigManager = sinon.stub().returns(rc)
3535

36-
RuleManager = {
37-
updateWafFromRC: sinon.stub()
38-
}
39-
4036
UserTracking = {
4137
setCollectionMode: sinon.stub()
4238
}
@@ -52,10 +48,8 @@ describe('Remote Config index', () => {
5248

5349
remoteConfig = proxyquire('../src/remote_config', {
5450
'./manager': RemoteConfigManager,
55-
'../rule_manager': RuleManager,
56-
'../user_tracking': UserTracking,
57-
'../../log': log,
58-
'..': appsec
51+
'../appsec/user_tracking': UserTracking,
52+
'../log': log,
5953
})
6054
})
6155

packages/dd-trace/test/remote_config/manager.spec.js renamed to packages/dd-trace/test/remote-config/manager.spec.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,11 +42,11 @@ describe('RemoteConfigManager', () => {
4242
RemoteConfigManager = proxyquire('../src/remote_config/manager', {
4343
'crypto-randomuuid': uuid,
4444
'./scheduler': Scheduler,
45-
'../../../../../package.json': { version: '3.0.0' },
46-
'../../exporters/common/request': request,
47-
'../../log': log,
45+
'../../../../package.json': { version: '3.0.0' },
46+
'../exporters/common/request': request,
47+
'../log': log,
4848
'../tagger': tagger,
49-
'../../service-naming/extra-services': {
49+
'../service-naming/extra-services': {
5050
getExtraServices: () => extraServices
5151
}
5252
})

0 commit comments

Comments
 (0)