Skip to content

Commit

Permalink
fix: insert new node providers into node_provider_lookup instead of s…
Browse files Browse the repository at this point in the history
…ubscribers table
  • Loading branch information
louisevelayo committed Nov 14, 2023
1 parent b0d3310 commit 6d3d7ce
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 5 deletions.
4 changes: 1 addition & 3 deletions node_monitor/node_monitor.py
Original file line number Diff line number Diff line change
Expand Up @@ -206,9 +206,7 @@ def update_node_provider_lookup_if_new(

if node_providers_new:
for node_provider in node_providers_new:
params = (node_provider.principal_id, False, False,
False, node_provider.display_name, False)
self.node_provider_db.execute_write(INSERT_SUBSCRIBER, params)
self.node_provider_db.insert_node_provider(node_provider)



Expand Down
4 changes: 2 additions & 2 deletions tests/test_node_monitor.py
Original file line number Diff line number Diff line change
Expand Up @@ -232,7 +232,7 @@ def test_no_new_node_provider():
mock_slack_bot, mock_telegram_bot)

nm.update_node_provider_lookup_if_new(cached['node_provider_control'])
assert mock_node_provider_db.execute_write.call_count == 0
assert mock_node_provider_db.insert_node_provider.call_count == 0



Expand All @@ -244,5 +244,5 @@ def test_one_new_node_provider():
mock_slack_bot, mock_telegram_bot)

nm.update_node_provider_lookup_if_new(cached['new_node_providers'])
assert mock_node_provider_db.execute_write.call_count == 2
assert mock_node_provider_db.insert_node_provider.call_count == 2

0 comments on commit 6d3d7ce

Please sign in to comment.