Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Submit feedback
Contribute to GitLab
Sign in
Toggle navigation
C
coopwire-hypothesis
Project
Project
Details
Activity
Releases
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
孙灵跃 Leon Sun
coopwire-hypothesis
Commits
d86d2ba1
Unverified
Commit
d86d2ba1
authored
May 10, 2019
by
Robert Knight
Committed by
GitHub
May 10, 2019
Browse files
Options
Browse Files
Download
Plain Diff
Merge pull request #1111 from hypothesis/add-clear-groups
Add abililty to clear groups in store
parents
2ddb5369
ecb43f74
Changes
2
Show whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
33 additions
and
0 deletions
+33
-0
groups.js
src/sidebar/store/modules/groups.js
+14
-0
groups-test.js
src/sidebar/store/modules/test/groups-test.js
+19
-0
No files found.
src/sidebar/store/modules/groups.js
View file @
d86d2ba1
...
@@ -55,10 +55,23 @@ const update = {
...
@@ -55,10 +55,23 @@ const update = {
groups
:
action
.
groups
,
groups
:
action
.
groups
,
};
};
},
},
CLEAR_GROUPS
()
{
return
{
focusedGroupId
:
null
,
groups
:
[],
};
},
};
};
const
actions
=
util
.
actionTypes
(
update
);
const
actions
=
util
.
actionTypes
(
update
);
function
clearGroups
()
{
return
{
type
:
actions
.
CLEAR_GROUPS
,
};
}
/**
/**
* Set the current focused group.
* Set the current focused group.
*
*
...
@@ -184,6 +197,7 @@ module.exports = {
...
@@ -184,6 +197,7 @@ module.exports = {
actions
:
{
actions
:
{
focusGroup
,
focusGroup
,
loadGroups
,
loadGroups
,
clearGroups
,
},
},
selectors
:
{
selectors
:
{
allGroups
,
allGroups
,
...
...
src/sidebar/store/modules/test/groups-test.js
View file @
d86d2ba1
...
@@ -127,6 +127,25 @@ describe('sidebar.store.modules.groups', () => {
...
@@ -127,6 +127,25 @@ describe('sidebar.store.modules.groups', () => {
});
});
});
});
describe
(
'clearGroups'
,
()
=>
{
it
(
'clears the list of groups'
,
()
=>
{
store
.
loadGroups
([
publicGroup
]);
store
.
clearGroups
();
assert
.
equal
(
store
.
getState
().
groups
.
length
,
0
);
});
it
(
'clears the focused group id'
,
()
=>
{
store
.
loadGroups
([
publicGroup
]);
store
.
focusGroup
(
publicGroup
.
id
);
store
.
clearGroups
();
assert
.
equal
(
store
.
getState
().
focusedGroupId
,
null
);
});
});
describe
(
'allGroups'
,
()
=>
{
describe
(
'allGroups'
,
()
=>
{
it
(
'returns all groups'
,
()
=>
{
it
(
'returns all groups'
,
()
=>
{
store
.
loadGroups
([
publicGroup
,
privateGroup
]);
store
.
loadGroups
([
publicGroup
,
privateGroup
]);
...
...
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