Craftadoc/clsi
2
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

move texliveImageNameOveride further down request so it works for

Browse Source 
compile tests
This commit is contained in:
Henry Oswald
2018-07-13 11:52:49 +01:00
parent 3545852173
commit 8d846f64a9
3 changed files with 10 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
test/unit/coffee/DockerRunnerTests.coffee
View File

@@ -135,6 +135,16 @@ describe "DockerRunner", ->
@DockerRunner._getContainerOptions
.calledWith(@command_with_dir, @defaultImage, @volumes, @timeout)
.should.equal true
describe "with image override", ->
beforeEach ->
@Settings.texliveImageNameOveride = "overrideimage/here"
@DockerRunner._runAndWaitForContainer = sinon.stub().callsArgWith(3, null, @output = "mock-output")
@DockerRunner.run @project_id, @command, @directory, @image, @timeout, @env, @callback
it "should use the override and keep the tag", ->
image = @DockerRunner._getContainerOptions.args[0][1]
image.should.equal "overrideimage/here:2016.2"
describe "_runAndWaitForContainer", ->
beforeEach ->

8
test/unit/coffee/RequestParserTests.coffee
View File

@@ -66,14 +66,6 @@ describe "RequestParser", ->
it "should set the imageName", ->
@data.imageName.should.equal "basicImageName/here:2017-1"
describe "with texliveImageNameOveride set", ->
beforeEach ->
@settings.texliveImageNameOveride = "usethisoveride/overhere"
@RequestParser.parse @validRequest, (error, @data) =>
it "should override the image path", ->
@data.imageName.should.equal "usethisoveride/overhere:2017-1"
describe "without a timeout specified", ->
beforeEach ->
delete @validRequest.compile.options.timeout