Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Fermín Galán Márquez <[email protected]>
  • Loading branch information
mapedraza and fgalan authored Aug 26, 2024
1 parent b94b414 commit e07371d
Show file tree
Hide file tree
Showing 8 changed files with 12 additions and 12 deletions.
2 changes: 1 addition & 1 deletion lib/services/northBound/deviceGroupAdministrationServer.js
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ const mandatoryHeaders = ['fiware-service', 'fiware-servicepath'];
const mandatoryParameters = ['resource', 'apikey'];
const constants = require('../../constants');

// Create new templeate for configuration groups replacing services by CONFIGGROUP_TERM
// Create new template for configuration groups replacing services by CONFIGGROUP_TERM
const templateGroup = JSON.parse(JSON.stringify(templateGroupService));
templateGroup.properties[constants.CONFIGGROUP_TERM] = templateGroup.properties.services;
delete templateGroup.properties.services;
Expand Down
4 changes: 2 additions & 2 deletions test/unit/general/deviceService-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -210,7 +210,7 @@ describe('NGSI-v2 - Device Service: utils', function () {
async.series([iotAgentLib.clearAll, iotAgentLib.deactivate], done);
});

//FIXME: this test will be removed if at the end /iot/services API (now Deprecated) is removed
// #FIXME1649: this test will be removed if at the end /iot/services API (now Deprecated) is removed
describe('When an existing device tries to be retrieved with retrieveOrCreate()', function () {
beforeEach(function (done) {
contextBrokerMock = nock('http://192.168.1.1:1026')
Expand Down Expand Up @@ -277,7 +277,7 @@ describe('NGSI-v2 - Device Service: utils', function () {
});
});

//FIXME: this test will be removed if at the end /iot/services API (now Deprecated) is removed
// #FIXME1649: this test will be removed if at the end /iot/services API (now Deprecated) is removed

describe('When an unexisting device tries to be retrieved for an existing APIKey', function () {
beforeEach(function (done) {
Expand Down
2 changes: 1 addition & 1 deletion test/unit/mongodb/mongodb-configGroup-registry-test.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright 2015 Telefonica Investigación y Desarrollo, S.A.U
* Copyright 2024 Telefonica Investigación y Desarrollo, S.A.U
*
* This file is part of fiware-iotagent-lib
*
Expand Down
2 changes: 1 addition & 1 deletion test/unit/mongodb/mongodb-group-registry-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
* Modified by: Daniel Calvo - ATOS Research & Innovation
*/

// FIXME: parallel tests in mongodb-configGroup-registry-test.js. Remove this file if at the end /iot/services API (now Deprecated) is removed
// #FIXME1649: parallel tests in mongodb-configGroup-registry-test.js. Remove this file if at the end /iot/services API (now Deprecated) is removed
/* eslint-disable no-unused-vars */

const iotAgentLib = require('../../../lib/fiware-iotagent-lib');
Expand Down
4 changes: 2 additions & 2 deletions test/unit/ngsiv2/general/deviceService-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -210,7 +210,7 @@ describe('NGSI-v2 - Device Service: utils', function () {
nock.cleanAll();
async.series([iotAgentLib.clearAll, iotAgentLib.deactivate], done);
});
//FIXME: this test will be removed if at the end /iot/services API (now Deprecated) is removed
// #FIXME1649: this test will be removed if at the end /iot/services API (now Deprecated) is removed
describe('When an existing device tries to be retrieved with retrieveOrCreate()', function () {
beforeEach(function (done) {
// This mock does not check the payload since the aim of the test is not to verify
Expand Down Expand Up @@ -272,7 +272,7 @@ describe('NGSI-v2 - Device Service: utils', function () {
});
});
});
//FIXME: this test will be removed if at the end /iot/services API (now Deprecated) is removed
// #FIXME1649: this test will be removed if at the end /iot/services API (now Deprecated) is removed
describe('When an unexisting device tries to be retrieved for an existing APIKey', function () {
beforeEach(function (done) {
// This mock does not check the payload since the aim of the test is not to verify
Expand Down
6 changes: 3 additions & 3 deletions test/unit/ngsiv2/general/iotam-autoregistration-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -351,7 +351,7 @@ describe('NGSI-v2 - IoT Manager autoregistration', function () {
});
});

//FIXME: this test will be removed if at the end /iot/services API (now Deprecated) is removed
// #FIXME1649: this test will be removed if at the end /iot/services API (now Deprecated) is removed
describe('When a new service is created in the IoT Agent', function () {
beforeEach(function (done) {
nock.cleanAll();
Expand Down Expand Up @@ -422,7 +422,7 @@ describe('NGSI-v2 - IoT Manager autoregistration', function () {
});
});

//FIXME: this test will be removed if at the end /iot/services API (now Deprecated) is removed
// #FIXME1649: this test will be removed if at the end /iot/services API (now Deprecated) is removed
describe('When a service is removed from the IoT Agent', function () {
beforeEach(function (done) {
nock.cleanAll();
Expand Down Expand Up @@ -493,7 +493,7 @@ describe('NGSI-v2 - IoT Manager autoregistration', function () {
});
});

//FIXME: this test will be removed if at the end /iot/services API (now Deprecated) is removed
// #FIXME1649: this test will be removed if at the end /iot/services API (now Deprecated) is removed
describe('When a new service with static attributes is created in the IoT Agent', function () {
beforeEach(function (done) {
nock.cleanAll();
Expand Down
2 changes: 1 addition & 1 deletion test/unit/ngsiv2/provisioning/device-group-api-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@

/* eslint-disable no-unused-vars */

// FIXME: parallel tests in device-provisioning-configGroup-api_test.js.
// #FIXME1649: parallel tests in device-provisioning-configGroup-api_test.js.

const iotAgentLib = require('../../../../lib/fiware-iotagent-lib');
const _ = require('underscore');
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright 2014 Telefonica Investigación y Desarrollo, S.A.U
* Copyright 2024 Telefonica Investigación y Desarrollo, S.A.U
*
* This file is part of fiware-iotagent-lib
*
Expand Down

0 comments on commit e07371d

Please sign in to comment.