Skip to content

Commit f52425d

Browse files
author
Jerry Cheung
committed
make #write_request private
1 parent 262819a commit f52425d

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

lib/net/ldap/connection.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -146,6 +146,7 @@ def write(packet)
146146
def write_request(request, controls = nil)
147147
write([next_msgid.to_ber, request, controls].compact.to_ber_sequence)
148148
end
149+
private :write_request
149150

150151
def next_msgid
151152
@msgid ||= 0

test/test_ldap_connection.rb

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,22 +26,22 @@ def test_write_request
2626
mock = Minitest::Mock.new
2727
mock.expect(:write, true, [[1.to_ber, "request"].to_ber_sequence])
2828
conn = Net::LDAP::Connection.new(:socket => mock)
29-
conn.write_request("request")
29+
conn.send(:write_request, "request")
3030
end
3131

3232
def test_write_request_with_controls
3333
mock = Minitest::Mock.new
3434
mock.expect(:write, true, [[1.to_ber, "request", "controls"].to_ber_sequence])
3535
conn = Net::LDAP::Connection.new(:socket => mock)
36-
conn.write_request("request", "controls")
36+
conn.send(:write_request, "request", "controls")
3737
end
3838

3939
def test_write_request_increments_msgid
4040
mock = Minitest::Mock.new
4141
mock.expect(:write, true, [[1.to_ber, "request1"].to_ber_sequence])
4242
mock.expect(:write, true, [[2.to_ber, "request2"].to_ber_sequence])
4343
conn = Net::LDAP::Connection.new(:socket => mock)
44-
conn.write_request("request1")
45-
conn.write_request("request2")
44+
conn.send(:write_request, "request1")
45+
conn.send(:write_request, "request2")
4646
end
4747
end

0 commit comments

Comments
 (0)