Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
G
gitlab-ce
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
1
Merge Requests
1
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
nexedi
gitlab-ce
Commits
9079b29a
Commit
9079b29a
authored
Apr 28, 2021
by
Nicolò Maria Mezzopera
Committed by
Natalia Tepluhina
Apr 28, 2021
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Container Registry Details: split details from tags call
parent
a5c6976c
Changes
8
Hide whitespace changes
Inline
Side-by-side
Showing
8 changed files
with
385 additions
and
244 deletions
+385
-244
app/assets/javascripts/registry/explorer/components/details_page/tags_list.vue
...s/registry/explorer/components/details_page/tags_list.vue
+124
-31
app/assets/javascripts/registry/explorer/graphql/queries/get_container_repository_details.query.graphql
...ql/queries/get_container_repository_details.query.graphql
+1
-25
app/assets/javascripts/registry/explorer/graphql/queries/get_container_repository_tags.query.graphql
...aphql/queries/get_container_repository_tags.query.graphql
+29
-0
app/assets/javascripts/registry/explorer/pages/details.vue
app/assets/javascripts/registry/explorer/pages/details.vue
+11
-65
changelogs/unreleased/296962-container-registry-details-split-details-from-tags-call.yml
...ntainer-registry-details-split-details-from-tags-call.yml
+5
-0
spec/frontend/registry/explorer/components/details_page/tags_list_spec.js
...gistry/explorer/components/details_page/tags_list_spec.js
+197
-21
spec/frontend/registry/explorer/mock_data.js
spec/frontend/registry/explorer/mock_data.js
+14
-0
spec/frontend/registry/explorer/pages/details_spec.js
spec/frontend/registry/explorer/pages/details_spec.js
+4
-102
No files found.
app/assets/javascripts/registry/explorer/components/details_page/tags_list.vue
View file @
9079b29a
<
script
>
import
{
GlButton
}
from
'
@gitlab/ui
'
;
import
{
REMOVE_TAGS_BUTTON_TITLE
,
TAGS_LIST_TITLE
}
from
'
../../constants/index
'
;
import
{
GlButton
,
GlKeysetPagination
}
from
'
@gitlab/ui
'
;
import
createFlash
from
'
~/flash
'
;
import
{
joinPaths
}
from
'
~/lib/utils/url_utility
'
;
import
{
REMOVE_TAGS_BUTTON_TITLE
,
TAGS_LIST_TITLE
,
GRAPHQL_PAGE_SIZE
,
FETCH_IMAGES_LIST_ERROR_MESSAGE
,
}
from
'
../../constants/index
'
;
import
getContainerRepositoryTagsQuery
from
'
../../graphql/queries/get_container_repository_tags.query.graphql
'
;
import
EmptyState
from
'
./empty_state.vue
'
;
import
TagsListRow
from
'
./tags_list_row.vue
'
;
import
TagsLoader
from
'
./tags_loader.vue
'
;
export
default
{
name
:
'
TagsList
'
,
components
:
{
GlButton
,
GlKeysetPagination
,
TagsListRow
,
EmptyState
,
TagsLoader
,
},
inject
:
[
'
config
'
],
props
:
{
tags
:
{
type
:
Array
,
required
:
false
,
default
:
()
=>
[],
id
:
{
type
:
[
Number
,
String
],
required
:
true
,
},
isMobile
:
{
type
:
Boolean
,
...
...
@@ -25,17 +38,46 @@ export default {
default
:
false
,
required
:
false
,
},
isImageLoading
:
{
type
:
Boolean
,
default
:
false
,
required
:
false
,
},
},
i18n
:
{
REMOVE_TAGS_BUTTON_TITLE
,
TAGS_LIST_TITLE
,
},
apollo
:
{
containerRepository
:
{
query
:
getContainerRepositoryTagsQuery
,
variables
()
{
return
this
.
queryVariables
;
},
error
()
{
createFlash
({
message
:
FETCH_IMAGES_LIST_ERROR_MESSAGE
});
},
},
},
data
()
{
return
{
selectedItems
:
{},
containerRepository
:
{},
};
},
computed
:
{
tags
()
{
return
this
.
containerRepository
?.
tags
?.
nodes
||
[];
},
tagsPageInfo
()
{
return
this
.
containerRepository
?.
tags
?.
pageInfo
;
},
queryVariables
()
{
return
{
id
:
joinPaths
(
this
.
config
.
gidPrefix
,
`
${
this
.
id
}
`
),
first
:
GRAPHQL_PAGE_SIZE
,
};
},
hasSelectedItems
()
{
return
this
.
tags
.
some
((
tag
)
=>
this
.
selectedItems
[
tag
.
name
]);
},
...
...
@@ -45,42 +87,93 @@ export default {
multiDeleteButtonIsDisabled
()
{
return
!
this
.
hasSelectedItems
||
this
.
disabled
;
},
showPagination
()
{
return
this
.
tagsPageInfo
.
hasPreviousPage
||
this
.
tagsPageInfo
.
hasNextPage
;
},
hasNoTags
()
{
return
this
.
tags
.
length
===
0
;
},
isLoading
()
{
return
this
.
isImageLoading
||
this
.
$apollo
.
queries
.
containerRepository
.
loading
;
},
},
methods
:
{
updateSelectedItems
(
name
)
{
this
.
$set
(
this
.
selectedItems
,
name
,
!
this
.
selectedItems
[
name
]);
},
mapTagsToBeDleeted
(
items
)
{
return
this
.
tags
.
filter
((
tag
)
=>
items
[
tag
.
name
]);
},
fetchNextPage
()
{
this
.
$apollo
.
queries
.
containerRepository
.
fetchMore
({
variables
:
{
after
:
this
.
tagsPageInfo
?.
endCursor
,
first
:
GRAPHQL_PAGE_SIZE
,
},
updateQuery
(
previousResult
,
{
fetchMoreResult
})
{
return
fetchMoreResult
;
},
});
},
fetchPreviousPage
()
{
this
.
$apollo
.
queries
.
containerRepository
.
fetchMore
({
variables
:
{
first
:
null
,
before
:
this
.
tagsPageInfo
?.
startCursor
,
last
:
GRAPHQL_PAGE_SIZE
,
},
updateQuery
(
previousResult
,
{
fetchMoreResult
})
{
return
fetchMoreResult
;
},
});
},
},
};
</
script
>
<
template
>
<div>
<div
class=
"gl-display-flex gl-justify-content-space-between gl-mb-3"
>
<h5
data-testid=
"list-title"
>
{{
$options
.
i18n
.
TAGS_LIST_TITLE
}}
</h5>
<tags-loader
v-if=
"isLoading"
/>
<template
v-else
>
<empty-state
v-if=
"hasNoTags"
:no-containers-image=
"config.noContainersImage"
/>
<template
v-else
>
<div
class=
"gl-display-flex gl-justify-content-space-between gl-mb-3"
>
<h5
data-testid=
"list-title"
>
{{
$options
.
i18n
.
TAGS_LIST_TITLE
}}
</h5>
<gl-button
v-if=
"showMultiDeleteButton"
:disabled=
"multiDeleteButtonIsDisabled"
category=
"secondary"
variant=
"danger"
@
click=
"$emit('delete', selectedItems)"
>
{{
$options
.
i18n
.
REMOVE_TAGS_BUTTON_TITLE
}}
</gl-button>
</div>
<tags-list-row
v-for=
"(tag, index) in tags"
:key=
"tag.path"
:tag=
"tag"
:first=
"index === 0"
:selected=
"selectedItems[tag.name]"
:is-mobile=
"isMobile"
:disabled=
"disabled"
@
select=
"updateSelectedItems(tag.name)"
@
delete=
"$emit('delete',
{ [tag.name]: true })"
/>
<gl-button
v-if=
"showMultiDeleteButton"
:disabled=
"multiDeleteButtonIsDisabled"
category=
"secondary"
variant=
"danger"
@
click=
"$emit('delete', mapTagsToBeDleeted(selectedItems))"
>
{{
$options
.
i18n
.
REMOVE_TAGS_BUTTON_TITLE
}}
</gl-button>
</div>
<tags-list-row
v-for=
"(tag, index) in tags"
:key=
"tag.path"
:tag=
"tag"
:first=
"index === 0"
:selected=
"selectedItems[tag.name]"
:is-mobile=
"isMobile"
:disabled=
"disabled"
@
select=
"updateSelectedItems(tag.name)"
@
delete=
"$emit('delete', mapTagsToBeDleeted(
{ [tag.name]: true }))"
/>
<div
class=
"gl-display-flex gl-justify-content-center"
>
<gl-keyset-pagination
v-if=
"showPagination"
:has-next-page=
"tagsPageInfo.hasNextPage"
:has-previous-page=
"tagsPageInfo.hasPreviousPage"
class=
"gl-mt-3"
@
prev=
"fetchPreviousPage"
@
next=
"fetchNextPage"
/>
</div>
</
template
>
</template>
</div>
</template>
app/assets/javascripts/registry/explorer/graphql/queries/get_container_repository_details.query.graphql
View file @
9079b29a
#import "~/graphql_shared/fragments/pageInfo.fragment.graphql"
query
getContainerRepositoryDetails
(
$id
:
ID
!
$first
:
Int
$last
:
Int
$after
:
String
$before
:
String
)
{
query
getContainerRepositoryDetails
(
$id
:
ID
!)
{
containerRepository
(
id
:
$id
)
{
id
name
...
...
@@ -19,22 +11,6 @@ query getContainerRepositoryDetails(
tagsCount
expirationPolicyStartedAt
expirationPolicyCleanupStatus
tags
(
after
:
$after
,
before
:
$before
,
first
:
$first
,
last
:
$last
)
{
nodes
{
digest
location
path
name
revision
shortRevision
createdAt
totalSize
canDelete
}
pageInfo
{
...
PageInfo
}
}
project
{
visibility
containerExpirationPolicy
{
...
...
app/assets/javascripts/registry/explorer/graphql/queries/get_container_repository_tags.query.graphql
0 → 100644
View file @
9079b29a
#import "~/graphql_shared/fragments/pageInfo.fragment.graphql"
query
getContainerRepositoryDetails
(
$id
:
ID
!
$first
:
Int
$last
:
Int
$after
:
String
$before
:
String
)
{
containerRepository
(
id
:
$id
)
{
id
tags
(
after
:
$after
,
before
:
$before
,
first
:
$first
,
last
:
$last
)
{
nodes
{
digest
location
path
name
revision
shortRevision
createdAt
totalSize
canDelete
}
pageInfo
{
...
PageInfo
}
}
}
}
app/assets/javascripts/registry/explorer/pages/details.vue
View file @
9079b29a
<
script
>
import
{
Gl
KeysetPagination
,
Gl
ResizeObserverDirective
}
from
'
@gitlab/ui
'
;
import
{
GlResizeObserverDirective
}
from
'
@gitlab/ui
'
;
import
{
GlBreakpointInstance
}
from
'
@gitlab/ui/dist/utils
'
;
import
createFlash
from
'
~/flash
'
;
import
axios
from
'
~/lib/utils/axios_utils
'
;
...
...
@@ -21,7 +21,6 @@ import {
ALERT_SUCCESS_TAGS
,
ALERT_DANGER_TAGS
,
ALERT_DANGER_IMAGE
,
GRAPHQL_PAGE_SIZE
,
FETCH_IMAGES_LIST_ERROR_MESSAGE
,
UNFINISHED_STATUS
,
MISSING_OR_DELETED_IMAGE_BREADCRUMB
,
...
...
@@ -36,7 +35,6 @@ export default {
DeleteAlert
,
PartialCleanupAlert
,
DetailsHeader
,
GlKeysetPagination
,
DeleteModal
,
TagsList
,
TagsLoader
,
...
...
@@ -58,8 +56,7 @@ export default {
update
(
data
)
{
return
data
.
containerRepository
;
},
result
({
data
})
{
this
.
tagsPageInfo
=
data
.
containerRepository
?.
tags
?.
pageInfo
;
result
()
{
this
.
updateBreadcrumb
();
},
error
()
{
...
...
@@ -70,7 +67,6 @@ export default {
data
()
{
return
{
image
:
{},
tagsPageInfo
:
{},
itemsToBeDeleted
:
[],
isMobile
:
false
,
mutationLoading
:
false
,
...
...
@@ -83,15 +79,11 @@ export default {
queryVariables
()
{
return
{
id
:
joinPaths
(
this
.
config
.
gidPrefix
,
`
${
this
.
$route
.
params
.
id
}
`
),
first
:
GRAPHQL_PAGE_SIZE
,
};
},
isLoading
()
{
return
this
.
$apollo
.
queries
.
image
.
loading
||
this
.
mutationLoading
;
},
tags
()
{
return
this
.
image
?.
tags
?.
nodes
||
[];
},
showPartialCleanupWarning
()
{
return
(
this
.
config
.
showUnfinishedTagCleanupCallout
&&
...
...
@@ -105,12 +97,6 @@ export default {
this
.
itemsToBeDeleted
?.
length
>
1
?
'
bulk_registry_tag_delete
'
:
'
registry_tag_delete
'
,
};
},
showPagination
()
{
return
this
.
tagsPageInfo
.
hasPreviousPage
||
this
.
tagsPageInfo
.
hasNextPage
;
},
hasNoTags
()
{
return
this
.
tags
.
length
===
0
;
},
pageActionsAreDisabled
()
{
return
Boolean
(
this
.
image
?.
status
);
},
...
...
@@ -124,7 +110,7 @@ export default {
},
deleteTags
(
toBeDeleted
)
{
this
.
deleteImageAlert
=
false
;
this
.
itemsToBeDeleted
=
t
his
.
tags
.
filter
((
tag
)
=>
toBeDeleted
[
tag
.
name
])
;
this
.
itemsToBeDeleted
=
t
oBeDeleted
;
this
.
track
(
'
click_button
'
);
this
.
$refs
.
deleteModal
.
show
();
},
...
...
@@ -170,33 +156,6 @@ export default {
handleResize
()
{
this
.
isMobile
=
GlBreakpointInstance
.
getBreakpointSize
()
===
'
xs
'
;
},
fetchNextPage
()
{
if
(
this
.
tagsPageInfo
?.
hasNextPage
)
{
this
.
$apollo
.
queries
.
image
.
fetchMore
({
variables
:
{
after
:
this
.
tagsPageInfo
?.
endCursor
,
first
:
GRAPHQL_PAGE_SIZE
,
},
updateQuery
(
previousResult
,
{
fetchMoreResult
})
{
return
fetchMoreResult
;
},
});
}
},
fetchPreviousPage
()
{
if
(
this
.
tagsPageInfo
?.
hasPreviousPage
)
{
this
.
$apollo
.
queries
.
image
.
fetchMore
({
variables
:
{
first
:
null
,
before
:
this
.
tagsPageInfo
?.
startCursor
,
last
:
GRAPHQL_PAGE_SIZE
,
},
updateQuery
(
previousResult
,
{
fetchMoreResult
})
{
return
fetchMoreResult
;
},
});
}
},
dismissPartialCleanupWarning
()
{
this
.
hidePartialCleanupWarning
=
true
;
axios
.
post
(
this
.
config
.
userCalloutsPath
,
{
...
...
@@ -246,27 +205,14 @@ export default {
/>
<tags-loader
v-if=
"isLoading"
/>
<template
v-else
>
<empty-state
v-if=
"hasNoTags"
:no-containers-image=
"config.noContainersImage"
/>
<template
v-else
>
<tags-list
:tags=
"tags"
:is-mobile=
"isMobile"
:disabled=
"pageActionsAreDisabled"
@
delete=
"deleteTags"
/>
<div
class=
"gl-display-flex gl-justify-content-center"
>
<gl-keyset-pagination
v-if=
"showPagination"
:has-next-page=
"tagsPageInfo.hasNextPage"
:has-previous-page=
"tagsPageInfo.hasPreviousPage"
class=
"gl-mt-3"
@
prev=
"fetchPreviousPage"
@
next=
"fetchNextPage"
/>
</div>
</
template
>
</template>
<tags-list
v-else
:id=
"$route.params.id"
:is-image-loading=
"isLoading"
:is-mobile=
"isMobile"
:disabled=
"pageActionsAreDisabled"
@
delete=
"deleteTags"
/>
<delete-image
:id=
"image.id"
...
...
changelogs/unreleased/296962-container-registry-details-split-details-from-tags-call.yml
0 → 100644
View file @
9079b29a
---
title
:
'
Container
Registry
Details:
split
details
from
tags
call'
merge_request
:
59969
author
:
type
:
changed
spec/frontend/registry/explorer/components/details_page/tags_list_spec.js
View file @
9079b29a
import
{
GlButton
}
from
'
@gitlab/ui
'
;
import
{
shallowMount
}
from
'
@vue/test-utils
'
;
import
{
GlButton
,
GlKeysetPagination
}
from
'
@gitlab/ui
'
;
import
{
shallowMount
,
createLocalVue
}
from
'
@vue/test-utils
'
;
import
{
nextTick
}
from
'
vue
'
;
import
VueApollo
from
'
vue-apollo
'
;
import
createMockApollo
from
'
helpers/mock_apollo_helper
'
;
import
waitForPromises
from
'
helpers/wait_for_promises
'
;
import
EmptyTagsState
from
'
~/registry/explorer/components/details_page/empty_state.vue
'
;
import
component
from
'
~/registry/explorer/components/details_page/tags_list.vue
'
;
import
TagsListRow
from
'
~/registry/explorer/components/details_page/tags_list_row.vue
'
;
import
TagsLoader
from
'
~/registry/explorer/components/details_page/tags_loader.vue
'
;
import
{
TAGS_LIST_TITLE
,
REMOVE_TAGS_BUTTON_TITLE
}
from
'
~/registry/explorer/constants/index
'
;
import
{
tagsMock
}
from
'
../../mock_data
'
;
import
getContainerRepositoryTagsQuery
from
'
~/registry/explorer/graphql/queries/get_container_repository_tags.query.graphql
'
;
import
{
tagsMock
,
imageTagsMock
,
tagsPageInfo
}
from
'
../../mock_data
'
;
const
localVue
=
createLocalVue
();
describe
(
'
Tags List
'
,
()
=>
{
let
wrapper
;
let
apolloProvider
;
const
tags
=
[...
tagsMock
];
const
readOnlyTags
=
tags
.
map
((
t
)
=>
({
...
t
,
canDelete
:
false
}));
const
findTagsListRow
=
()
=>
wrapper
.
findAll
(
TagsListRow
);
const
findDeleteButton
=
()
=>
wrapper
.
find
(
GlButton
);
const
findListTitle
=
()
=>
wrapper
.
find
(
'
[data-testid="list-title"]
'
);
const
findPagination
=
()
=>
wrapper
.
find
(
GlKeysetPagination
);
const
findEmptyState
=
()
=>
wrapper
.
find
(
EmptyTagsState
);
const
findTagsLoader
=
()
=>
wrapper
.
find
(
TagsLoader
);
const
waitForApolloRequestRender
=
async
()
=>
{
await
waitForPromises
();
await
nextTick
();
};
const
mountComponent
=
({
propsData
=
{
isMobile
:
false
,
id
:
1
},
resolver
=
jest
.
fn
().
mockResolvedValue
(
imageTagsMock
()),
}
=
{})
=>
{
localVue
.
use
(
VueApollo
);
const
requestHandlers
=
[[
getContainerRepositoryTagsQuery
,
resolver
]];
const
mountComponent
=
(
propsData
=
{
tags
,
isMobile
:
false
})
=>
{
apolloProvider
=
createMockApollo
(
requestHandlers
);
wrapper
=
shallowMount
(
component
,
{
localVue
,
apolloProvider
,
propsData
,
provide
()
{
return
{
config
:
{},
};
},
});
};
...
...
@@ -26,15 +59,19 @@ describe('Tags List', () => {
});
describe
(
'
List title
'
,
()
=>
{
it
(
'
exists
'
,
()
=>
{
it
(
'
exists
'
,
async
()
=>
{
mountComponent
();
await
waitForApolloRequestRender
();
expect
(
findListTitle
().
exists
()).
toBe
(
true
);
});
it
(
'
has the correct text
'
,
()
=>
{
it
(
'
has the correct text
'
,
async
()
=>
{
mountComponent
();
await
waitForApolloRequestRender
();
expect
(
findListTitle
().
text
()).
toBe
(
TAGS_LIST_TITLE
);
});
});
...
...
@@ -48,21 +85,29 @@ describe('Tags List', () => {
${
readOnlyTags
}
|
${
true
}
|
${
false
}
`
(
'
is $isVisible that delete button exists when tags is $inputTags and isMobile is $isMobile
'
,
({
inputTags
,
isMobile
,
isVisible
})
=>
{
mountComponent
({
tags
:
inputTags
,
isMobile
});
async
({
inputTags
,
isMobile
,
isVisible
})
=>
{
mountComponent
({
propsData
:
{
tags
:
inputTags
,
isMobile
,
id
:
1
},
resolver
:
jest
.
fn
().
mockResolvedValue
(
imageTagsMock
(
inputTags
)),
});
await
waitForApolloRequestRender
();
expect
(
findDeleteButton
().
exists
()).
toBe
(
isVisible
);
},
);
it
(
'
has the correct text
'
,
()
=>
{
it
(
'
has the correct text
'
,
async
()
=>
{
mountComponent
();
await
waitForApolloRequestRender
();
expect
(
findDeleteButton
().
text
()).
toBe
(
REMOVE_TAGS_BUTTON_TITLE
);
});
it
(
'
has the correct props
'
,
()
=>
{
it
(
'
has the correct props
'
,
async
()
=>
{
mountComponent
();
await
waitForApolloRequestRender
();
expect
(
findDeleteButton
().
attributes
()).
toMatchObject
({
category
:
'
secondary
'
,
...
...
@@ -79,35 +124,44 @@ describe('Tags List', () => {
`
(
'
is $buttonDisabled that the button is disabled when the component disabled state is $disabled and is $doSelect that the user selected a tag
'
,
async
({
disabled
,
buttonDisabled
,
doSelect
})
=>
{
mountComponent
({
tags
,
disabled
,
isMobile
:
false
});
mountComponent
({
propsData
:
{
tags
,
disabled
,
isMobile
:
false
,
id
:
1
}
});
await
waitForApolloRequestRender
();
if
(
doSelect
)
{
findTagsListRow
().
at
(
0
).
vm
.
$emit
(
'
select
'
);
await
wrapper
.
vm
.
$
nextTick
();
await
nextTick
();
}
expect
(
findDeleteButton
().
attributes
(
'
disabled
'
)).
toBe
(
buttonDisabled
);
},
);
it
(
'
click event emits a deleted event with selected items
'
,
()
=>
{
it
(
'
click event emits a deleted event with selected items
'
,
async
()
=>
{
mountComponent
();
findTagsListRow
().
at
(
0
).
vm
.
$emit
(
'
select
'
);
await
waitForApolloRequestRender
();
findTagsListRow
().
at
(
0
).
vm
.
$emit
(
'
select
'
);
findDeleteButton
().
vm
.
$emit
(
'
click
'
);
expect
(
wrapper
.
emitted
(
'
delete
'
)).
toEqual
([[{
'
beta-24753
'
:
true
}]]);
expect
(
wrapper
.
emitted
(
'
delete
'
)[
0
][
0
][
0
].
name
).
toBe
(
tags
[
0
].
name
);
});
});
describe
(
'
list rows
'
,
()
=>
{
it
(
'
one row exist for each tag
'
,
()
=>
{
it
(
'
one row exist for each tag
'
,
async
()
=>
{
mountComponent
();
await
waitForApolloRequestRender
();
expect
(
findTagsListRow
()).
toHaveLength
(
tags
.
length
);
});
it
(
'
the correct props are bound to it
'
,
()
=>
{
mountComponent
({
tags
,
disabled
:
true
});
it
(
'
the correct props are bound to it
'
,
async
()
=>
{
mountComponent
({
propsData
:
{
disabled
:
true
,
id
:
1
}
});
await
waitForApolloRequestRender
();
const
rows
=
findTagsListRow
();
...
...
@@ -120,16 +174,138 @@ describe('Tags List', () => {
describe
(
'
events
'
,
()
=>
{
it
(
'
select event update the selected items
'
,
async
()
=>
{
mountComponent
();
await
waitForApolloRequestRender
();
findTagsListRow
().
at
(
0
).
vm
.
$emit
(
'
select
'
);
await
wrapper
.
vm
.
$nextTick
();
await
nextTick
();
expect
(
findTagsListRow
().
at
(
0
).
attributes
(
'
selected
'
)).
toBe
(
'
true
'
);
});
it
(
'
delete event emit a delete event
'
,
()
=>
{
it
(
'
delete event emit a delete event
'
,
async
()
=>
{
mountComponent
();
await
waitForApolloRequestRender
();
findTagsListRow
().
at
(
0
).
vm
.
$emit
(
'
delete
'
);
expect
(
wrapper
.
emitted
(
'
delete
'
)).
toEqual
([[{
'
beta-24753
'
:
true
}]]);
expect
(
wrapper
.
emitted
(
'
delete
'
)[
0
][
0
][
0
].
name
).
toBe
(
tags
[
0
].
name
);
});
});
});
describe
(
'
when the list of tags is empty
'
,
()
=>
{
const
resolver
=
jest
.
fn
().
mockResolvedValue
(
imageTagsMock
([]));
it
(
'
has the empty state
'
,
async
()
=>
{
mountComponent
({
resolver
});
await
waitForApolloRequestRender
();
expect
(
findEmptyState
().
exists
()).
toBe
(
true
);
});
it
(
'
does not show the loader
'
,
async
()
=>
{
mountComponent
({
resolver
});
await
waitForApolloRequestRender
();
expect
(
findTagsLoader
().
exists
()).
toBe
(
false
);
});
it
(
'
does not show the list
'
,
async
()
=>
{
mountComponent
({
resolver
});
await
waitForApolloRequestRender
();
expect
(
findTagsListRow
().
exists
()).
toBe
(
false
);
expect
(
findListTitle
().
exists
()).
toBe
(
false
);
});
});
describe
(
'
pagination
'
,
()
=>
{
it
(
'
exists
'
,
async
()
=>
{
mountComponent
();
await
waitForApolloRequestRender
();
expect
(
findPagination
().
exists
()).
toBe
(
true
);
});
it
(
'
is hidden when loading
'
,
()
=>
{
mountComponent
();
expect
(
findPagination
().
exists
()).
toBe
(
false
);
});
it
(
'
is hidden when there are no more pages
'
,
async
()
=>
{
mountComponent
({
resolver
:
jest
.
fn
().
mockResolvedValue
(
imageTagsMock
([]))
});
await
waitForApolloRequestRender
();
expect
(
findPagination
().
exists
()).
toBe
(
false
);
});
it
(
'
is wired to the correct pagination props
'
,
async
()
=>
{
mountComponent
();
await
waitForApolloRequestRender
();
expect
(
findPagination
().
props
()).
toMatchObject
({
hasNextPage
:
tagsPageInfo
.
hasNextPage
,
hasPreviousPage
:
tagsPageInfo
.
hasPreviousPage
,
});
});
it
(
'
fetch next page when user clicks next
'
,
async
()
=>
{
const
resolver
=
jest
.
fn
().
mockResolvedValue
(
imageTagsMock
());
mountComponent
({
resolver
});
await
waitForApolloRequestRender
();
findPagination
().
vm
.
$emit
(
'
next
'
);
expect
(
resolver
).
toHaveBeenCalledWith
(
expect
.
objectContaining
({
after
:
tagsPageInfo
.
endCursor
}),
);
});
it
(
'
fetch previous page when user clicks prev
'
,
async
()
=>
{
const
resolver
=
jest
.
fn
().
mockResolvedValue
(
imageTagsMock
());
mountComponent
({
resolver
});
await
waitForApolloRequestRender
();
findPagination
().
vm
.
$emit
(
'
prev
'
);
expect
(
resolver
).
toHaveBeenCalledWith
(
expect
.
objectContaining
({
first
:
null
,
before
:
tagsPageInfo
.
startCursor
}),
);
});
});
describe
(
'
loading state
'
,
()
=>
{
it
.
each
`
isImageLoading | queryExecuting | loadingVisible
${
true
}
|
${
true
}
|
${
true
}
${
true
}
|
${
false
}
|
${
true
}
${
false
}
|
${
true
}
|
${
true
}
${
false
}
|
${
false
}
|
${
false
}
`
(
'
when the isImageLoading is $isImageLoading, and is $queryExecuting that the query is still executing is $loadingVisible that the loader is shown
'
,
async
({
isImageLoading
,
queryExecuting
,
loadingVisible
})
=>
{
mountComponent
({
propsData
:
{
isImageLoading
,
isMobile
:
false
,
id
:
1
}
});
if
(
!
queryExecuting
)
{
await
waitForApolloRequestRender
();
}
expect
(
findTagsLoader
().
exists
()).
toBe
(
loadingVisible
);
expect
(
findTagsListRow
().
exists
()).
toBe
(
!
loadingVisible
);
expect
(
findListTitle
().
exists
()).
toBe
(
!
loadingVisible
);
expect
(
findPagination
().
exists
()).
toBe
(
!
loadingVisible
);
},
);
});
});
spec/frontend/registry/explorer/mock_data.js
View file @
9079b29a
...
...
@@ -161,6 +161,20 @@ export const tagsMock = [
},
];
export
const
imageTagsMock
=
(
nodes
=
tagsMock
)
=>
({
data
:
{
containerRepository
:
{
id
:
containerRepositoryMock
.
id
,
tags
:
{
nodes
,
pageInfo
:
{
...
tagsPageInfo
},
__typename
:
'
ContainerRepositoryTagConnection
'
,
},
__typename
:
'
ContainerRepositoryDetails
'
,
},
},
});
export
const
graphQLImageDetailsMock
=
(
override
)
=>
({
data
:
{
containerRepository
:
{
...
...
spec/frontend/registry/explorer/pages/details_spec.js
View file @
9079b29a
...
...
@@ -28,12 +28,10 @@ import Tracking from '~/tracking';
import
{
graphQLImageDetailsMock
,
graphQLImageDetailsEmptyTagsMock
,
graphQLDeleteImageRepositoryTagsMock
,
containerRepositoryMock
,
graphQLEmptyImageDetailsMock
,
tagsMock
,
tagsPageInfo
,
}
from
'
../mock_data
'
;
import
{
DeleteModal
}
from
'
../stubs
'
;
...
...
@@ -72,12 +70,6 @@ describe('Details Page', () => {
await
wrapper
.
vm
.
$nextTick
();
};
const
tagsArrayToSelectedTags
=
(
tags
)
=>
tags
.
reduce
((
acc
,
c
)
=>
{
acc
[
c
.
name
]
=
true
;
return
acc
;
},
{});
const
mountComponent
=
({
resolver
=
jest
.
fn
().
mockResolvedValue
(
graphQLImageDetailsMock
()),
mutationResolver
=
jest
.
fn
().
mockResolvedValue
(
graphQLDeleteImageRepositoryTagsMock
),
...
...
@@ -138,12 +130,6 @@ describe('Details Page', () => {
expect
(
findTagsList
().
exists
()).
toBe
(
false
);
});
it
(
'
does not show pagination
'
,
()
=>
{
mountComponent
();
expect
(
findPagination
().
exists
()).
toBe
(
false
);
});
});
describe
(
'
when the image does not exist
'
,
()
=>
{
...
...
@@ -167,34 +153,6 @@ describe('Details Page', () => {
});
});
describe
(
'
when the list of tags is empty
'
,
()
=>
{
const
resolver
=
jest
.
fn
().
mockResolvedValue
(
graphQLImageDetailsEmptyTagsMock
);
it
(
'
has the empty state
'
,
async
()
=>
{
mountComponent
({
resolver
});
await
waitForApolloRequestRender
();
expect
(
findEmptyState
().
exists
()).
toBe
(
true
);
});
it
(
'
does not show the loader
'
,
async
()
=>
{
mountComponent
({
resolver
});
await
waitForApolloRequestRender
();
expect
(
findTagsLoader
().
exists
()).
toBe
(
false
);
});
it
(
'
does not show the list
'
,
async
()
=>
{
mountComponent
({
resolver
});
await
waitForApolloRequestRender
();
expect
(
findTagsList
().
exists
()).
toBe
(
false
);
});
});
describe
(
'
list
'
,
()
=>
{
it
(
'
exists
'
,
async
()
=>
{
mountComponent
();
...
...
@@ -211,7 +169,6 @@ describe('Details Page', () => {
expect
(
findTagsList
().
props
()).
toMatchObject
({
isMobile
:
false
,
tags
:
cleanTags
,
});
});
...
...
@@ -224,7 +181,7 @@ describe('Details Page', () => {
await
waitForApolloRequestRender
();
[
tagToBeDeleted
]
=
cleanTags
;
findTagsList
().
vm
.
$emit
(
'
delete
'
,
{
[
tagToBeDeleted
.
name
]:
true
}
);
findTagsList
().
vm
.
$emit
(
'
delete
'
,
[
tagToBeDeleted
]
);
});
it
(
'
open the modal
'
,
async
()
=>
{
...
...
@@ -244,7 +201,7 @@ describe('Details Page', () => {
await
waitForApolloRequestRender
();
findTagsList
().
vm
.
$emit
(
'
delete
'
,
tagsArrayToSelectedTags
(
cleanTags
)
);
findTagsList
().
vm
.
$emit
(
'
delete
'
,
cleanTags
);
});
it
(
'
open the modal
'
,
()
=>
{
...
...
@@ -260,61 +217,6 @@ describe('Details Page', () => {
});
});
describe
(
'
pagination
'
,
()
=>
{
it
(
'
exists
'
,
async
()
=>
{
mountComponent
();
await
waitForApolloRequestRender
();
expect
(
findPagination
().
exists
()).
toBe
(
true
);
});
it
(
'
is hidden when there are no more pages
'
,
async
()
=>
{
mountComponent
({
resolver
:
jest
.
fn
().
mockResolvedValue
(
graphQLImageDetailsEmptyTagsMock
)
});
await
waitForApolloRequestRender
();
expect
(
findPagination
().
exists
()).
toBe
(
false
);
});
it
(
'
is wired to the correct pagination props
'
,
async
()
=>
{
mountComponent
();
await
waitForApolloRequestRender
();
expect
(
findPagination
().
props
()).
toMatchObject
({
hasNextPage
:
tagsPageInfo
.
hasNextPage
,
hasPreviousPage
:
tagsPageInfo
.
hasPreviousPage
,
});
});
it
(
'
fetch next page when user clicks next
'
,
async
()
=>
{
const
resolver
=
jest
.
fn
().
mockResolvedValue
(
graphQLImageDetailsMock
());
mountComponent
({
resolver
});
await
waitForApolloRequestRender
();
findPagination
().
vm
.
$emit
(
'
next
'
);
expect
(
resolver
).
toHaveBeenCalledWith
(
expect
.
objectContaining
({
after
:
tagsPageInfo
.
endCursor
}),
);
});
it
(
'
fetch previous page when user clicks prev
'
,
async
()
=>
{
const
resolver
=
jest
.
fn
().
mockResolvedValue
(
graphQLImageDetailsMock
());
mountComponent
({
resolver
});
await
waitForApolloRequestRender
();
findPagination
().
vm
.
$emit
(
'
prev
'
);
expect
(
resolver
).
toHaveBeenCalledWith
(
expect
.
objectContaining
({
first
:
null
,
before
:
tagsPageInfo
.
startCursor
}),
);
});
});
describe
(
'
modal
'
,
()
=>
{
it
(
'
exists
'
,
async
()
=>
{
mountComponent
();
...
...
@@ -349,7 +251,7 @@ describe('Details Page', () => {
});
describe
(
'
when one item is selected to be deleted
'
,
()
=>
{
it
(
'
calls apollo mutation with the right parameters
'
,
async
()
=>
{
findTagsList
().
vm
.
$emit
(
'
delete
'
,
{
[
cleanTags
[
0
].
name
]:
true
}
);
findTagsList
().
vm
.
$emit
(
'
delete
'
,
[
cleanTags
[
0
]]
);
await
wrapper
.
vm
.
$nextTick
();
...
...
@@ -363,7 +265,7 @@ describe('Details Page', () => {
describe
(
'
when more than one item is selected to be deleted
'
,
()
=>
{
it
(
'
calls apollo mutation with the right parameters
'
,
async
()
=>
{
findTagsList
().
vm
.
$emit
(
'
delete
'
,
{
...
tagsArrayToSelectedTags
(
tagsMock
)
}
);
findTagsList
().
vm
.
$emit
(
'
delete
'
,
tagsMock
);
await
wrapper
.
vm
.
$nextTick
();
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment