-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feature/content package #2
Conversation
bogdanguranda
commented
Mar 17, 2017
•
edited
edited
- Added content-package support.
- Added UT and IT tests for content package.
- Update README.md
- Added vendoring.
# Conflicts: # relations/cypher.go
# Conflicts: # relations/cypher.go
…ntent as node labels. Added integration tests that are commented until content-collection-rw-neo4j is in production.
relations/cypher.go
Outdated
neoCRC := []neoRelatedContent{} | ||
//neo content package contained contents | ||
neoCPCC := []neoRelatedContent{} |
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.
I think it would be better if the naming remain consistent in the different vars (e.g. neoCPCC
vs cpcQuery
vs mappedCPC
.
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.
I agree, thx
relations/cypher_integration_test.go
Outdated
assert.Equal(t, len(expectedResponse.CuratedRelatedContents), len(actualCRC.CuratedRelatedContents), "Didn't get the same number of curated related content") | ||
assertListContainsAll(t, actualCRC.CuratedRelatedContents, expectedResponse.CuratedRelatedContents) | ||
} | ||
|
||
//func TestRetrieveContainsContent(t *testing.T) { |
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.
Commenting this out... I guess it's not deliberate.
Edit: just read the task description, ignore this.
f5965e2
to
0b87e1c
Compare