[6.15.z] Check that breadcrumb is displayed

[Satellite-QE]

Cherrypick of PR: #1281

error occurred in 6.15.z

[LadislavVasina1]

In my opinion, this does totally nothing.
Correct me if I am wrong.
It just returns bool and nothing is done with it.

Can't you just use self.browser.wait_for_element(self.breadcrumb, exception=False)

[dosas]

@LadislavVasina1 That is not correct it does not do nothing, it waits for the element to be displayed.

No, it makes much more sense to use the abstraction (https://github.com/SatelliteQE/airgun/blob/master/airgun/views/os.py#L81). That is why it is there.

What I generally do not understand is why a test should continue (exception=False) when an element is not found or why there is no implicit check for this. Since this makes tests very brittle.
#1247

[LadislavVasina1]

@dosas exception false is just from this implementation, that your implementation actually uses

airgun/airgun/views/contenthost.py

Lines 122 to 134 in 7995b52

	class ContentHostDetailsView(BaseLoggedInView):
	breadcrumb = BreadCrumb()
	unregister = Text(".//button[@ng-click='openModal()']")
	dialog = ConfirmationDialog()
	@property
	def is_displayed(self):
	breadcrumb_loaded = self.browser.wait_for_element(self.breadcrumb, exception=False)
	return (
	breadcrumb_loaded
	and self.breadcrumb.locations[0] == 'Content Hosts'
	and len(self.breadcrumb.locations) > 1
	)

See: #1281 (comment)

[LadislavVasina1]

exception false is set because you want to get True or False from this property and do with this information whatever you want

[omkarkhatavkar]

@dosas please see my comment here #1281 (comment)

[dosas]

Closed in favor of #1299

[omkarkhatavkar]

Closing this as per @dosas last comment