Skip to content
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

Added helper methods to set up the ACLs and Authorizations for testing. #26

Closed
wants to merge 3 commits into from

Conversation

peichman-umd
Copy link
Contributor

Note: building this branch still fails due to the java.lang.VerifyError: Bad type on operand stack error. But I wanted to share this with the team to get your input.

@awoods
Copy link
Member

awoods commented Sep 4, 2015

I am not seeing a VerifyError. Rather, when I run mvn clean install locally with this PR I get:

Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 10.042 sec <<< FAILURE! - in org.fcrepo.integration.auth.webac.WebACRecipesIT
test(org.fcrepo.integration.auth.webac.WebACRecipesIT)  Time elapsed: 3.413 sec  <<< ERROR!
java.io.IOException: POST to http://localhost:33012/rest/ to create /acls/01/acl.ttl FAILED
        at org.fcrepo.integration.auth.webac.WebACRecipesIT.ingestTurtleResource(WebACRecipesIT.java:145)
        at org.fcrepo.integration.auth.webac.WebACRecipesIT.ingestAcl(WebACRecipesIT.java:96)
        at org.fcrepo.integration.auth.webac.WebACRecipesIT.setUp(WebACRecipesIT.java:73)

@peichman-umd
Copy link
Contributor Author

@awoods, the VerifyError is still occurring for me when I run the test through Eclipse. The IOException causing the build failure is happening because the request is not returning a 201 response (its a 500 instead), see line 144

@awoods
Copy link
Member

awoods commented Sep 4, 2015

Running through Eclipse??!!
Give the command line a shot.

Changed POST to PUT, changed IOException to assertion of status code value.
* the HTTP response from that request. Throws an IOException if the server responds with anything other than a
* 201 Created response code.
*
* @param credentials
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove these empty @param statements.

@awoods
Copy link
Member

awoods commented Sep 4, 2015

Superseded by: #27

@awoods awoods closed this Sep 4, 2015
@peichman-umd peichman-umd deleted the it-setup branch October 23, 2015 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants