Skip to content

Commit ced7971

Browse files
committed
Fix lint issues
1 parent 9147560 commit ced7971

File tree

4 files changed

+20
-18
lines changed

4 files changed

+20
-18
lines changed

__tests__/allowHalfOpen.test.js

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
21
import { expect, test, jest } from '@jest/globals';
32
import net from '../src/index';
43
import { nativeEventEmitter } from '../src/Globals';
@@ -14,7 +13,7 @@ jest.mock('../src/Globals', () => {
1413
originalAddListener(event, listener);
1514
return { remove: () => emitter.removeListener(event, listener) };
1615
};
17-
16+
1817
let idCounter = 1000;
1918
return {
2019
__esModule: true,
@@ -30,7 +29,7 @@ test('allowHalfOpen: false (default) should call Sockets.end() on end event', (d
3029
const server = net.createServer(); // allowHalfOpen default false
3130
const serverId = server._id;
3231
server.listen(12345);
33-
32+
3433
server.on('connection', (socket) => {
3534
socket.on('end', () => {
3635
try {
@@ -54,9 +53,9 @@ test('allowHalfOpen: false (default) should call Sockets.end() on end event', (d
5453
localPort: 12345,
5554
remoteAddress: '127.0.0.1',
5655
remotePort: 54321,
57-
remoteFamily: 'IPv4'
58-
}
59-
}
56+
remoteFamily: 'IPv4',
57+
},
58+
},
6059
});
6160

6261
// Simulate 'end' event from native for socket 456
@@ -70,7 +69,7 @@ test('allowHalfOpen: true should NOT call Sockets.end() on end event', (done) =>
7069
const server = net.createServer({ allowHalfOpen: true });
7170
const serverId = server._id;
7271
server.listen(12346);
73-
72+
7473
server.on('connection', (socket) => {
7574
socket.on('end', () => {
7675
try {
@@ -93,9 +92,9 @@ test('allowHalfOpen: true should NOT call Sockets.end() on end event', (done) =>
9392
localPort: 12346,
9493
remoteAddress: '127.0.0.1',
9594
remotePort: 54321,
96-
remoteFamily: 'IPv4'
97-
}
98-
}
95+
remoteFamily: 'IPv4',
96+
},
97+
},
9998
});
10099

101100
// Simulate 'end' event from native for socket 457

__tests__/server_options.test.js

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
21
import { expect, test, jest } from '@jest/globals';
32

43
jest.mock('../src/Globals', () => {
@@ -21,9 +20,9 @@ import { nativeEventEmitter } from '../src/Globals';
2120

2221
test('server option pauseOnConnect should pause the socket', (done) => {
2322
const server = net.createServer({ pauseOnConnect: true });
24-
23+
2524
server.listen(12345);
26-
25+
2726
server.on('connection', (socket) => {
2827
try {
2928
// Check if socket is paused
@@ -43,8 +42,8 @@ test('server option pauseOnConnect should pause the socket', (done) => {
4342
localPort: 12345,
4443
remoteAddress: '127.0.0.1',
4544
remotePort: 54321,
46-
remoteFamily: 'IPv4'
47-
}
48-
}
45+
remoteFamily: 'IPv4',
46+
},
47+
},
4948
});
5049
});

src/Server.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@ export default class Server extends EventEmitter {
3838
this._id = getNextId();
3939
/** @protected @readonly */
4040
this._eventEmitter = nativeEventEmitter;
41-
// console.log('Server eventEmitter:', this._eventEmitter);
4241
/** @private @type {Set<Socket>} */
4342
this._connections = new Set();
4443
/** @private */

src/Socket.js

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,12 @@ export default class Socket extends EventEmitter {
165165
});
166166
this._connecting = true;
167167
this._readyState = 'opening';
168-
NativeModules.TcpSockets.connect(this._id, customOptions.host, customOptions.port, customOptions);
168+
NativeModules.TcpSockets.connect(
169+
this._id,
170+
customOptions.host,
171+
customOptions.port,
172+
customOptions
173+
);
169174
return this;
170175
}
171176

0 commit comments

Comments
 (0)