trustchain-oc1-orchestral
/
devicehub-teal
Archived
10
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

changes messages for device add form

This commit is contained in:
Cayo Puigdefabregas 2022-09-22 13:16:14 +02:00
parent 9ac3781f82
commit d8e33e680d
2 changed files with 7 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
ereuse_devicehub/inventory/forms.py
View File

@ -481,7 +481,7 @@ class NewDeviceForm(FlaskForm):
Placeholder.phid == self.phid.data, Placeholder.owner == g.user Placeholder.phid == self.phid.data, Placeholder.owner == g.user
).first() ).first()
if dev: if dev:
msg = "Sorry, exist one snapshot device with this HID" msg = "Error, exist one Placeholder device with this PHID"
self.phid.errors = [msg] self.phid.errors = [msg]
is_valid = False is_valid = False
@ -494,7 +494,7 @@ class NewDeviceForm(FlaskForm):
Placeholder.phid == self.phid.data, Device.owner == g.user Placeholder.phid == self.phid.data, Device.owner == g.user
).first() ).first()
if dev: if dev:
msg = "Sorry, exist one snapshot device with this HID" msg = "Error, exist one Placeholder device with this PHID"
self.phid.errors = [msg] self.phid.errors = [msg]
is_valid = False is_valid = False
@ -908,7 +908,7 @@ class AllocateForm(ActionFormMixin):
return True return True
def check_deallocate(self): def check_deallocate(self):
txt = "Sorry some of this devices are actually deallocate" txt = "Error, some of this devices are actually deallocate"
for device in self._devices: for device in self._devices:
allocates = [ allocates = [
ac for ac in device.actions if ac.type in ['Allocate', 'Deallocate'] ac for ac in device.actions if ac.type in ['Allocate', 'Deallocate']

11
tests/test_render_2_0.py
View File

@ -268,7 +268,6 @@ def test_export_devices(user3: UserClientFlask):
fixture_csv[1][:29] == export_csv[1][:29] fixture_csv[1][:29] == export_csv[1][:29]
), 'Computer information are not equal' ), 'Computer information are not equal'
assert fixture_csv[1][30] == export_csv[1][30], 'Computer information are not equal' assert fixture_csv[1][30] == export_csv[1][30], 'Computer information are not equal'
# import pdb; pdb.set_trace()
assert ( assert (
fixture_csv[1][32:93] == export_csv[1][32:93] fixture_csv[1][32:93] == export_csv[1][32:93]
), 'Computer information are not equal' ), 'Computer information are not equal'
@ -577,7 +576,7 @@ def test_update_monitor(user3: UserClientFlask):
} }
body, status = user3.post(uri, data=data) body, status = user3.post(uri, data=data)
assert status == '200 OK' assert status == '200 OK'
assert 'Sorry, exist one snapshot device with this HID' in body assert 'Error, exist one Placeholder device with this PHID' in body
dev = Device.query.one() dev = Device.query.one()
assert dev.type == 'Monitor' assert dev.type == 'Monitor'
assert dev.placeholder.id_device_supplier == "b2" assert dev.placeholder.id_device_supplier == "b2"
@ -764,7 +763,7 @@ def test_add_laptop_duplicate(user3: UserClientFlask):
assert status == '200 OK' assert status == '200 OK'
assert Device.query.count() == 1 assert Device.query.count() == 1
body, status = user3.post(uri, data=data) body, status = user3.post(uri, data=data)
assert 'Sorry, exist one snapshot device with this HID' in body assert 'Error, exist one Placeholder device with this PHID' in body
assert Device.query.count() == 1 assert Device.query.count() == 1
@ -1109,7 +1108,7 @@ def test_action_deallocate_error(user3: UserClientFlask):
assert status == '200 OK' assert status == '200 OK'
assert dev.binding.device.allocated_status.type != 'Deallocate' assert dev.binding.device.allocated_status.type != 'Deallocate'
assert 'Action Deallocate error!' in body assert 'Action Deallocate error!' in body
assert 'Sorry some of this devices are actually deallocate' in body assert 'Error, some of this devices are actually deallocate' in body
@pytest.mark.mvp @pytest.mark.mvp
@ -1163,9 +1162,7 @@ def test_action_allocate_deallocate_error(user3: UserClientFlask):
user3.post(uri, data=data) user3.post(uri, data=data)
assert dev.binding.device.allocated_status.type == 'Deallocate' assert dev.binding.device.allocated_status.type == 'Deallocate'
# assert 'Action Deallocate error!' in body
# assert 'Sorry some of this devices are actually deallocate' in body
#
data = { data = {
'csrf_token': generate_csrf(), 'csrf_token': generate_csrf(),
'type': "Deallocate", 'type': "Deallocate",