Skip to content

Commit

Permalink
Fix: syncing data from a background process
Browse files Browse the repository at this point in the history
Will no longer work so update the relevant test
  • Loading branch information
jenshnielsen authored and giulioungaretti committed Jan 17, 2017
1 parent 17ef57e commit 0edc9ac
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion qcodes/tests/test_loop.py
Original file line number Diff line number Diff line change
Expand Up @@ -118,6 +118,9 @@ def test_local_instrument(self):
self.check_loop_data(data)

def test_background_no_datamanager(self):
# We don't support syncing data from a background process
# if not using a datamanager. See warning in ActiveLoop.run()
# So we expect the data to be empty even after running.
data = self.loop.run(location=self.location,
background=True,
data_manager=False,
Expand All @@ -127,7 +130,7 @@ def test_background_no_datamanager(self):
self.loop.process.join()

data.sync()
self.check_loop_data(data)
self.check_empty_data(data)

def test_foreground_and_datamanager(self):
data = self.loop.run(location=self.location, background=False,
Expand Down

0 comments on commit 0edc9ac

Please sign in to comment.