Skip to content

Commit

Permalink
[ServiceBus] CI Test hotfixes (Azure#14195)
Browse files Browse the repository at this point in the history
* See if we can identify why the preparer is going sideways with the cosmos changes in CI.

* try using testclassinstance islive as a more proper approach.  if this doesn't work, may try a devtoolsutils import in a try catch as a final hail mairy for is_live() to match the decorator itself.

* Remove debug prints from preparer hotfix
  • Loading branch information
KieranBrantnerMagee authored Oct 2, 2020
1 parent 0895b0c commit 5c56b81
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ def _prepare_create_resource(self, test_class_instance, **kwargs):
# generated, so in_recording will be True even if live_test is false, so a random name would be given.
# In cached mode we need to avoid this because then for tests with recordings, they would not have a moniker.
if (self.live_test or test_class_instance.in_recording) \
and not (not self.live_test and test_class_instance.in_recording and self._use_cache):
and not (not test_class_instance.is_live and test_class_instance.in_recording and self._use_cache):
resource_name = self.random_name
if not self.live_test and isinstance(self, RecordingProcessor):
test_class_instance.recording_processors.append(self)
Expand Down

0 comments on commit 5c56b81

Please sign in to comment.