Skip to content

Commit 85dce79

Browse files
Juliana MashonJuliana Mashon
Juliana Mashon
authored and
Juliana Mashon
committed
Removed set_token()
1 parent c87e370 commit 85dce79

File tree

2 files changed

+17
-22
lines changed

2 files changed

+17
-22
lines changed

main.py

+5-10
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,11 @@ def __init__(self):
99
self.token = None
1010
self.error = None
1111
self.last_message = None
12+
self.verbosity = 1
1213

1314
class Farmbot():
1415
def __init__(self):
1516
self.state = State()
16-
self.echo = True
1717

1818
# Initialize other components without the token initially
1919
self.auth = Authentication(self.state)
@@ -28,16 +28,11 @@ def __init__(self):
2828
self.resources = Resources(self.state)
2929
self.tools = ToolControls(self.state)
3030

31-
def set_token(self, token):
32-
self.state.token = token
31+
def set_verbosity(self, value):
32+
if value <= -1 or value >= 3:
33+
return print("ERROR: verbosity must be between 0-2")
3334

34-
def set_echo(self, value):
35-
if value is True:
36-
self.echo = True
37-
elif value is False:
38-
self.echo = False
39-
else:
40-
return print("ERROR: `Echo` can only be True or False.")
35+
self.state.verbosity = value
4136

4237
# authentication.py
4338

tests/tests_main.py

+12-12
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ def test_get_info_endpoint_only(self, mock_request):
110110
mock_response.status_code = 200
111111
mock_request.return_value = mock_response
112112
fb = Farmbot()
113-
fb.set_token(MOCK_TOKEN)
113+
fb.state.token = MOCK_TOKEN
114114
# Call with endpoint only
115115
response = fb.get_info('device')
116116
mock_request.assert_called_once_with(
@@ -133,7 +133,7 @@ def test_get_info_with_id(self, mock_request):
133133
mock_response.status_code = 200
134134
mock_request.return_value = mock_response
135135
fb = Farmbot()
136-
fb.set_token(MOCK_TOKEN)
136+
fb.state.token = MOCK_TOKEN
137137
# Call with specific ID
138138
response = fb.get_info('peripherals', '12345')
139139
mock_request.assert_called_once_with(
@@ -155,7 +155,7 @@ def test_set_info(self, mock_request):
155155
mock_response.json.return_value = {'name': 'new name'}
156156
mock_request.return_value = mock_response
157157
fb = Farmbot()
158-
fb.set_token(MOCK_TOKEN)
158+
fb.state.token = MOCK_TOKEN
159159
device_info = fb.set_info('device', 'name', 'new name')
160160
mock_request.assert_has_calls([call(
161161
'PATCH',
@@ -188,7 +188,7 @@ def test_group(self, mock_request):
188188
mock_response.status_code = 200
189189
mock_request.return_value = mock_response
190190
fb = Farmbot()
191-
fb.set_token(MOCK_TOKEN)
191+
fb.state.token = MOCK_TOKEN
192192
group_info = fb.group(12345)
193193
mock_request.assert_called_once_with(
194194
'GET',
@@ -209,7 +209,7 @@ def test_curve(self, mock_request):
209209
mock_response.status_code = 200
210210
mock_request.return_value = mock_response
211211
fb = Farmbot()
212-
fb.set_token(MOCK_TOKEN)
212+
fb.state.token = MOCK_TOKEN
213213
curve_info = fb.curve(12345)
214214
mock_request.assert_called_once_with(
215215
'GET',
@@ -230,7 +230,7 @@ def test_safe_z(self, mock_request):
230230
mock_response.status_code = 200
231231
mock_request.return_value = mock_response
232232
fb = Farmbot()
233-
fb.set_token(MOCK_TOKEN)
233+
fb.state.token = MOCK_TOKEN
234234
safe_height = fb.safe_z()
235235
mock_request.assert_called_once_with(
236236
'GET',
@@ -256,7 +256,7 @@ def test_garden_size(self, mock_request):
256256
mock_response.status_code = 200
257257
mock_request.return_value = mock_response
258258
fb = Farmbot()
259-
fb.set_token(MOCK_TOKEN)
259+
fb.state.token = MOCK_TOKEN
260260
garden_size = fb.garden_size()
261261
mock_request.assert_called_once_with(
262262
'GET',
@@ -276,7 +276,7 @@ def test_log(self, mock_request):
276276
mock_response.status_code = 200
277277
mock_request.return_value = mock_response
278278
fb = Farmbot()
279-
fb.set_token(MOCK_TOKEN)
279+
fb.state.token = MOCK_TOKEN
280280
fb.log('test message', 'info', ['toast'])
281281
mock_request.assert_called_once_with(
282282
'POST',
@@ -298,7 +298,7 @@ def test_connect_broker(self, mock_mqtt):
298298
mock_client = Mock()
299299
mock_mqtt.return_value = mock_client
300300
fb = Farmbot()
301-
fb.set_token(MOCK_TOKEN)
301+
fb.state.token = MOCK_TOKEN
302302
fb.connect_broker()
303303
mock_client.username_pw_set.assert_called_once_with(
304304
username='device_0',
@@ -324,7 +324,7 @@ def test_start_listen(self, mock_mqtt):
324324
mock_client = Mock()
325325
mock_mqtt.return_value = mock_client
326326
fb = Farmbot()
327-
fb.set_token(MOCK_TOKEN)
327+
fb.state.token = MOCK_TOKEN
328328
fb.start_listen()
329329
mock_client.on_connect('', '', '', '')
330330

@@ -360,7 +360,7 @@ def send_command_test_helper(self, *args, **kwargs):
360360
mock_response.status_code = 200
361361
mock_request.return_value = mock_response
362362
fb = Farmbot()
363-
fb.set_token(MOCK_TOKEN)
363+
fb.state.token = MOCK_TOKEN
364364
execute_command(fb)
365365
expected_payload = {
366366
'kind': 'rpc_request',
@@ -883,7 +883,7 @@ def test_get_seed_tray_cell(self, mock_request, _mock_client):
883883
mock_response.status_code = 200
884884
mock_request.return_value = mock_response
885885
fb = Farmbot()
886-
fb.set_token(MOCK_TOKEN)
886+
fb.state.token = MOCK_TOKEN
887887
cell = fb.get_seed_tray_cell(123, 'd4')
888888
mock_request.assert_called_once_with(
889889
'GET',

0 commit comments

Comments
 (0)