-
Notifications
You must be signed in to change notification settings - Fork 10
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
Updated tests to use getLightblueFactory() #63
Updated tests to use getLightblueFactory() #63
Conversation
@dcrissman I merged that PR, merged master to this PR, and local builds still fail. |
@@ -8,6 +8,6 @@ | |||
"objectClass": ["top", "groupofNames"], | |||
"cn": "#cn", | |||
"description": "#description", | |||
"member": [#members] | |||
"member": ["#members"] | |||
} |
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.
This change needs to be reverted. The test methods themselves are ensuring the quotation marks. I am fine if we want to change this, but lets do it in another issue/PR.
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.
That, or I suppose the unit tests could be changed to not include the surrounding quotation marks.
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.
Will revert, created #65 to fix
Issues should be fixed now, local build is successful at least.. ready for review. |
Updated tests to use getLightblueFactory()
Depends on lightblue-platform/lightblue-core#355 being merged.
Local builds are failing but not because of anything I changed.
@dcrissman if this still fails here, might need your eyes on it. It's complaining when trying to create metadata from test resource
metadata/datatype-metadata.json
.