Commit 9ca1673d authored by dbeltran's avatar dbeltran
Browse files

Fixed additional_content in template_content

parent e08c8197
  • @dbeltran this commit broke the test pipeline on master.

    ======================================================================
    FAIL: test_create_script (test.unit.test_job.TestJob)
    ----------------------------------------------------------------------
    Traceback (most recent call last):
      File "/home/kinow/Development/python/workspace/autosubmit/test/unit/test_job.py", line 188, in test_create_script
        write_mock.write.assert_called_with(b'some-content: 999, 777, 666 % %some-content: 999, 777, 666 % %')
      File "/home/kinow/mambaforge/envs/autosubmit4/lib/python3.7/site-packages/mock/mock.py", line 945, in assert_called_with
        raise AssertionError(_error_message()) from cause
    AssertionError: expected call not found.
    Expected: write(b'some-content: 999, 777, 666 % %some-content: 999, 777, 666 % %')
    Actual: write(b'some-content: 999, 777, 666 % %')

    I think that test may need to be updated now too?

  • mentioned in merge request !316 (merged)

    Toggle commit list
  • Author Maintainer

    Right, this test was not working correctly since the additional_files introduction and now it crashed because of the changes.

    I've fixed it for the executable file(on master), but the additional_file mock is still not implemented

    Edited by dbeltran
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment