Add test on docker.#Run#1475
Conversation
There was a problem hiding this comment.
what is this testing that verifyOutput is not?
There was a problem hiding this comment.
It tests with another way and verify that the exported folder contains both files
There was a problem hiding this comment.
What error can verifyTest catch that verifyOutput cannot?
If the answer is none, this is a duplicate test which costs maintenance for no added benefit
There was a problem hiding this comment.
Okay I understand, I'll fix it
There was a problem hiding this comment.
nit: No need to #ReadFile here
verify: "hello world" & run.export.files."/test/output.txt".contents
There was a problem hiding this comment.
It was to be sure that we read the file and compare them
There was a problem hiding this comment.
if both #ReadFile and export fail in the same way (which they should, export uses #ReadFile underneath) then this test will pass ("" & "").
Signed-off-by: Vasek - Tom C <tom.chauveau@epitech.eu>
|
Thank you @TomChv! |
2 participants
Changes
Related to #1407, add the following list of test on
docker.#Run