Skip to content

Commit 0495633

Browse files
amomchilovst0012
authored andcommitted
Fix edge case for falsy values
1 parent 23b23f8 commit 0495633

File tree

3 files changed

+10
-6
lines changed

3 files changed

+10
-6
lines changed

lib/debug/server_cdp.rb

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1113,7 +1113,9 @@ def process_cdp args
11131113
when :properties
11141114
oid = args.shift
11151115

1116-
if obj = @obj_map[oid]
1116+
if @obj_map.key?(oid)
1117+
obj = @obj_map[oid]
1118+
11171119
members = if Array === obj
11181120
VariableInspector.new.indexed_members_of(obj, start: 0, count: obj.size)
11191121
else

lib/debug/server_dap.rb

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -869,8 +869,10 @@ def process_dap args
869869
event! :protocol_result, :scope, req, variables: vars, tid: self.id
870870
when :variable
871871
vid = args.shift
872-
obj = @var_map[vid]
873-
if obj
872+
873+
if @var_map.key?(vid)
874+
obj = @var_map[vid]
875+
874876
members = case req.dig('arguments', 'filter')
875877
when 'indexed'
876878
VariableInspector.new.indexed_members_of(

test/protocol/step_back_raw_dap_test.rb

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -883,7 +883,7 @@ def test_step_back_works_correctly
883883
namedVariables: /\d+/,
884884
indexedVariables: 0,
885885
expensive: false,
886-
variablesReference: 11
886+
variablesReference: 12
887887
},
888888
{
889889
name: "Global variables",
@@ -917,15 +917,15 @@ def test_step_back_works_correctly
917917
name: "%self",
918918
value: "Foo",
919919
type: "Module",
920-
variablesReference: 12,
920+
variablesReference: 13,
921921
indexedVariables: 0,
922922
namedVariables: /\d+/
923923
},
924924
{
925925
name: "bar",
926926
value: "nil",
927927
type: "NilClass",
928-
variablesReference: 13,
928+
variablesReference: 14,
929929
indexedVariables: 0,
930930
namedVariables: /\d+/
931931
}

0 commit comments

Comments
 (0)