Skip to content
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

Refactored groups service #313

Merged
merged 37 commits into from
Nov 7, 2023

Ensure the 'import' option is set to true when callbacks are passed t…

3fa882a
Select commit
Loading
Failed to load commit list.
Merged

Refactored groups service #313

Ensure the 'import' option is set to true when callbacks are passed t…
3fa882a
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Nov 6, 2023 in 1s

36.74% of diff hit (target 67.68%)

View this Pull Request on Codecov

36.74% of diff hit (target 67.68%)

Annotations

Check warning on line 30 in app/controllers/groups-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/groups-controller.js#L29-L30

Added lines #L29 - L30 were not covered by tests

Check warning on line 64 in app/controllers/groups-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/groups-controller.js#L52-L64

Added lines #L52 - L64 were not covered by tests

Check warning on line 74 in app/controllers/groups-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/groups-controller.js#L73-L74

Added lines #L73 - L74 were not covered by tests

Check warning on line 86 in app/controllers/groups-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/groups-controller.js#L80-L86

Added lines #L80 - L86 were not covered by tests

Check warning on line 131 in app/controllers/groups-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/groups-controller.js#L131

Added line #L131 was not covered by tests

Check warning on line 139 in app/controllers/groups-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/groups-controller.js#L137-L139

Added lines #L137 - L139 were not covered by tests

Check warning on line 148 in app/controllers/groups-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/groups-controller.js#L148

Added line #L148 was not covered by tests

Check warning on line 156 in app/controllers/groups-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/groups-controller.js#L154-L156

Added lines #L154 - L156 were not covered by tests

Check warning on line 174 in app/controllers/groups-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/groups-controller.js#L172-L174

Added lines #L172 - L174 were not covered by tests

Check warning on line 7 in app/repository/collection-bundles.repository.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/repository/collection-bundles.repository.js#L2-L7

Added lines #L2 - L7 were not covered by tests

Check warning on line 7 in app/repository/notes.repository.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/repository/notes.repository.js#L2-L7

Added lines #L2 - L7 were not covered by tests

Check warning on line 8 in app/repository/references.repository.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/repository/references.repository.js#L2-L8

Added lines #L2 - L8 were not covered by tests

Check warning on line 23 in app/repository/teams-repository.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/repository/teams-repository.js#L23

Added line #L23 was not covered by tests

Check warning on line 27 in app/repository/teams-repository.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/repository/teams-repository.js#L27

Added line #L27 was not covered by tests

Check warning on line 57 in app/repository/user-accounts-repository.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/repository/user-accounts-repository.js#L57

Added line #L57 was not covered by tests

Check warning on line 50 in app/services/_base.service.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/services/_base.service.js#L49-L50

Added lines #L49 - L50 were not covered by tests

Check warning on line 60 in app/services/_base.service.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/services/_base.service.js#L56-L60

Added lines #L56 - L60 were not covered by tests

Check warning on line 66 in app/services/_base.service.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/services/_base.service.js#L66

Added line #L66 was not covered by tests

Check warning on line 75 in app/services/_base.service.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/services/_base.service.js#L70-L75

Added lines #L70 - L75 were not covered by tests

Check warning on line 79 in app/services/_base.service.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/services/_base.service.js#L79

Added line #L79 was not covered by tests

Check warning on line 95 in app/services/_base.service.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/services/_base.service.js#L91-L95

Added lines #L91 - L95 were not covered by tests

Check warning on line 105 in app/services/_base.service.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/services/_base.service.js#L105

Added line #L105 was not covered by tests

Check warning on line 112 in app/services/_base.service.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/services/_base.service.js#L109-L112

Added lines #L109 - L112 were not covered by tests

Check warning on line 126 in app/services/_base.service.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/services/_base.service.js#L126

Added line #L126 was not covered by tests

Check warning on line 134 in app/services/_base.service.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/services/_base.service.js#L130-L134

Added lines #L130 - L134 were not covered by tests