Skip to content

Commit 6e75390

Browse files
author
kkuzmin
authored
Merge pull request #31 from alertlogic/npm
merge Npm branch to master
2 parents ead9287 + edd6a2c commit 6e75390

File tree

5 files changed

+38
-4
lines changed

5 files changed

+38
-4
lines changed

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,3 +8,4 @@ nbproject/
88
coverage/
99
test/ide_test.js
1010
.env
11+
*.swp

.travis.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,6 @@ language: node_js
33
node_js:
44
- "8"
55
install:
6-
- make deps
6+
- npm install
77
script:
8-
- make test
8+
- npm run test

al_aws_collector.js

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,13 @@ class AlAwsCollector {
221221
);
222222
collector._azcollectc.checkin(checkin)
223223
.then(resp => {
224-
return callback(null);
224+
if(resp && resp.force_update === true){
225+
console.info("Force update");
226+
return collector.update(callback);
227+
}
228+
else{
229+
return callback(null);
230+
}
225231
})
226232
.catch(exception => {
227233
return callback(exception);

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@alertlogic/al-aws-collector-js",
3-
"version": "1.2.0",
3+
"version": "1.3.0",
44
"license": "MIT",
55
"description": "Alert Logic AWS Collector Common Library",
66
"repository": {

test/al_aws_collector_test.js

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -240,6 +240,33 @@ describe('al_aws_collector tests', function() {
240240
});
241241
});
242242

243+
it('checkin forced update success', function(done) {
244+
alserviceStub.post.restore();
245+
alserviceStub.post = sinon.stub(m_alCollector.AlServiceC.prototype, 'post').callsFake(
246+
function fakeFn(path, extraOptions) {
247+
return new Promise(function(resolve, reject) {
248+
return resolve({force_update: true});
249+
});
250+
});
251+
let fakeSelfUpdate = sinon.stub(AlAwsCollector.prototype, 'selfUpdate').callsFake(
252+
(callback) => { callback(); });
253+
let fakeSelfConfigUpdate = sinon.stub(AlAwsCollector.prototype, 'selfConfigUpdate').callsFake(
254+
(callback) => { callback(); });
255+
AlAwsCollector.load().then(function(creds) {
256+
var collector = new AlAwsCollector(
257+
mockContext, 'cwe', AlAwsCollector.IngestTypes.SECMSGS,'1.0.0', creds, undefined, [], []);
258+
collector.checkin(function(error) {
259+
assert.equal(error, undefined);
260+
sinon.assert.calledWith(alserviceStub.post, colMock.CHECKIN_URL, colMock.CHECKIN_AZCOLLECT_QUERY);
261+
sinon.assert.called(fakeSelfConfigUpdate);
262+
sinon.assert.called(fakeSelfUpdate);
263+
fakeSelfUpdate.restore();
264+
fakeSelfConfigUpdate.restore();
265+
done();
266+
});
267+
});
268+
});
269+
243270
it('checkin with custom check success', function(done) {
244271
AlAwsCollector.load().then(function(creds) {
245272
var spyHealthCheck = sinon.spy(function(callback) {

0 commit comments

Comments
 (0)