-
Notifications
You must be signed in to change notification settings - Fork 7
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
MMT-3417: As a MMT dev, I want to be able to CRUD UMM-V records in CMR GraphQL #81
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #81 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 69 69
Lines 1516 1538 +22
Branches 208 210 +2
=========================================
+ Hits 1516 1538 +22 ☔ View full report in Codecov by Sentry. |
|
||
import { deleteTool as toolSourceDelete, fetchTools as toolSourceFetch } from '../datasources/tool' | ||
|
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.
remove extra line
@@ -25,14 +25,19 @@ import orderOptionSource from '../datasources/orderOption' | |||
import serviceDraftSource from '../datasources/serviceDraft' | |||
import toolDraftSource from '../datasources/toolDraft' | |||
import variableDraftSource from '../datasources/variableDraft' | |||
import variableSource from '../datasources/variable' | |||
|
|||
import { deleteTool as toolSourceDelete, fetchTools as toolSourceFetch } from '../datasources/tool' |
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.
try to split this
@@ -1759,6 +1813,55 @@ describe('Draft', () => { | |||
} | |||
}) | |||
}) | |||
|
|||
test('returns the cmr result when publishing a Publish Draft and collection concept not provided', async () => { |
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.
update test title
@@ -1759,6 +1813,55 @@ describe('Draft', () => { | |||
} | |||
}) | |||
}) | |||
|
|||
test('returns the cmr result when collection concept id not provided', async () => { |
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.
"returns an error when collection concept id not provided"
Overview
Added support for Publish Variable Draft and Published Delete Draft.
List impacted areas.
Testing
1.) Has a Published Collection Record handy.
2.) Publish a variable draft
Variables:
Testing Delete
Variables:
Checklist