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

[ARRCONN-154] Added shema models and tests #21

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

vladimir-trifonov
Copy link
Contributor

@vladimir-trifonov
Copy link
Contributor Author

The rest was also added!

@vladimir-trifonov vladimir-trifonov force-pushed the ARRCONN-154 branch 3 times, most recently from 9003aa4 to 29f46f7 Compare December 19, 2016 11:51
Copy link
Collaborator

@janvennemann janvennemann left a comment

Choose a reason for hiding this comment

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

The Like model is still failing the unit tests for me:

  Like
    ✓ should create like based on place
Uncaught Exception expected undefined to be an object
AssertionError: expected undefined to be an object
    at Assertion.fail (/Users/jvennemann/Development/appc/appc.arrowdb/node_modules/should/lib/assertion.js:228:17)
    at Assertion.prop.(anonymous function) [as Object] (/Users/jvennemann/Development/appc/appc.arrowdb/node_modules/should/lib/assertion.js:69:14)
    at /Users/jvennemann/Development/appc/appc.arrowdb/test/test-like.js:58:23

Can you please check?

@vladimir-trifonov
Copy link
Contributor Author

vladimir-trifonov commented Jan 17, 2017

The following test returns the following error:

{ "errorCode":1004, "docUrl":"TBD", "message":"Request returned with HTTP status code 500 Internal server error. The Appcelerator Cloud Services Team has just been notified of this problem.", "statusCode":500, "reason":"Internal server error. The Appcelerator Cloud Services Team has just been notified of this problem.", "response":{ "statusCode":500, "body":"{\"meta\":{\"code\":500,\"status\":\"fail\",\"message\":\"Internal server error. The Appcelerator Cloud Services Team has just been notified of this problem.\",\"method_name\":\"queryLikes\"}}", "headers":{ "server":"nginx/1.8.1", "date":"Tue, 17 Jan 2017 11:42:43 GMT", "content-type":"application/json; charset=utf-8", "transfer-encoding":"chunked", "connection":"close", "status":"500", "x-powered-by":"Phusion Passenger (mod_rails/mod_rack) 3.0.9", "access-control-allow-methods":"POST, GET, PUT, DELETE, OPTIONS", "access-control-allow-headers":"x-requested-with, Authorization", "access-control-allow-credentials":"true", "cache-control":"no-cache", "x-ua-compatible":"IE=Edge,chrome=1", "x-runtime":"0.347916" }, "request":{ "uri":{ "protocol":"https:", "slashes":true, "auth":null, "host":"api.cloud.appcelerator.com", "port":null, "hostname":"api.cloud.appcelerator.com", "hash":null, "search":"?key=xxx&where=%7B%7D&per_page=10&limit=10&page=1&skip=0", "query":"key=xxx&where=%7B%7D&per_page=10&limit=10&page=1&skip=0", "pathname":"/v1/likes/query.json", "path":"/v1/likes/query.json?key=xxx&where=%7B%7D&per_page=10&limit=10&page=1&skip=0", "href":"https://api.cloud.appcelerator.com/v1/likes/query.json?key=sX5SETgQP9470gddvEIBrgmiQS9p7TPd&where=%7B%7D&per_page=10&limit=10&page=1&skip=0" }, "method":"GET", "headers":{ "cookie":"_session_id=xxx" } } }, "body":{ "meta":{ "code":500, "status":"fail", "message":"Internal server error. The Appcelerator Cloud Services Team has just been notified of this problem.", "method_name":"queryLikes" } } }

# Conflicts:
#	package.json
@vladimir-trifonov
Copy link
Contributor Author

I have reverted the commit where the chat unit test were skipped, because I guess it was an issue related to the arrowdb restriction found by you.

@janvennemann
Copy link
Collaborator

Although the chat model doesn't allow batch operations it could indeed be possible that this was caused by the API rate limiting. I'll do a new FR when #23 is merged.

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.

2 participants