-
Notifications
You must be signed in to change notification settings - Fork 99
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Http Test can now post files. Usage: post file <fileName> to <url>. #49
Conversation
File is expected to reside in fitnesse's files folder.
Cool! I assume you will also update the documentation in the example pages. |
Documentation in example is updated and this method is now als called in the acceptanceTests |
@@ -5,6 +5,7 @@ The tests in this suite show example usage of this fixture: | |||
>HttpGetTest shows how an example GET requests, specifying parameters in the query string. | |||
>HttpPost1Test sends a POST request to a SOAP service, also adding an HTTP header. | |||
The >HttpPost2UsingScenarioTest makes multiple SOAP calls to a service, where scenario parameters are used to change an input variable's value for each call. | |||
>HttpPostFileTest sends a binary file to a mocked endpoint. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here I would just mention it posts a binary file, and in the page describe its to a mocked endpoint
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
… a file for post. Example test page added
Ik zie nog falen acceptatie tests in travis:
Time elapsed: 3.293 sec <<< FAILURE!
|
test.png was niet gecommit, daarom gaat de upload test mis. Staat nu wel in de tcnh branch in mijn fork. Gaat die niet automatisch mee met deze PR? |
Can we also support hyperlinks to the files section when posting? We probably need something like |
Should not be too hard to do that. It would make sense that this folder doesn't exist. Is this something we should fix in the build config or in code? The AcceptanceTest does succeed when it runs locally. |
The files section is moved when running tests in the CI environment. So far I mostly had files I downloaded during the test, and I wanted these to be available in the test report (and not a file in the files section available before the test started to be read by the test). To work around this for this test we can first download a file in a test, and place that in a variable, we can then use that variable as input for the post (the fixture will see these as a wiki url). |
nevermind. still broken, just with a different error |
@@ -35,6 +35,25 @@ public void setWaitAfterSwipe(int waitAfterSwipe) { | |||
this.waitAfterSwipe = waitAfterSwipe; | |||
} | |||
|
|||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any chance the changes to this class are meant for your other branch?
Http Test can now post files. Usage: post file to .
File is expected to reside in fitnesse's files folder. Acceptance test and demo test will follow.