Ensure that writing to a non existing output shows an error

The test added by this merge request ensure that writing to an output that does not exists is reported.

The error will be that "NoneType" as no property "write" which is expected following how outputs work.

Part of #99 (closed)

changed milestone to %Soft loops

@amohammadi Can you run the errors/user/single/1/write_in_wrong_output experiment ?

For that, you can modify test/__init__.py and add the following line:

sp.check_call(["rsync", "-arz", path, "/somewhere/beat/beat.test.prefix"])

under the original rsync call so you'll get the same prefix as the tests and you'll be able to run errors/user/single/1/write_in_wrong_output which should show you an error message when using beat/beat.cmdline> to run it.

Running `errors/write_in_wrong_output/1' for block `echo'
Start the execution of 'errors/write_in_wrong_output/1'
Block did not execute properly - outputs were reset
  Standard output:
    
  Standard error:
    
  Captured user error:
    File "/tmp/beat.core.execution.local_c8igoll/prefix/algorithms/errors/write_in_wrong_output/1.py", line 54, in process
        outputs["wrong_output"].write(
    AttributeError: 'NoneType' object has no attribute 'write'
  Captured system error:
    
Removing cache files: No data written
Error: Error occured: returned value is 1

why am I not hitting this issue: #98 (closed)

added 1 commit

• 2afba8ed - [test][execution] Ensure that writing to a non existing output shows an error

Compare with previous version

mentioned in merge request beat.backend.python!75 (merged)

added 1 commit

• bbd36507 - [test][execution] Ensure that writing to a non existing output shows an error

Compare with previous version

LGTM, merge when green.

enabled an automatic merge when the pipeline for bbd36507 succeeds

assigned to @samuel.gaist and unassigned @amohammadi

aborted the automatic merge because source branch was updated

added 1 commit

• 0561bd60 - [test][utils] Update docker test images version

Compare with previous version

added 1 commit

• 4de963e0 - [test][prefix] Update CXX algorithm minimal standard to C++14

Compare with previous version

Merging based on the prior comment.

Additional commits just ensure that:

• The correct environment are used for the test
• The CXX code builds successfully

None of these introduce behaviour changes with regard to the original goal of this merge request.

merged

mentioned in commit 4a4e2c50

mentioned in issue #99 (closed)